fix(Icon): added aria-hidden attribute to icon

Semantic-Org · Dec 8, 2016 · e521bdf · e521bdf
1 parent 5a1b82d
commit e521bdf
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/elements/Icon/Icon.js b/src/elements/Icon/Icon.js
@@ -45,7 +45,7 @@ function Icon(props) {
   const ElementType = getElementType(Icon, props)
 
   return (
-    <ElementType {...rest} className={classes} />
+    <ElementType {...rest} aria-hidden='true' className={classes} />
   )
 }
 

diff --git a/test/specs/elements/Icon/Icon-test.js b/test/specs/elements/Icon/Icon-test.js
@@ -28,4 +28,12 @@ describe('Icon', () => {
     shallow(<Icon />)
       .should.have.tagName('i')
   })
+
+  describe('aria', () => {
+    it('should add aria-hidden to icon', () => {
+      const wrapper = shallow(<Icon />)
+
+      wrapper.should.have.prop('aria-hidden', 'true')
+    })
+  })
 })