Merge pull request #405 from appuniversum/fix/aria-hidden-icons

Use aria-hidden=true as default on icons
appuniversum · Jul 18, 2023 · d75d28e · d75d28e
2 parents 3ecd203 + 22eeffa
commit d75d28e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/addon/components/au-icon.hbs b/addon/components/au-icon.hbs
@@ -2,9 +2,9 @@
   role="img"
   class="au-c-icon au-c-icon--{{@icon}} {{this.alignment}} {{this.size}}"
   aria-hidden="{{if
-    (or (eq @ariaHidden true) (eq @ariaHidden 'true'))
-    'true'
+    (or (eq @ariaHidden false) (eq @ariaHidden 'false'))
     'false'
+    'true'
   }}"
   ...attributes
 >

diff --git a/stories/5-components/Brand/AuIcon.stories.js b/stories/5-components/Brand/AuIcon.stories.js
@@ -54,7 +54,7 @@ Component.args = {
   icon: 'info-circle',
   size: 'large',
   alignment: '',
-  ariaHidden: false,
+  ariaHidden: true,
 };
 
 export const Icons = IconList.bind({});