ID-48 Fix mobile menu button click handler

id-sk · May 18, 2023 · e23378c · e23378c
1 parent eba9799
commit e23378c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,6 +1,8 @@
 node_modules/
 npm-debug.log
 .DS_Store
+.idea
+.vscode
 preview/
 tmp/
 axeReports/

diff --git a/src/idsk/components/header/header.js b/src/idsk/components/header/header.js
@@ -46,11 +46,11 @@ Header.prototype.handleClick = function (event) {
 
   // If a button with aria-controls, handle click
   if ($toggleButton && $target) {
-    this.toggleClass($target, 'govuk-header__navigation--open')
-    this.toggleClass($toggleButton, 'govuk-header__menu-button--open')
+    this.toggleClass($target, 'idsk-header__navigation--open')
+    this.toggleClass($toggleButton, 'idsk-header__menu-button--open')
 
     $toggleButton.setAttribute('aria-expanded', $toggleButton.getAttribute('aria-expanded') !== 'true')
-    $target.setAttribute('aria-hidden', $target.getAttribute('aria-hidden') === 'false')
+    $target.setAttribute('aria-hidden', ($target.getAttribute('aria-hidden') === 'false').toString())
   }
 }