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

Global sprite should have a aria-hidden="true" attribute #90

Open
zhell opened this issue Jun 2, 2023 · 1 comment
Open

Global sprite should have a aria-hidden="true" attribute #90

zhell opened this issue Jun 2, 2023 · 1 comment

Comments

@zhell
Copy link

zhell commented Jun 2, 2023

Hi, I really love the plugin, I've been using it on almost all my front-end from scratch projects.
On thing I noticed, is in terms of accessibility, the sprite appended to the page should have an aria-hidden="true" attribute, preventing the screen reader's ability to focus on it, even if it's positioned outside of the screen via css.

@beakid
Copy link

beakid commented Jul 5, 2023

Came here to say the same! 👍

EDIT: A temporary solution is to add
#__svg__icons__dom__ { display: none; }

to your css file.

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

2 participants