From 34c926691d78480a58662fd3f19f2f42b5fab951 Mon Sep 17 00:00:00 2001
From: Yannick Armand <yannick.armand@protonmail.com>
Date: Thu, 8 Feb 2024 21:42:14 +0100
Subject: [PATCH] Accessibility: add some "aria-label"

---
 modules/ps_customersignin/ps_customersignin.tpl        | 3 ++-
 modules/ps_imageslider/views/templates/hook/slider.tpl | 7 +++----
 modules/ps_shoppingcart/ps_shoppingcart.tpl            | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/modules/ps_customersignin/ps_customersignin.tpl b/modules/ps_customersignin/ps_customersignin.tpl
index caf0d1d51..f6737c3b5 100644
--- a/modules/ps_customersignin/ps_customersignin.tpl
+++ b/modules/ps_customersignin/ps_customersignin.tpl
@@ -14,7 +14,8 @@
           id="userMenuButton"
           data-bs-toggle="dropdown"
           aria-haspopup="true"
-          aria-expanded="false">
+          aria-expanded="false"
+          aria-label="{l s='View my account (%s)' d='Shop.Theme.Customeraccount' sprintf=[$customerName]}">
           <i class="material-icons header-block__icon" aria-hidden="true">&#xE7FD;</i>
           <span class="header-block__title d-lg-inline d-none">{$customerName|truncate:22:"..":true}</span>
         </a>
diff --git a/modules/ps_imageslider/views/templates/hook/slider.tpl b/modules/ps_imageslider/views/templates/hook/slider.tpl
index 9edbe9a80..b7aaef844 100644
--- a/modules/ps_imageslider/views/templates/hook/slider.tpl
+++ b/modules/ps_imageslider/views/templates/hook/slider.tpl
@@ -33,13 +33,12 @@
         {/foreach}
       </div>
 
-      <button class="carousel-control-prev" type="button" data-bs-target="#home-slider .carousel" data-bs-slide="prev">
+      <button class="carousel-control-prev" type="button" data-bs-target="#home-slider .carousel" data-bs-slide="prev" aria-label="{l s='Previous' d='Shop.Theme.Actions'}">
         <span class="carousel-control-prev-icon" aria-hidden="true"></span>
-        <span class="visually-hidden">{l s='Previous' d='Shop.Theme.Actions'}</span>
       </button>
-      <button class="carousel-control-next" type="button" data-bs-target="#home-slider .carousel" data-bs-slide="next">
+
+      <button class="carousel-control-next" type="button" data-bs-target="#home-slider .carousel" data-bs-slide="next" aria-label="{l s='Next' d='Shop.Theme.Actions'}">
         <span class="carousel-control-next-icon" aria-hidden="true"></span>
-        <span class="visually-hidden">{l s='Next' d='Shop.Theme.Actions'}</span>
       </button>
     </div>
   </div>
diff --git a/modules/ps_shoppingcart/ps_shoppingcart.tpl b/modules/ps_shoppingcart/ps_shoppingcart.tpl
index 3c793e919..5c6aa7aed 100644
--- a/modules/ps_shoppingcart/ps_shoppingcart.tpl
+++ b/modules/ps_shoppingcart/ps_shoppingcart.tpl
@@ -5,7 +5,7 @@
 <div id="_desktop_cart">
   <div class="header-block blockcart cart-preview {if $cart.products_count> 0}header-block--active{else}inactive{/if}" data-refresh-url="{$refresh_url}">
     {if $cart.products_count> 0}
-      <a class="header-block__action-btn" rel="nofollow" href="{$cart_url}">
+      <a class="header-block__action-btn" rel="nofollow" href="{$cart_url}" aria-label="{l s='View cart (%d products)' d='Shop.Theme.Checkout' sprintf=[$cart.products_count]}">
     {else}
       <span class="header-block__action-btn">
     {/if}