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

[ISSUE#1896][MAS1.3.1][Screen Reader-Connect the bot] Incorrect position of the controls are announced by voiceover for the buttons available at the top most menu bar. #1935

Merged
merged 3 commits into from Oct 22, 2019

Conversation

denscollo
Copy link
Contributor

Solves #1896

Description

Fixes how voiceover announces the position of the editor menu bar's buttons controls

Changes made

We added an aria-hidden attribute to the separator bar between the restart conversation button and the sub-menu button. This way voiceover skips this element which was causing the problem.

Additionally, we noticed that in windows, the last element of the menu bar was not being announced as 3/3. To solve this we removed a div around the save transcript button and moved the role label menuitem inside it.

Testing

In the next image, you can see the voiceover reading the order of the elements as intended.

asda

@coveralls
Copy link

coveralls commented Oct 21, 2019

Coverage Status

Coverage remained the same at 66.949% when pulling b1c0239 on fix/incorrect-position-announced into 2717295 on master.

tonyanziano
tonyanziano previously approved these changes Oct 22, 2019
Copy link
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Add a changelog entry to CHANGELOG.md and I'll merge it in. :)

@denscollo
Copy link
Contributor Author

Thanks @tonyanziano ! I justed pushed the updated changelog

@tonyanziano tonyanziano merged commit ffa96e8 into master Oct 22, 2019
@tonyanziano tonyanziano deleted the fix/incorrect-position-announced branch October 22, 2019 17:33
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

Successfully merging this pull request may close these issues.

None yet

4 participants