Fix missing a11y labels on project page buttons

modrinth · Aug 24, 2024 · 9fd0f68 · 9fd0f68
1 parent 58dac27
commit 9fd0f68
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/apps/frontend/src/pages/[type]/[id].vue b/apps/frontend/src/pages/[type]/[id].vue
@@ -506,16 +506,17 @@
                 <button
                   v-if="auth.user"
                   v-tooltip="following ? `Unfollow` : `Follow`"
+                  :aria-label="following ? `Unfollow` : `Follow`"
                   @click="userFollowProject(project)"
                 >
                   <HeartIcon :fill="following ? 'currentColor' : 'none'" aria-hidden="true" />
                 </button>
-                <nuxt-link v-else v-tooltip="'Follow'" to="/auth/sign-in">
+                <nuxt-link v-else v-tooltip="'Follow'" to="/auth/sign-in" aria-label="Follow">
                   <HeartIcon aria-hidden="true" />
                 </nuxt-link>
               </ButtonStyled>
               <ButtonStyled size="large" circular>
-                <PopoutMenu v-if="auth.user" v-tooltip="'Save'" from="top-right">
+                <PopoutMenu v-if="auth.user" v-tooltip="'Save'" from="top-right" aria-label="Save">
                   <BookmarkIcon
                     aria-hidden="true"
                     :fill="
@@ -556,7 +557,7 @@
                     </button>
                   </template>
                 </PopoutMenu>
-                <nuxt-link v-else v-tooltip="'Save'" to="/auth/sign-in">
+                <nuxt-link v-else v-tooltip="'Save'" to="/auth/sign-in" aria-label="Save">
                   <BookmarkIcon aria-hidden="true" />
                 </nuxt-link>
               </ButtonStyled>