Merge pull request #123 from sowasred2012/fix-aria-hidden

Don't set `aria-hidden` on the `body` tag
canonical · Jul 12, 2021 · e5b2de6 · e5b2de6
2 parents 4d04de8 + f194333
commit e5b2de6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@canonical/cookie-policy",
-  "version": "3.2.0",
+  "version": "3.3.0",
   "description": "A script and style sheet that displays a cookie policy notification",
   "main": "build/js/module.js",
   "iife": "build/js/cookie-policy.js",

diff --git a/src/js/main.js b/src/js/main.js
@@ -15,7 +15,6 @@ export const cookiePolicy = (callback = null) => {
       cookiePolicyContainer = document.createElement('dialog');
       cookiePolicyContainer.classList.add('cookie-policy');
       cookiePolicyContainer.setAttribute('open', true);
-      document.body.setAttribute('aria-hidden', true);
       document.body.appendChild(cookiePolicyContainer);
       const notifiation = new Notification(
         cookiePolicyContainer,
@@ -37,7 +36,6 @@ export const cookiePolicy = (callback = null) => {
       callback();
     }
     document.body.removeChild(cookiePolicyContainer);
-    document.body.removeAttribute('aria-hidden');
     cookiePolicyContainer = null;
   };