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

New color picker improvements, second round #2699

Closed
afercia opened this issue Sep 7, 2017 · 15 comments
Closed

New color picker improvements, second round #2699

afercia opened this issue Sep 7, 2017 · 15 comments
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).

Comments

@afercia
Copy link
Contributor

afercia commented Sep 7, 2017

Follow up to #2679

See also Slack discussion https://wordpress.slack.com/archives/C02QB2JS7/p1504633514000516

I'd like to propose to consider a few further improvements for the color picker:

  • add some indication about the currently selected color before the set of buttons: "visually it’s clear, but if you are on a screen reader you may have to go through a number of items before you get to the one selected."
  • ideally, colors should have a human-understandable named color. While the required format is hex or rgb or maybe other that will be added, I guess many users (not just a11y users) would just like to pick up a color. Other applications already refer to colors with a name:

screen_shot_2017-09-05_at_20 03 37

also for color variations:

screen_shot_2017-09-05_at_20 10 40

this could be implemented requiring to specify a color name in gutenberg_color_palette().

@gschoppe
Copy link

gschoppe commented Sep 9, 2017

This change would be useful to agencies, to provide branding hints... e.g. "Primary Brand Color" "Secondary Brand Color", "Green Accent", etc

@afercia
Copy link
Contributor Author

afercia commented Sep 9, 2017

Yep, not to mention accessibility because a string like, for example, #8ed1fc gets read out as:
number eight ed one f c which is not so useful for assistive technology users.

screen shot 2017-09-09 at 12 14 29

@gschoppe
Copy link

gschoppe commented Sep 9, 2017

Sorry, yes, I meant that as "in addition to a11y concerns, it would assist agencies"

@afercia
Copy link
Contributor Author

afercia commented Sep 9, 2017

Sure 🙂 And your point makes perfectly sense. I haven't thought at that and seems to me one more good reason to consider named colors.

@gschoppe
Copy link

gschoppe commented Sep 9, 2017

Just a side note, thank you for the incredible a11y work you've been doing for the team. I work for an agency that often contracts out with an accessibility-oriented firm, and the level of thought that you put into your work is very heartening to see. I'm well versed in how little fun it is to boot up a screen reader and test interfaces over and over again that were designed by sighted developers for mouse-driven interactions.

@afercia
Copy link
Contributor Author

afercia commented Sep 9, 2017

@gschoppe thanks. I share that feeling on a daily basis... 😬

@afercia afercia added the [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). label Sep 11, 2017
@00travelgirl00
Copy link

@afercia okay, now I understand your point with the labels for colors.

How it would work with the open color picker? Or do we only define labels for specific colors?

also how @gschoppe mentioned it would be nice to define own colors with labels.

@gschoppe
Copy link

The label could expand, tool-tip-like on focus/hover (although that often leaves touch users in the cold). and, of course, it needn't be visible for screen readers. However, human-readable, visible color labels would also assist sighted users with any form of color impairment.

@afercia
Copy link
Contributor Author

afercia commented Sep 11, 2017

Gutenberg tooltips work in a way that they use the same text used for the aria-label attribute, so that would work for sighted users and screen reader users. Worth also considering not all screen reader users are completely blind; many users with low vision use screen readers as an aid.

@gschoppe
Copy link

how do gutenberg tooltips perform on touch devices, like phones?

@afercia
Copy link
Contributor Author

afercia commented Sep 11, 2017

@gschoppe I think that's untested and still to address.

@afercia
Copy link
Contributor Author

afercia commented Nov 28, 2017

Re: better identifying colors, apart from giving them a human-understandable named color, here's some feedback from users about giving them also a tooltip, which makes sense to me, see https://make.wordpress.org/accessibility/2017/11/28/screen-reader-user-experience-with-gutenberg/

the user can select colours for text and background. It would be useful to have the name visible on hover. This will be useful for colourblind people.

@afercia
Copy link
Contributor Author

afercia commented Mar 19, 2018

Reported by one of the accessibility testers:

  • What the user hears is the hexadecimal code of the color. It will be better if the name of the color was used instead of the code hex.

screen shot 2018-03-20 at 00 55 36

@jorgefilipecosta
Copy link
Member

Hi all, thank you for the discussion that happened here. The improvements to the color related components were merged. I think we can now close this issue if there is any other improvement needed to feel free to reopen.

@afercia
Copy link
Contributor Author

afercia commented Apr 30, 2018

Thanks everyone for the discussion, feedback, and thanks @jorgefilipecosta very good job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).
Projects
None yet
Development

No branches or pull requests

4 participants