Update notifications twig and container for status role

Resolves #12714
craftcms · Feb 22, 2023 · b5e310e · b5e310e
1 parent df58763
commit b5e310e
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/templates/_layouts/components/notifications.twig b/src/templates/_layouts/components/notifications.twig
@@ -1,4 +1,4 @@
-<div id="notifications" aria-labelledby="cp-notification-heading" role="region">
+<div id="notifications" role="status">
   <h2 id="cp-notification-heading" class="visually-hidden">{{ 'Notifications'|t('app') }}</h2>
 </div>
 

diff --git a/src/web/assets/cp/dist/cp.js b/src/web/assets/cp/dist/cp.js
diff --git a/src/web/assets/cp/dist/cp.js.map b/src/web/assets/cp/dist/cp.js.map
diff --git a/src/web/assets/cp/src/js/CP.js b/src/web/assets/cp/src/js/CP.js
@@ -1467,9 +1467,9 @@ Craft.CP.Notification = Garnish.Base.extend({
       'data-type': this.type,
     }).appendTo(Craft.cp.$notificationContainer);
 
-    const $body = $('<div class="notification-body"/>')
-      .appendTo(this.$container)
-      .attr('role', 'status');
+    const $body = $('<div class="notification-body"/>').appendTo(
+      this.$container
+    );
 
     if (this.settings.icon) {
       const $icon = $('<span/>', {