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

Improve Custom Color Button #5258

Closed
karmatosed opened this issue Feb 26, 2018 · 22 comments
Closed

Improve Custom Color Button #5258

karmatosed opened this issue Feb 26, 2018 · 22 comments
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.
Milestone

Comments

@karmatosed
Copy link
Member

karmatosed commented Feb 26, 2018

From a recent meetup there was feedback that the color picker wasn't that obvious for everyone. Right now we just show the rainbow circle:

color-picker

A suggestion was to add the 'droplet picker' icon, props to @rickybanister, but we could also add a droplet itself. I have done 2 versions and I think the droplet works well as a suggestion but visually easy to work out.

color-picker-obvious

What do people think? Does this make it more obvious? Is this enough or should we do more? What else could we do?

@karmatosed karmatosed added the Needs Design Feedback Needs general design feedback. label Feb 26, 2018
@aristath
Copy link
Member

The picker does make it a lot more obvious and seeing it I know right away what it does.

@rickybanister
Copy link

I also prefer the picker!

This looks so much better than it did in my head, nice work @karmatosed

@joanrho
Copy link
Contributor

joanrho commented Feb 26, 2018

Agreed on picker. Thanks @rickybanister and @karmatosed!

@Rahmon
Copy link
Contributor

Rahmon commented Feb 26, 2018

I prefer the picker too.

@carlhancock
Copy link

I prefer the droplet picker icon in the first example and agree it would make the UI better.

What would be even more ideal is when using the color picker the rainbow background then changed to the color that they picked. Although this makes things complicated as you would then need to know based on the custom color if the droplet picker icon should then be displayed using a dark or light color.

But the UI already warns you if the color combination may be hard for people to read so the detection to know if the droplet picker icon should be light or dark based on the custom color selected appears to already be available.

@maddisondesigns
Copy link

+1 for picker icon over droplet

@rileybrook
Copy link

Picker over droplet for me

@karmatosed
Copy link
Member Author

karmatosed commented Mar 5, 2018

Picker seems the winner, let's mark this to be developed! Thanks everyone.

2018-03-05 at 21 37

@karmatosed karmatosed added Design Needs Dev Ready for, and needs developer efforts and removed Needs Design Feedback Needs general design feedback. labels Mar 5, 2018
@karmatosed karmatosed added this to the 2.4 milestone Mar 5, 2018
faishal added a commit to faishal/gutenberg that referenced this issue Mar 6, 2018
…r, and required css fix

Signed-off-by: Faishal Saiyed <[email protected]>
@faishal
Copy link
Contributor

faishal commented Mar 6, 2018

I started working on it.

@karmatosed From where can I get picker dashicon? Should I request it in https://github.com/WordPress/dashicons ?

@jasmussen
Copy link
Contributor

Forgive me for jumping in way after the fact, you have permission to ignore me.

Could we remove the rainbow swatch entirely, perhaps add another theme swatch in that spot, and simply show a blue link below that says "Custom Color" next to the "Clear" link? Seems like it would be easier to implement, and users wouldn't have to interpret the icon.

@karmatosed karmatosed removed the Design label Mar 6, 2018
@karmatosed
Copy link
Member Author

I wouldn't be against that @jasmussen, although my concern is with 2 actions beside each other again of the 'custom color' and 'clear'. With our link styling that could get a little hard to work out which is which.

@faishal it would be requesting this as a dashicon, however right now let's make sure we are going that path. Thank you for stepping up and once we have a route agreed we can help you get your PR committed.

@karmatosed karmatosed added the Needs Design Feedback Needs general design feedback. label Mar 6, 2018
@jasmussen
Copy link
Contributor

I wouldn't be against that @jasmussen, although my concern is with 2 actions beside each other again of the 'custom color' and 'clear'. With our link styling that could get a little hard to work out which is which.

We could use gray buttons perhaps. Also I know how super annoying it is I'm coming in from the sidelines here, I do apologize, though I figure it's easier to test a quick link before drawing a new dashicon.

@karmatosed karmatosed modified the milestones: 2.4, 2.5, Merge Proposal Mar 7, 2018
@karmatosed
Copy link
Member Author

We could use gray buttons perhaps. Also I know how super annoying it is I'm coming in from the sidelines here, I do apologize, though I figure it's easier to test a quick link before drawing a new dashicon.

I'd be up for testing, although I kind of feel link here would be confusing. Up for seeing.

@mtias mtias modified the milestones: Merge Proposal, WordPress 5.0 Mar 7, 2018
@IgorWpserved
Copy link

IgorWpserved commented Apr 6, 2018

Is it possible to set gradient in my theme support for Gutenberg ?
For example like that:

add_theme_support( 'gutenberg', array(
    'colors' => array(
        'linear-gradient(to right, #33ccff 0%, #0000cc 100%)',
    )
) ).

I want to add a few gradients to color picker list.
When I wanted use this code at front-end block hasn't background but when u go to edit saw this:
image

Front-end:
image

Is it possible to change at front-page "background-color" to "background" ?
Or get user to choose how it have to display ?

