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

docs SVGs: Allow passing a custom title. #28660

Merged
merged 3 commits into from Apr 29, 2019
Merged

Conversation

XhmikosR
Copy link
Member

No description provided.

@XhmikosR XhmikosR added this to Inbox in v5 via automation Apr 15, 2019
@patrickhlauke
Copy link
Member

patrickhlauke commented Apr 15, 2019

Noting that as they're aria-hidden, even the current title won't be announced, as we generally set the label/name on whatever uses those icons

[edit: correction, SOME of them are aria-hidden only. so yeh, as you were...]

@XhmikosR
Copy link
Member Author

Well, the thing is that only 3 of them have aria-hidden=true and secondly, the title is useful for hovering.

See https://deploy-preview-28449--twbs-bootstrap.netlify.com/ after hovering on the Toggle Dark Mode navbar button

Not sure if we should have aria-hidden on all of them, I leave this stuff to you :)

@XhmikosR
Copy link
Member Author

Ping @MartijnCuppens @Johann-S

v5 automation moved this from Inbox to Approved Apr 29, 2019
@XhmikosR XhmikosR merged commit 952c3d3 into master Apr 29, 2019
v5 automation moved this from Approved to Shipped Apr 29, 2019
@XhmikosR XhmikosR deleted the master-xmr-docs-svg-title branch April 29, 2019 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5
  
Shipped
Development

Successfully merging this pull request may close these issues.

None yet

3 participants