Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(Suggestion) Enhancing Web Accessibility #336

Open
wherehows opened this issue Jan 17, 2024 · 0 comments
Open

(Suggestion) Enhancing Web Accessibility #336

wherehows opened this issue Jan 17, 2024 · 0 comments

Comments

@wherehows
Copy link

For better accessibility, the current design involves passing aria-role and aria-live through ariaProps. However, since aria-role already functions as aria-live, it could be beneficial to eliminate ariaProps and structure it to allow passing only ariaRole as a toast option.

Furthermore, referencing MDN reveals a potential double-speaking issue when assigning "alert" to aria-role and "assertive" to aria-live. To address this, it is recommended to set aria-live to "polite" for compatibility when aria-role is "status." Conversely, when aria-role is "alert," it is advisable not to assign a separate aria-live. Consider integrating these considerations into the code.

For more information on the aforementioned points, you can refer to ARIA live regions - Accessibility | MDN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant