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

[Accessibility] Edit buttons in Management Advanced Settings provide no context for screen reader #12872

Closed
aphelionz opened this issue Jul 14, 2017 · 6 comments
Labels
bug Fixes for quality problems that affect the customer experience Project:Accessibility

Comments

@aphelionz
Copy link
Contributor

Currently, the edit buttons on the right read "edit" with no indication of what you're editing. They should read "Edit sort options," "edit date format", etc

image

@tbragin tbragin added the bug Fixes for quality problems that affect the customer experience label Jul 28, 2017
@chrisronline
Copy link
Contributor

Thanks for the call out here!

I'm not sure if showing the name of what you're editing will scale, as some of the options are quite long: search:queryLanguage:switcher:enable. Is it viable to only show up to a max width in the UI but leave the full text for the screen reader? Will that work?

@aphelionz
Copy link
Contributor Author

Yep! The visually readable text and the audible text can be presented slightly differently. No problem there.

@chrisronline
Copy link
Contributor

Okay great thanks!

@BigFunger
Copy link
Contributor

@aphelionz: Which would be a better for the aria-label: 'Edit xPack:defaultAdminEmail' or 'Edit x pack default admin email'?

@aphelionz
Copy link
Contributor Author

@BigFunger It depends on what the screen reader says out loud. I appreciate that the former, 'Edit xPack:defaultAdminEmail' is easier to code because it doesn't require transliterating any strings... so if that works, go for it! However if it sounds funky its going to confuse people and needs to be split up.

@BigFunger
Copy link
Contributor

I'm going to post a PR with the strings transliterated. Check it out and post your comments there. Will link soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Project:Accessibility
Projects
None yet
Development

No branches or pull requests

4 participants