From eeea8b06519e14b5d290ec19b3abbb59334e86d4 Mon Sep 17 00:00:00 2001
From: Abhiram Haridas <abhiram.haridas@gmail.com>
Date: Sat, 8 Dec 2018 23:40:49 +0530
Subject: [PATCH] feat(Button) : Add aria-pressed attribute for toggle button

---
 src/elements/Button/Button.js             |  2 ++
 test/specs/elements/Button/Button-test.js | 18 ++++++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/src/elements/Button/Button.js b/src/elements/Button/Button.js
index e837ada865..26f3f1df09 100644
--- a/src/elements/Button/Button.js
+++ b/src/elements/Button/Button.js
@@ -268,6 +268,7 @@ class Button extends Component {
           {labelPosition === 'left' && labelElement}
           <button
             className={buttonClasses}
+            aria-pressed={toggle ? (!!active) : undefined}
             disabled={disabled}
             ref={this.handleRef}
             tabIndex={tabIndex}
@@ -287,6 +288,7 @@ class Button extends Component {
       <ElementType
         {...rest}
         className={classes}
+        aria-pressed={toggle ? (!!active) : undefined}
         disabled={(disabled && ElementType === 'button') || undefined}
         onClick={this.handleClick}
         ref={this.handleRef}
diff --git a/test/specs/elements/Button/Button-test.js b/test/specs/elements/Button/Button-test.js
index 579460b58e..818e037bce 100644
--- a/test/specs/elements/Button/Button-test.js
+++ b/test/specs/elements/Button/Button-test.js
@@ -113,6 +113,24 @@ describe('Button', () => {
     })
   })
 
+  describe('toggle', () => {
+    it('is not set by default', () => {
+      shallow(<Button />).should.not.have.prop('toggle')
+    })
+
+    it('should have aria-pressed', () => {
+      shallow(<Button toggle />).should.have.prop('aria-pressed')
+    })
+
+    it('aria-pressed should be true when active', () => {
+      shallow(<Button toggle active />).should.have.prop('aria-pressed', true)
+    })
+
+    it('aria-pressed should be false when inactive', () => {
+      shallow(<Button toggle />).should.have.prop('aria-pressed', false)
+    })
+  })
+
   describe('icon', () => {
     it('adds className icon', () => {
       shallow(<Button icon='user' />).should.have.className('icon')