From 0c0e28dd35b9f0ca78f858ab2e26e1cd05a0b5bc Mon Sep 17 00:00:00 2001 From: Andrea Martz <14165788+andreamartz@users.noreply.github.com> Date: Fri, 29 Jan 2021 16:51:18 -0600 Subject: [PATCH] Add missing aria label for campaign close button (#12489) --- .../javascripts/initializers/initializeHeroBannerClose.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/app/assets/javascripts/initializers/initializeHeroBannerClose.js b/app/assets/javascripts/initializers/initializeHeroBannerClose.js index 6b8426671..063bc9928 100644 --- a/app/assets/javascripts/initializers/initializeHeroBannerClose.js +++ b/app/assets/javascripts/initializers/initializeHeroBannerClose.js @@ -3,10 +3,12 @@ function initializeHeroBannerClose() { let bannerWrapper = document.getElementById('hero-html-wrapper'); let closeIcon = document.getElementById('js-hero-banner__x'); + // Currently js-hero-banner__x button icon ID needs to be written into the abstract html // In the future this could be extracted so the implementer doesn't need to worry about it. if (bannerWrapper && closeIcon) { + closeIcon.setAttribute('aria-label', 'Close campaign banner'); closeIcon.addEventListener('click', () => { localStorage.setItem('exited_hero', bannerWrapper.dataset.name); bannerWrapper.style.display = 'none';