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

bug(matTooltip): Add option to set aria-labelledby rather than aria-describedby #23284

Open
kseamon opened this issue Jul 30, 2021 · 6 comments
Labels
Accessibility This issue is related to accessibility (a11y) area: material/tooltip feature This issue represents a new feature or feature request rather than a bug or bug fix P4 A relatively minor issue that is not relevant to core functions

Comments

@kseamon
Copy link
Collaborator

kseamon commented Jul 30, 2021

Reproduction

matTooltip is only able to apply aria-describedby to its host element. This is not optimal for host elements that have no aria-label or text contents.

Expected Behavior

What behavior were you expecting to see?

For a button with no textual label, eg a MatIconButton, a tooltip typically provides a label rather than a description.
Having matTooltip apply aria-labeledby rather than would be better in these cases.

At the very least, it would be good to have an Input to configure matTooltip's labeledby vs describedby preference.
Extra credit: change the default for obvious cases like MatIconButton.

Actual Behavior

What behavior did you actually see?

matTooltip applies aria-describedby, with no option to change this behavior.

Environment

  • Angular: 12
  • CDK/Material: 12
  • Browser(s): any
  • Operating System (e.g. Windows, macOS, Ubuntu): any
@kseamon kseamon added the needs triage This issue needs to be triaged by the team label Jul 30, 2021
@kseamon
Copy link
Collaborator Author

kseamon commented Jul 30, 2021

Related to internal ticket # 195133734

@SalemlocalAAA

This comment has been minimized.

@SalemlocalAAA

This comment has been minimized.

@SalemlocalAAA

This comment has been minimized.

@jelbourn jelbourn added Accessibility This issue is related to accessibility (a11y) feature This issue represents a new feature or feature request rather than a bug or bug fix P4 A relatively minor issue that is not relevant to core functions area: material/tooltip and removed needs triage This issue needs to be triaged by the team labels Aug 2, 2021
@Splaktar Splaktar changed the title bug(matTooltip): Add option to set aria-labeledby rather than aria-describedby bug(matTooltip): Add option to set aria-labelledby rather than aria-describedby Aug 2, 2021
@angular-robot
Copy link
Contributor

angular-robot bot commented Mar 13, 2022

Just a heads up that we kicked off a community voting process for your feature request. There are 20 days until the voting process ends.

Find more details about Angular's feature request process in our documentation.

@angular-robot
Copy link
Contributor

angular-robot bot commented Apr 1, 2022

Thank you for submitting your feature request! Looks like during the polling process it didn't collect a sufficient number of votes to move to the next stage.

We want to keep Angular rich and ergonomic and at the same time be mindful about its scope and learning journey. If you think your request could live outside Angular's scope, we'd encourage you to collaborate with the community on publishing it as an open source package.

You can find more details about the feature request process in our documentation.

@zarend zarend self-assigned this Jan 26, 2023
@zarend zarend removed their assignment Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility This issue is related to accessibility (a11y) area: material/tooltip feature This issue represents a new feature or feature request rather than a bug or bug fix P4 A relatively minor issue that is not relevant to core functions
Projects
None yet
Development

No branches or pull requests

5 participants