Fix various issues reported on the Trello (#341)

* Fix various issues reported on the Trello

* Remove accidental additions
This commit is contained in:
Emma 2022-01-12 11:37:20 -05:00 committed by GitHub
parent a2266adb3f
commit b5f438aa0c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
11 changed files with 64 additions and 58 deletions

View File

@ -54,7 +54,7 @@
> >
<MoonIcon v-if="$colorMode.value === 'light'" /> <MoonIcon v-if="$colorMode.value === 'light'" />
<SunIcon v-else /> <SunIcon v-else />
Change Theme Change theme
</button> </button>
<div <div
v-if="$auth.user" v-if="$auth.user"
@ -92,7 +92,7 @@
<li class="hide-desktop"> <li class="hide-desktop">
<NuxtLink to="/create/project"> <NuxtLink to="/create/project">
<PlusIcon /> <PlusIcon />
<span>Create Project</span> <span>Create project</span>
</NuxtLink> </NuxtLink>
</li> </li>
<li> <li>
@ -112,13 +112,13 @@
<button @click="changeTheme"> <button @click="changeTheme">
<MoonIcon v-if="$colorMode.value === 'light'" /> <MoonIcon v-if="$colorMode.value === 'light'" />
<SunIcon v-else /> <SunIcon v-else />
<span>Change Theme</span> <span>Change theme</span>
</button> </button>
</li> </li>
<li> <li>
<button @click="logout"> <button @click="logout">
<LogOutIcon /> <LogOutIcon />
<span>Log Out</span> <span>Log out</span>
</button> </button>
</li> </li>
<hr class="hide-desktop" /> <hr class="hide-desktop" />
@ -184,7 +184,7 @@
<nuxt-link to="/settings/privacy"> <nuxt-link to="/settings/privacy">
<button class="iconified-button"> <button class="iconified-button">
<ShieldIcon /> <ShieldIcon />
Set privacy settings Privacy settings
</button> </button>
</nuxt-link> </nuxt-link>
<button class="iconified-button" @click="changeTheme"> <button class="iconified-button" @click="changeTheme">

View File

@ -137,7 +137,7 @@
:disabled="(currentMember.permissions & EDIT_DETAILS) !== EDIT_DETAILS" :disabled="(currentMember.permissions & EDIT_DETAILS) !== EDIT_DETAILS"
> >
<TrashIcon /> <TrashIcon />
Reset icon Reset
</button> </button>
</section> </section>
<section class="card game-sides"> <section class="card game-sides">
@ -508,7 +508,7 @@ export default {
this.MANAGE_INVITES = 1 << 4 this.MANAGE_INVITES = 1 << 4
this.REMOVE_MEMBER = 1 << 5 this.REMOVE_MEMBER = 1 << 5
this.EDIT_MEMBER = 1 << 6 this.EDIT_MEMBER = 1 << 6
this.DELETE_MOD = 1 << 7 this.DELETE_PROJECT = 1 << 7
}, },
methods: { methods: {
async saveProjectReview() { async saveProjectReview() {

View File

@ -424,8 +424,8 @@ export default {
}, },
showPopup() { showPopup() {
if ( if (
(this.currentMember.permissions & this.DELETE_MOD) === (this.currentMember.permissions & this.DELETE_PROJECT) ===
this.DELETE_MOD this.DELETE_PROJECT
) { ) {
this.$refs.delete_popup.show() this.$refs.delete_popup.show()
} }

View File

@ -218,7 +218,13 @@
:hide-selected="true" :hide-selected="true"
placeholder="Choose mod loaders..." placeholder="Choose mod loaders..."
/> />
<p v-else class="value">{{ version.loaders.join(',') }}</p> <p v-else class="value">
{{
version.loaders
.map((x) => x.charAt(0).toUpperCase() + x.slice(1))
.join(', ')
}}
</p>
</div> </div>
<div v-if="mode === 'version'" class="data"> <div v-if="mode === 'version'" class="data">
<p class="title">Downloads</p> <p class="title">Downloads</p>
@ -251,7 +257,13 @@
:hide-selected="true" :hide-selected="true"
placeholder="Choose versions..." placeholder="Choose versions..."
/> />
<p v-else class="value">{{ version.game_versions.join(',') }}</p> <p v-else class="value">
{{
version.game_versions
.map((x) => x.charAt(0).toUpperCase() + x.slice(1))
.join(', ')
}}
</p>
</div> </div>
<div v-if="mode === 'version'" class="data"> <div v-if="mode === 'version'" class="data">
<p class="title">Published</p> <p class="title">Published</p>
@ -363,7 +375,7 @@
<input <input
v-model="newDependencyId" v-model="newDependencyId"
type="text" type="text"
:placeholder="`Enter the ${dependencyAddMode} id...`" :placeholder="`Enter the ${dependencyAddMode} ID...`"
/> />
<Multiselect <Multiselect
v-model="newDependencyType" v-model="newDependencyType"

View File

@ -117,7 +117,7 @@
" "
> >
<TrashIcon /> <TrashIcon />
Reset icon Reset
</button> </button>
</section> </section>
<section class="card game-sides"> <section class="card game-sides">
@ -225,8 +225,20 @@
> >
<td>{{ version.version_title }}</td> <td>{{ version.version_title }}</td>
<td>{{ version.version_number }}</td> <td>{{ version.version_number }}</td>
<td>{{ version.loaders.join(', ') }}</td> <td>
<td>{{ version.game_versions.join(', ') }}</td> {{
version.loaders
.map((x) => x.charAt(0).toUpperCase() + x.slice(1))
.join(', ')
}}
</td>
<td>
{{
version.game_versions
.map((x) => x.charAt(0).toUpperCase() + x.slice(1))
.join(', ')
}}
</td>
<td> <td>
<VersionBadge <VersionBadge
v-if="version.release_channel === 'release'" v-if="version.release_channel === 'release'"
@ -393,7 +405,7 @@
<input <input
v-model="newDependencyId" v-model="newDependencyId"
type="text" type="text"
:placeholder="`Enter the ${dependencyAddMode} id...`" :placeholder="`Enter the ${dependencyAddMode} ID...`"
/> />
<Multiselect <Multiselect
v-model="newDependencyType" v-model="newDependencyType"
@ -506,7 +518,7 @@
@click="gallery.push({})" @click="gallery.push({})"
> >
<PlusIcon /> <PlusIcon />
Add a image Add an image
</button> </button>
</div> </div>
<div v-for="(item, index) in gallery" :key="index" class="gallery-item"> <div v-for="(item, index) in gallery" :key="index" class="gallery-item">
@ -554,14 +566,14 @@
" "
> >
<TrashIcon /> <TrashIcon />
Reset image Reset
</button> </button>
</div> </div>
</div> </div>
<div class="buttons"> <div class="buttons">
<button class="iconified-button" @click="gallery.splice(index, 1)"> <button class="iconified-button" @click="gallery.splice(index, 1)">
<TrashIcon /> <TrashIcon />
Remove Gallery Image Remove
</button> </button>
<hr v-if="gallery.length > 0 && index !== gallery.length - 1" /> <hr v-if="gallery.length > 0 && index !== gallery.length - 1" />

View File

@ -2,14 +2,14 @@
<div class="page-container"> <div class="page-container">
<Popup v-if="currentProject" :show-popup="true"> <Popup v-if="currentProject" :show-popup="true">
<div class="moderation-popup"> <div class="moderation-popup">
<h2>Moderation Form</h2> <h2>Moderation form</h2>
<p> <p>
Both of these fields are optional, but can be used to communicate Both of these fields are optional, but can be used to communicate
problems with a project team members. The body supports markdown problems with a project's team members. The body supports markdown
formatting! formatting!
</p> </p>
<div class="status"> <div class="status">
<span>New Project Status: </span> <span>New project status: </span>
<Badge <Badge
v-if="currentProject.newStatus === 'approved'" v-if="currentProject.newStatus === 'approved'"
color="green" color="green"
@ -18,6 +18,7 @@
<Badge <Badge
v-else-if=" v-else-if="
currentProject.newStatus === 'processing' || currentProject.newStatus === 'processing' ||
currentProject.newStatus === 'unlisted' ||
currentProject.newStatus === 'archived' currentProject.newStatus === 'archived'
" "
color="yellow" color="yellow"
@ -59,7 +60,7 @@
@click="saveProject" @click="saveProject"
> >
<CheckIcon /> <CheckIcon />
Save project status Confirm
</button> </button>
</div> </div>
</div> </div>
@ -298,7 +299,7 @@ export default {
<style lang="scss" scoped> <style lang="scss" scoped>
.moderation-popup { .moderation-popup {
width: 480px; width: auto;
padding: var(--spacing-card-md) var(--spacing-card-lg); padding: var(--spacing-card-md) var(--spacing-card-lg);
.status { .status {

View File

@ -6,7 +6,7 @@
<h1 v-else>Settings</h1> <h1 v-else>Settings</h1>
<div class="card styled-tabs"> <div class="card styled-tabs">
<nuxt-link v-if="$auth.user" class="tab" to="/settings" exact <nuxt-link v-if="$auth.user" class="tab" to="/settings" exact
><span>Profile</span></nuxt-link ><span>General</span></nuxt-link
> >
<nuxt-link v-if="$auth.user" class="tab" to="/settings/follows"> <nuxt-link v-if="$auth.user" class="tab" to="/settings/follows">
<span>Followed projects</span> <span>Followed projects</span>

View File

@ -20,7 +20,7 @@
" "
> >
<TrashIcon /> <TrashIcon />
Reset icon Reset
</button> </button>
</div> </div>
<div class="recap"> <div class="recap">
@ -92,11 +92,10 @@
</label> </label>
<h3>Theme</h3> <h3>Theme</h3>
<label> <label>
<span <span>
>Change the global site theme of Modrinth. You can switch it here or Change the global site theme. It can also be changed between light
anywhere by accessing the theme switcher in the navigation bar and dark in the navigation bar.
dropdown.</span </span>
>
<Multiselect <Multiselect
v-model="$colorMode.preference" v-model="$colorMode.preference"
:options="['light', 'dark', 'oled']" :options="['light', 'dark', 'oled']"
@ -145,14 +144,14 @@ export default {
} }
}, },
fetch() { fetch() {
this.$emit('update:action-button', 'Save profile settings') this.$emit('update:action-button', 'Save')
this.$emit('update:action-button-callback', this.saveChanges) this.$emit('update:action-button-callback', this.saveChanges)
}, },
head: { head: {
title: 'Settings - Modrinth', title: 'Settings - Modrinth',
}, },
created() { created() {
this.$emit('update:action-button', 'Save profile settings') this.$emit('update:action-button', 'Save')
this.$emit('update:action-button-callback', this.saveChanges) this.$emit('update:action-button-callback', this.saveChanges)
}, },
methods: { methods: {

View File

@ -1,6 +1,6 @@
<template> <template>
<div class="rows card"> <div class="rows card">
<div class="consent-container"> <div class="privacy-settings-container">
<div> <div>
Modrinth relies on different providers and in-house tools to allow us to Modrinth relies on different providers and in-house tools to allow us to
provide custom-tailored experiences and personalized advertising. You provide custom-tailored experiences and personalized advertising. You
@ -32,10 +32,10 @@
</div> </div>
<div class="actions"> <div class="actions">
<button class="iconified-button" @click="toggleAll(false)"> <button class="iconified-button" @click="toggleAll(false)">
Refuse All Select none
</button> </button>
<button class="iconified-button" @click="toggleAll(true)"> <button class="iconified-button" @click="toggleAll(true)">
Accept All Select all
</button> </button>
</div> </div>
</div> </div>
@ -54,7 +54,7 @@ export default {
} }
}, },
fetch() { fetch() {
this.$emit('update:action-button', 'Confirm my choices') this.$emit('update:action-button', 'Confirm')
this.$emit('update:action-button-callback', this.confirm) this.$emit('update:action-button-callback', this.confirm)
this.$store.dispatch('consent/loadFromCookies', this.$cookies) this.$store.dispatch('consent/loadFromCookies', this.$cookies)
@ -77,7 +77,7 @@ export default {
title: 'Privacy Settings - Modrinth', title: 'Privacy Settings - Modrinth',
}, },
created() { created() {
this.$emit('update:action-button', 'Confirm my choices') this.$emit('update:action-button', 'Confirm')
this.$emit('update:action-button-callback', this.confirm) this.$emit('update:action-button-callback', this.confirm)
}, },
options: { options: {
@ -132,7 +132,7 @@ export default {
} }
} }
.consent-container { .privacy-settings-container {
.divider { .divider {
margin-top: 1rem; margin-top: 1rem;
} }

View File

@ -48,7 +48,7 @@
This cannot be reversed, so be careful!</span This cannot be reversed, so be careful!</span
> >
<input <input
value="Delete Account" value="Delete account"
type="button" type="button"
class="iconified-button" class="iconified-button"
@click="$refs.delete_popup.show()" @click="$refs.delete_popup.show()"

View File

@ -47,13 +47,6 @@
<span class="primary-stat__label">downloads</span> <span class="primary-stat__label">downloads</span>
</div> </div>
</div> </div>
<div class="stats-block__item primary-stat">
<HeartIcon class="primary-stat__icon" />
<div class="primary-stat__text">
<span class="primary-stat__counter">{{ sumFollows() }}</span>
<span class="primary-stat__label">followers of projects</span>
</div>
</div>
</div> </div>
</aside> </aside>
</div> </div>
@ -130,7 +123,6 @@ import Advertisement from '~/components/ads/Advertisement'
import ReportIcon from '~/assets/images/utils/report.svg?inline' import ReportIcon from '~/assets/images/utils/report.svg?inline'
import SunriseIcon from '~/assets/images/utils/sunrise.svg?inline' import SunriseIcon from '~/assets/images/utils/sunrise.svg?inline'
import DownloadIcon from '~/assets/images/utils/download-alt.svg?inline' import DownloadIcon from '~/assets/images/utils/download-alt.svg?inline'
import HeartIcon from '~/assets/images/utils/heart.svg?inline'
import SettingsIcon from '~/assets/images/utils/settings.svg?inline' import SettingsIcon from '~/assets/images/utils/settings.svg?inline'
import PlusIcon from '~/assets/images/utils/plus.svg?inline' import PlusIcon from '~/assets/images/utils/plus.svg?inline'
import UpToDate from '~/assets/images/illustrations/up_to_date.svg?inline' import UpToDate from '~/assets/images/illustrations/up_to_date.svg?inline'
@ -143,7 +135,6 @@ export default {
SunriseIcon, SunriseIcon,
DownloadIcon, DownloadIcon,
ReportIcon, ReportIcon,
HeartIcon,
Badge, Badge,
SettingsIcon, SettingsIcon,
PlusIcon, PlusIcon,
@ -243,15 +234,6 @@ export default {
sum += projects.downloads sum += projects.downloads
} }
return this.formatNumber(sum)
},
sumFollows() {
let sum = 0
for (const projects of this.projects) {
sum += projects.followers
}
return this.formatNumber(sum) return this.formatNumber(sum)
}, },
}, },