[aria-hidden="true"] elements contain focusable descendents

[emanuelechiari]

I got this advice from pagespeed…
[aria-hidden=“true”] elements contain focusable descendents
It refer to a simple header template with logo on the left and nav on the right with one link that open a modal (contacts) with some details inside…
link of the page is
https://dev.marcellomontarsi.com/cwi24/
Inside there is also the button “REEL” that open a video inside a lightbox where I can not find a solution for give to the light box video the height and the width
Thanks

[Marius]

Accessibility issue with modal close container

Description: the a-element a.cc-mdl-close which is used to close modals results in this error message (see screenshot) because it seems to be focusable. Screenshots, screen recording, code snippet [Bildschirmfoto 2023-07-04 um 19.41.42] Environment info WordPress version: latest Cwicly Plugin version: latest

Pagespeed doesn't seem to like the modal close overlay

Hi, I’m not a screen reader expert, not even a user, but Pagespeed doesn’t seem to like the modal close overlay to be aria-hidden. Is there something we could do to improve this? What is strange is that the mentionned overlay <a> tag looks empty in the page source code. [image] [image]

Accessibility query regarding modal block

I have used the ‘modal’ block a number of times on a page, with a trigger set on a element - which seems to me to be semantically correct for this use case. I see in the modal markup that there is an empty anchor link rendered <a href="#" class="cc-mdl-close" aria-hidden="true"></a>. This getting picked up in various accessibility testing tools as erroneous (Wave, Google Page Insights, WP Equalize Digital testing plugin) - and seems un-needed. Indeed, if I remove it from the DOM using devtoo…

[Graphnic]

Yes, it’s a confusing experience for screenreader users. I hope the team can revisit this sometime soon.

FYI I ended up using Lity (Lity - Lightweight, accessible and responsive lightbox) for video lightboxes as it doesn’t strip the closed captions from YouTube embeds like the Cwicly one does (also an accessibility problem). It works well with a little custom CSS, but does mean you have to load jQuery on the front end. (see Closed Captions / Subtitles button missing in YouTube lightbox - #7 by Graphnic)