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

VoiceOver does not announce the comparision operators in the query editor in a usable way #207

Closed
MRayermannMSFT opened this issue May 22, 2018 · 1 comment
Labels
:accessibility: accessibility Related to keyboard accessibility :accessibility: MAS Required for MAS accessibility compliance 🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Milestone

Comments

@MRayermannMSFT
Copy link
Member

VoiceOver is currently literally announcing the characters in the comparison operators (< > >= <= ==) for the query builder. It should instead be saying things like "greater than", "less than", etc.

Closing this will fix:

@MRayermannMSFT MRayermannMSFT added 🪲 bug Issue is not intended behavior :accessibility: accessibility Related to keyboard accessibility :accessibility: MAS Required for MAS accessibility compliance labels May 22, 2018
@MRayermannMSFT MRayermannMSFT added this to the 1.3.0 milestone May 22, 2018
@craxal craxal added the pending label May 25, 2018
@craxal
Copy link
Contributor

craxal commented May 25, 2018

I have a fix ready. VoiceOver still doesn't work perfectly when the select element is expanded. For example, even though "<>" has an aria-label of "not equal", VoiceOver will pronounce this as "less than greater than" when reading from the expanded select element.

@craxal craxal added ✅ merged A fix for this issue has been merged and removed pending labels May 25, 2018
@craxal craxal closed this as completed May 25, 2018
@craxal craxal modified the milestones: 1.3.0, 1.2.0 May 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:accessibility: accessibility Related to keyboard accessibility :accessibility: MAS Required for MAS accessibility compliance 🪲 bug Issue is not intended behavior ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

2 participants