search

FriendsOfREDAXO / consent_manager

Stellt ein Opt-In Cookie Banner zur Verfügung
MIT License
57 stars 15 forks source link

Elements with role="dialog" or role="alertdialog" do not have accessible names. #304

Open iceman-fx opened 1 year ago

iceman-fx commented 1 year ago

1

IngoWinter commented 1 year ago

https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/dialog_role https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/alertdialog_role

Normalerweise sollte es reichen, dem Element mit der role dialog eine Beschreibung über aria-label mitzugeben. Ich frage mich, ob der das div für den Hintergrund überhaupt eine role haben sollte. Aus dem Bauch raus würde ich nur dem consent_manager_wrapper die role dialog (oder sogar alertdialog) geben. Das Hintergrund div sollte eigentlich für den Screenreader unsichtbar sein.

aeberhard commented 1 year ago

Ich werde das mal für die Version 5 berücksichtigen!