feat(theme-default): remove ari-hidden on title and use an empty alt …

…on brand image if not specified
nruffing · Jan 28, 2024 · 4250903 · 4250903
1 parent 2979bc4
commit 4250903
Showing 1 changed file with 1 addition and 9 deletions.
diff --git a/themes/theme-default/src/client/components/NavbarBrand.vue b/themes/theme-default/src/client/components/NavbarBrand.vue
@@ -24,14 +24,7 @@ const navbarBrandLogo = computed(() => {
   }
   return themeLocale.value.logo
 })
-const navbarBrandLogoAlt = computed(
-  () => themeLocale.value.logoAlt ?? navbarBrandTitle.value,
-)
-const navBarLogoAltMatchesTitle = computed(
-  () =>
-    navbarBrandTitle.value.toLocaleUpperCase().trim() ===
-    navbarBrandLogoAlt.value.toLocaleUpperCase().trim(),
-)
+const navbarBrandLogoAlt = computed(() => themeLocale.value.logoAlt ?? '')
 const NavbarBrandLogo: FunctionalComponent = () => {
   if (!navbarBrandLogo.value) return null
   const img = h('img', {
@@ -56,7 +49,6 @@ const NavbarBrandLogo: FunctionalComponent = () => {
       v-if="navbarBrandTitle"
       class="site-name"
       :class="{ 'can-hide': navbarBrandLogo }"
-      :aria-hidden="navBarLogoAltMatchesTitle"
     >
       {{ navbarBrandTitle }}
     </span>