Wrap notification icon in button to make it accessible #580

bkd-mba-fbi · Jul 26, 2023 · c465e29 · c465e29
1 parent 7c54598
commit c465e29
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/...pp/my-notifications/components/my-notifications-show/my-notifications-show.component.html b/...pp/my-notifications/components/my-notifications-show/my-notifications-show.component.html
@@ -6,10 +6,12 @@
     isAuthenticated: (isAuthenticated$ | async)
   } as data"
 >
-  <div
+  <button
+    type="button"
+    [attr.aria-label]="'my-notifications.title' | translate"
     (click)="toggleNotificationsPopup()"
     id="notifications-button"
-    class="btn notifications-button p-0 m-0"
+    class="btn btn-link notifications-button p-0 m-0"
   >
     <span
       class="notifications-bell"
@@ -40,7 +42,7 @@
       "
       >{{ data.notificationsCount }}</span
     >
-  </div>
+  </button>
   <div
     class="notifications-popup border"
     id="notifications-popup"