diff --git a/packages/govuk-frontend/src/govuk/components/service-navigation/service-navigation.mjs b/packages/govuk-frontend/src/govuk/components/service-navigation/service-navigation.mjs index b96ce04d35..5fdfa8d736 100644 --- a/packages/govuk-frontend/src/govuk/components/service-navigation/service-navigation.mjs +++ b/packages/govuk-frontend/src/govuk/components/service-navigation/service-navigation.mjs @@ -49,12 +49,13 @@ export class ServiceNavigation extends Component { return this } - const menuId = $menuButton.getAttribute('aria-controls') + const menuId = $menuButton.getAttribute('data-aria-controls') + if (!menuId) { throw new ElementError({ component: ServiceNavigation, identifier: - 'Navigation button (` {% endif %} diff --git a/packages/govuk-frontend/src/govuk/components/service-navigation/template.test.js b/packages/govuk-frontend/src/govuk/components/service-navigation/template.test.js index 4babd7144c..9d8e2bb689 100644 --- a/packages/govuk-frontend/src/govuk/components/service-navigation/template.test.js +++ b/packages/govuk-frontend/src/govuk/components/service-navigation/template.test.js @@ -85,7 +85,7 @@ describe('Service Navigation', () => { const navId = $nav.attr('id') expect(navId).toBe('navigation') - expect($navToggle.attr('aria-controls')).toBe(navId) + expect($navToggle.attr('data-aria-controls')).toBe(navId) }) it('omits empty items from the navigation', () => { @@ -158,7 +158,7 @@ describe('Service Navigation', () => { const navId = $nav.attr('id') expect(navId).toBe('main-nav') - expect($navToggle.attr('aria-controls')).toBe(navId) + expect($navToggle.attr('data-aria-controls')).toBe(navId) }) })