From 5b5159088f460cfe138a8a7f1adb001a9f6acd16 Mon Sep 17 00:00:00 2001 From: Colin Burn-Murdoch Date: Mon, 21 Jun 2021 10:40:36 -0400 Subject: [PATCH] Apply aria-expanded to the trigger, not the expandable area aria-controls is already set --- lib/assets/javascripts/_modules/collapsible-navigation.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/assets/javascripts/_modules/collapsible-navigation.js b/lib/assets/javascripts/_modules/collapsible-navigation.js index 2c03c662..3e3214ac 100644 --- a/lib/assets/javascripts/_modules/collapsible-navigation.js +++ b/lib/assets/javascripts/_modules/collapsible-navigation.js @@ -49,11 +49,11 @@ function toggleHeading ($topLevelItem) { var isOpen = $topLevelItem.hasClass('is-open') var $heading = $topLevelItem.find('> a') - var $body = $topLevelItem.find('.collapsible__body') + var $button = $topLevelItem.find('.collapsible__toggle') var $toggleLabel = $topLevelItem.find('.collapsible__toggle-label') $topLevelItem.toggleClass('is-open', !isOpen) - $body.attr('aria-expanded', isOpen ? 'false' : 'true') + $button.attr('aria-expanded', isOpen ? 'false' : 'true') $toggleLabel.text(isOpen ? 'Expand ' + $heading.text() : 'Collapse ' + $heading.text()) }