MDL-67663 core: Do not use aria-hidden="false" on drawer

From https://www.w3.org/TR/wai-aria-1.1/#aria-hidden:
* "aria-hidden="false" is known to work inconsistently in browsers"

diff --git a/lib/amd/build/drawer.min.js b/lib/amd/build/drawer.min.js
index 0f035c1..141e339 100644 (file)
Binary files a/lib/amd/build/drawer.min.js and b/lib/amd/build/drawer.min.js differ

diff --git a/lib/amd/build/drawer.min.js.map b/lib/amd/build/drawer.min.js.map
index 9f97f5b..267e20a 100644 (file)
Binary files a/lib/amd/build/drawer.min.js.map and b/lib/amd/build/drawer.min.js.map differ

diff --git a/lib/amd/src/drawer.js b/lib/amd/src/drawer.js
index a24cd30..3c77606 100644 (file)
--- a/lib/amd/src/drawer.js
+++ b/lib/amd/src/drawer.js
@@ -32,7 +32,7 @@ import DrawerEvents from 'core/drawer_events';
 const show = (root) => {
     root.removeClass('hidden');
     root.attr('aria-expanded', true);
-    root.attr('aria-hidden', false);
+    root.removeAttr('aria-hidden');
     root.focus();
 
     PubSub.publish(DrawerEvents.DRAWER_SHOWN, root);

diff --git a/lib/templates/drawer.mustache b/lib/templates/drawer.mustache
index e21fd2f..4daa4df 100644 (file)
--- a/lib/templates/drawer.mustache
+++ b/lib/templates/drawer.mustache
@@ -35,7 +35,7 @@
     id="{{$drawerid}}drawer-{{uniqid}}{{/drawerid}}"
     class="{{$drawerclasses}}{{/drawerclasses}} drawer bg-white {{^show}}hidden{{/show}}"
     aria-expanded="{{#show}}true{{/show}}{{^show}}false{{/show}}"
-    aria-hidden="{{#show}}false{{/show}}{{^show}}true{{/show}}"
+    {{^show}}aria-hidden="true"{{/show}}
     data-region="right-hand-drawer"
     role="region"
     tabindex="-1"