Skip to content

docs(multiple): adds screen reader friendly tooltip to hamburger icon #31746

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

Merged

Conversation

essjay05
Copy link
Contributor

Updates dev-app layout hamburger icon button to use a matTooltip similarly to other icon config buttons in the header. This adds a visible tooltip for sighted users as well as an announced screen reader value when focusing on the hamburger button.

Fixes b/435497550

@angular-robot angular-robot bot added the area: docs Related to the documentation label Aug 20, 2025
@essjay05 essjay05 marked this pull request as ready for review August 20, 2025 22:19
@essjay05 essjay05 requested a review from a team as a code owner August 20, 2025 22:19
@essjay05 essjay05 requested review from mmalerba and andrewseguin and removed request for a team August 20, 2025 22:19
@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Aug 21, 2025
@essjay05 essjay05 force-pushed the add-sr-announced-label-hamburger-icon-btn branch 2 times, most recently from c2d4082 to 5f146f7 Compare August 21, 2025 17:23
Updates dev-app layout hamburger icon button to use a matTooltip similarly to other icon config
buttons in the header. This adds a visible tooltip for sighted users as well as an announced
screen reader value when focusing on the hamburger button.

Fixes b/435497550
@essjay05 essjay05 force-pushed the add-sr-announced-label-hamburger-icon-btn branch from 5f146f7 to 1f97fb6 Compare August 21, 2025 20:50
@wagnermaciel wagnermaciel removed the request for review from andrewseguin August 22, 2025 17:35
@wagnermaciel wagnermaciel added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Aug 22, 2025
@wagnermaciel wagnermaciel merged commit c424a30 into angular:main Aug 22, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants