-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Fix: template editor header area is difficult to navigate with screenreaders #32938
Fix: template editor header area is difficult to navigate with screenreaders #32938
Conversation
Size Change: +20 B (0%) Total Size: 1.04 MB
ℹ️ View Unchanged
|
Code changes look good, I will give it a try and report back. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tested the PR with NVDA and I think it is an improvement that can be merged.
I ran into an edge case when the post has no post title, but will open a new issue for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works correctly, nice work @jorgefilipecosta :
Fixes: #32642
This PR applies some changes to the template title and dropdown menu on the post editor to make it more accessible.
How has this been tested?
I verified that the button to go back to post editor and the button to open the template options are both tabbable and their tab order respects their visual order.