- Issue created by @ksenzee
- πΊπΈUnited States ksenzee Washington state
Here's a (poor quality, low volume) video demonstrating the issue: https://photos.app.goo.gl/Mus1p6y83pDXwdYK9
Here's a (poor quality, higher volume) video demonstrating the fix: https://photos.app.goo.gl/jKvJPoYL7ZzqXMtb9
My colleague https://www.drupal.org/u/jacquelynmarcella β discovered this issue and fixed it, and I made her go get a d.o account because she deserves commit credit. :D
- Status changed to Needs review
over 1 year ago 9:15pm 16 June 2023 - last update
over 1 year ago 3 pass -
mark_fullmer β
committed 0fce39b4 on 8.x-1.x authored by
ksenzee β
Issue #3367357 by ksenzee, jacquelynmarcella: makeElementAccessible()...
-
mark_fullmer β
committed 0fce39b4 on 8.x-1.x authored by
ksenzee β
- Status changed to Fixed
over 1 year ago 9:48pm 17 June 2023 - πΊπΈUnited States jacquelynmarcella
mark_fullmer β credited jacquelynmarcella β .
- πΊπΈUnited States mark_fullmer Tucson
Thanks, @jacquelynmarcella , for identifying this and providing an appropriately more narrow targeting of the
aria-hidden
. And thanks, @ksenzee, for conveying this Drupal-ward.The logic change makes sense to me and won't have any negative repercussions for sighted visitors. Committed; marking as Fixed!
Automatically closed - issue fixed for 2 weeks with no activity.