@jasmussen
Copy link
Contributor

Is it possible to set gradient in my theme support for Gutenberg ?

No, not as part of the core offering. But incidentally I just saw a plugin in development allowing this: https://wordpress.slack.com/archives/C02QB2JS7/p1522937131000291

@karmatosed karmatosed removed the Needs Design Feedback Needs general design feedback. label Apr 10, 2018
@ajitbohra ajitbohra added [Type] Enhancement A suggestion for improvement. Good First Issue An issue that's suitable for someone looking to contribute for the first time labels Jul 8, 2018
@ajitbohra
Copy link
Member

ajitbohra commented Jul 10, 2018

@karmatosed @jasmussen

Want to confirm we are going with adding picker icon for custom color.

If yes can we get the icon added to dashicons? https://github.com/WordPress/dashicons/tree/master/sources/svg/gutenberg

@karmatosed
Copy link
Member Author

Let's go with the picker and yes we'd need to have it added to dashicons.

@mtias mtias changed the title Rainbow color picker isn't obvious to all users Improve Custom Color Button Oct 7, 2018
@mtias mtias added the [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later label Oct 7, 2018
@mtias mtias modified the milestones: WordPress 5.0, Future: Phase 2 Oct 12, 2018
@mapk
Copy link
Contributor

mapk commented Jan 24, 2019

Could we remove the rainbow swatch entirely, perhaps add another theme swatch in that spot, and simply show a blue link below that says "Custom Color" next to the "Clear" link?

Here's how that might look.

color-link

I still prefer the icon on the color swatch, myself.

Now because Gutenberg uses Material icons, can we just go with something like this?

screen shot 2019-01-23 at 7 42 42 pm

which is linked here: https://material.io/tools/icons/?icon=colorize&style=outline

Or does this necessitate a Dashicon to be created?

@jasmussen
Copy link
Contributor

The eye-dropper tool feels like the wrong tool here, as that's not actually what the custom color swatch does. And the fact that it is a swatch feels like we should show a swatch.

How about this?

screenshot 2019-01-24 at 08 58 25

SVG:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><path d="M14.4 9.6l4-4c-1.2-1.2-2.6-1.9-4-2.3l-1.5 5.4c.6.2 1.1.5 1.5.9z" fill="#8ac441"/><path d="M12 8.6c.3 0 .6.1.9.1l1.5-5.4c-1.6-.4-3.2-.4-4.7 0l1.5 5.4c.2 0 .5-.1.8-.1z" fill="#fbed1d"/><path d="M11.1 8.8L9.7 3.3c-1.6.4-3 1.2-4 2.3l4 4c.3-.4.8-.7 1.4-.8z" fill="#f59120"/><path d="M9.6 9.6l-4-4c-1.2 1.2-1.9 2.6-2.3 4l5.4 1.5c.2-.6.5-1.1.9-1.5z" fill="#ea2127"/><path d="M8.6 12c0-.3.1-.6.1-.9L3.3 9.7c-.4 1.6-.4 3.2 0 4.7l5.4-1.5c0-.3-.1-.6-.1-.9z" fill="#ea185a"/><path d="M9.6 14.4l-4 4c1.2 1.2 2.6 1.9 4 2.3l1.5-5.4c-.6-.2-1.1-.5-1.5-.9z" fill="#8f288c"/><path d="M3.3 14.3c.4 1.6 1.2 3 2.3 4l4-4c-.4-.4-.7-.9-.9-1.5l-5.4 1.5z" fill="#e80a89"/><path d="M14.4 14.4c-.4.4-.9.7-1.5.9l1.5 5.4c1.6-.4 3-1.2 4-2.3l-4-4z" fill="#0872b9"/><path d="M12 15.4c-.3 0-.6-.1-.9-.1l-1.5 5.4c1.6.4 3.2.4 4.7 0l-1.5-5.4c-.2 0-.5.1-.8.1z" fill="#2d368e"/><path d="M15.2 12.9c-.2.6-.4 1.1-.9 1.5l4 4c1.2-1.2 1.9-2.6 2.3-4l-5.4-1.5z" fill="#28aae1"/><path d="M15.2 11.1l5.4-1.5c-.4-1.6-1.2-3-2.3-4l-4 4c.5.4.8.9.9 1.5z" fill="#00a44f"/><path d="M20.7 9.7l-5.4 1.5c.1.3.1.6.1.9s-.1.6-.1.9l5.4 1.5c.4-1.8.4-3.4 0-4.8z" fill="#01a79b"/></svg>

@kjellr
Copy link
Contributor

kjellr commented Feb 4, 2019

I actually really like the text link option. It's very clear and puts a bit more weight on the theme's supplied palette, which guides people to more consistent choices.

@jasmussen
Copy link
Contributor

Let's go with that then, it's super simple and if it is insufficient it can be revisited after. Also way simpler than anything we've proposed so far.

@mapk
Copy link
Contributor

mapk commented Feb 5, 2019

I'm good with that decision. Let's get a PR up and ship it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests