From d53ee40ded7e4f99a7956e8a87ebbbdcf39aed5f Mon Sep 17 00:00:00 2001
From: Artur Paikin <artur@arturpaikin.com>
Date: Sun, 3 Apr 2016 23:04:39 -0400
Subject: [PATCH] Test command fix, modal hidden/visible/selected attr refactor

---
 package.json         |  2 +-
 src/plugins/Modal.js | 20 ++++----------------
 src/scss/_modal.scss |  6 +++---
 3 files changed, 8 insertions(+), 20 deletions(-)

diff --git a/package.json b/package.json
index 35193c2a8..3659bfeb4 100644
--- a/package.json
+++ b/package.json
@@ -11,7 +11,7 @@
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "test": "npm run lint && npm run test:unit",
-    "test:phantom": "zuul test/spec/upload.js --phantom",
+    "test:acceptance": "node test/multipart.spec.js",
     "test:unit": "node test/index.js",
     "build": "npm run build:lib && npm run build:bundle && npm run build:bundle:min && npm run build:css",
     "build:js": "npm run build:lib && npm run build:bundle && npm run build:bundle:min",
diff --git a/src/plugins/Modal.js b/src/plugins/Modal.js
index 4e7eb8fc6..ea71d9f86 100644
--- a/src/plugins/Modal.js
+++ b/src/plugins/Modal.js
@@ -60,15 +60,6 @@ export default class Modal extends Plugin {
     const modal = this.core.getState().modal
     modal.targets.push(target)
     this.core.setState({modal: modal})
-
-    // this.core.emitter.emit('modal-add-target', {
-    //   id: callerPluginId,
-    //   name: callerPluginName,
-    //   icon: callerPluginIcon,
-    //   type: callerPluginType,
-    //   el: el,
-    //   isVisible: false
-    // })
   }
 
   render (state) {
@@ -85,7 +76,7 @@ export default class Modal extends Plugin {
     })
 
     return yo`<div class="UppyModal"
-                   ${state.modal.isVisible ? '' : 'aria-hidden'}
+                   aria-hidden="${state.modal.isVisible ? 'false' : 'true'}"
                    aria-labelledby="modalTitle"
                    aria-describedby="modalDescription"
                    role="dialog">
@@ -102,10 +93,8 @@ export default class Modal extends Plugin {
               <button class="UppyModalTab-btn"
                       role="tab"
                       aria-controls="${target.id}"
-                      ${target.isVisible ? 'aria-selected' : ''}
-                      onclick=${() => {
-                        this.showTabPanel(target.id)
-                      }}}>
+                      aria-selected="${target.isVisible ? 'true' : 'false'}"
+                      onclick=${this.showTabPanel.bind(this, target.id)}>
                 ${target.icon}
                 <span class="UppyModalTab-name">${target.name}</span>
               </button>
@@ -119,7 +108,7 @@ export default class Modal extends Plugin {
             return yo`<div class="UppyModalContent-panel
                            ${this.opts.panelSelectorPrefix}--${target.id}"
                            role="tabpanel"
-                           ${target.isVisible ? '' : 'aria-hidden'}>
+                           aria-hidden="${target.isVisible ? 'false' : 'true'}">
               ${target.el}
             </div>`
           })}
@@ -129,7 +118,6 @@ export default class Modal extends Plugin {
             return target.el
           })}
         </div>
-
       </div>
     </div>`
   }
diff --git a/src/scss/_modal.scss b/src/scss/_modal.scss
index ea97b829f..23b4da3a1 100644
--- a/src/scss/_modal.scss
+++ b/src/scss/_modal.scss
@@ -6,7 +6,7 @@
   z-index: $zIndex-2;
 }
 
-.UppyModal[aria-hidden] {
+.UppyModal[aria-hidden=true] {
   display: none;
 }
 
@@ -101,7 +101,7 @@
   }
 }
 
-.UppyModalTab-btn[aria-selected] {
+.UppyModalTab-btn[aria-selected=true] {
   background-color: $color-cornflower-blue;
 }
 
@@ -128,7 +128,7 @@
   height: 100%;
 }
 
-.UppyModalContent-panel[aria-hidden] {
+.UppyModalContent-panel[aria-hidden=true] {
   display: none;
 }