Merge pull request #248 from plentymarkets/feat/add-missing-html-attr…

…ibutes feat: add missing html attributes
plentymarkets · Nov 29, 2023 · 620b300 · 620b300
2 parents ebb518d + 8a5d305
commit 620b300
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 6 deletions.
diff --git a/apps/web/components/Cookiebar/Cookiebar.vue b/apps/web/components/Cookiebar/Cookiebar.vue
@@ -111,7 +111,7 @@
             class="w-full"
             :aria-disabled="false"
             type="button"
-            aria-label="button"
+            :aria-label="$t('CookieBar.Accept All')"
             @click="setAllCookiesState(true)"
           >
             {{ $t('CookieBar.Accept All') }}
@@ -122,14 +122,21 @@
             class="w-full"
             :aria-disabled="false"
             type="button"
-            aria-label="button"
+            :aria-label="$t('CookieBar.Reject All')"
             @click="setAllCookiesState(false)"
           >
             {{ $t('CookieBar.Reject All') }}
           </SfButton>
         </div>
         <div class="flex-1">
-          <SfButton variant="secondary" class="w-full" :aria-disabled="false" type="button" @click="setConsent()">
+          <SfButton
+            variant="secondary"
+            class="w-full"
+            :aria-disabled="false"
+            type="button"
+            :aria-label="$t('CookieBar.Accept Selection')"
+            @click="setConsent()"
+          >
             {{ $t('CookieBar.Accept Selection') }}
           </SfButton>
         </div>
@@ -140,7 +147,7 @@
       v-else
       variant="secondary"
       class="z-10 fixed bottom-2 xl:bottom-2 xl:left-auto xl:right-2 bg-white"
-      aria-label="Cookie control"
+      :aria-label="$t('CookieBar.Cookie Settings')"
       @click="changeVisibilityState"
     >
       <SfIconCheckBox />

diff --git a/apps/web/lang/de.json b/apps/web/lang/de.json
@@ -31,6 +31,7 @@
     "asterixHint": "* markierte Zustimmungen sind erforderlich"
   },
   "CookieBar": {
+    "Cookie Settings": "Cookie-Einstellungen",
     "Privacy Settings": "Datenschutzeinstellungen",
     "More information": "Mehr Informationen",
     "Further Settings": "Weitere Einstellungen",

diff --git a/apps/web/lang/en.json b/apps/web/lang/en.json
@@ -31,6 +31,7 @@
     "asterixHint": "* marked consents are required"
   },
   "CookieBar": {
+    "Cookie Settings": "Cookie Settings",
     "Privacy Settings": "Privacy Settings",
     "More information": "More information",
     "Further Settings": "Further Settings",

diff --git a/apps/web/pages/index.vue b/apps/web/pages/index.vue
@@ -2,7 +2,11 @@
   <div class="relative min-h-[600px]">
     <picture>
       <source srcset="/images/homepage-hero-bg.webp" media="(min-width: 768px)" />
-      <img src="/images/homepage-hero-bg-mobile.webp" class="absolute w-full h-full z-[-1] md:object-cover" />
+      <img
+        src="/images/homepage-hero-bg-mobile.webp"
+        alt="Hero mobile background"
+        class="absolute w-full h-full z-[-1] md:object-cover"
+      />
     </picture>
     <div class="md:flex md:flex-row-reverse md:justify-center max-w[1536px] mx-auto md:min-h-[600px]">
       <div class="flex flex-col md:basis-2/4 md:items-stretch md:overflow-hidden">

diff --git a/docs/changelog/changelog_de.md b/docs/changelog/changelog_de.md
@@ -13,6 +13,7 @@
 ### Geändert
 
 - Die Cookiebar Logik wurde überarbeitet.
+- Die ARIA-Labels von Schaltflächen der Cookiebar wurden übersetzt.
 - Die Sprachauswahl wurde überarbeitet und zeigt jetzt Länderflaggen an.
 - Die Checkout-Seite wurde überarbeitet, um weniger Daten zu laden.
 
@@ -22,7 +23,7 @@
 - Beim Unterschreiten der Mindestkaufpreisanforderung wurde beim Bezahlvorgang kein Fehler ausgegeben.
 - UI Benachrichtigungen führten zu einer Layoutverschiebung für den Benutzer.
 - Kategorien wurden in der falschen Sprache geladen, wenn die Website zum ersten Mal mit SSR gerendert wurde.
-– Die Variationsauswahl hat die Variation nicht aus der URL entfernt, wenn die Basisvariation ausgewählt wurde.
+  – Die Variationsauswahl hat die Variation nicht aus der URL entfernt, wenn die Basisvariation ausgewählt wurde.
 - Produkte auf der Bestellbestätigungsseite haben nicht auf das korrekte Produkt verlinkt.
 
 ## v1.1.0 (03.11.2023) <a href="https://github.com/plentymarkets/plentyshop-pwa/compare/v1.0.0...v1.1.0" target="_blank" rel="noopener"><b>Übersicht aller Änderungen</b></a>

diff --git a/docs/changelog/changelog_en.md b/docs/changelog/changelog_en.md
@@ -13,6 +13,7 @@
 ### Changed
 
 - Reworked logic of cookie bar.
+- Added ARIA label translations for cookie bar buttons.
 - The language select has been refactored and is now showing country flags.
 - Refactored the checkout page to load less data.