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

Global Styles: Can't open Colour Picker by clicking on colour circle #44138

Closed
noisysocks opened this issue Sep 14, 2022 · 3 comments · Fixed by #45681
Closed

Global Styles: Can't open Colour Picker by clicking on colour circle #44138

noisysocks opened this issue Sep 14, 2022 · 3 comments · Fixed by #45681
Assignees
Labels
Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json [Type] Enhancement A suggestion for improvement.

Comments

@noisysocks
Copy link
Member

Description

In Global Styles one can't open the Colour Picker by clicking on the colour circle. You need to first click on the ellipsis and then press Edit which is confusing since the colour circles are interactive but do nothing. It's okay for editing the name to be buried but changing the colour should be quicker.

Step-by-step reproduction instructions

  1. Go to Site Editor → Global Styles
  2. Click Colors
  3. Open a pallette
  4. Click on one of the colour circles

Expected result: The colour picker should open.
Actual result: Nothing happens.

Screenshots, screen recording, code snippet

Kapture.2022-09-14.at.10.08.05.mp4

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@noisysocks noisysocks added [Type] Bug An existing feature does not function as intended Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json labels Sep 14, 2022
@noisysocks
Copy link
Member Author

@ramonjd said this might be due to dynamic ref resolution in theme.json which he was looking at in #43249.

@ramonjd ramonjd added [Type] Enhancement A suggestion for improvement. and removed [Type] Bug An existing feature does not function as intended labels Sep 20, 2022
@ramonjd
Copy link
Member

ramonjd commented Sep 20, 2022

@ramonjd said this might be due to dynamic ref resolution in theme.json which he was looking at in #43249.

Actually, looking at this again, I was thinking of an entirely different case. So I don't think this is a bug.

This would be a good feature enhancement however.

While backtracing I noticed that the onChange event for palette options is empty:

Maybe there's a good place from which to trigger the ColorPicker component?

@ramonjd
Copy link
Member

ramonjd commented Nov 9, 2022

I know @pbking has a PR up to enable edit mode all the time.

Just in case, I have an alternative that opens edit mode and selects the color over at #45681

2022-11-10 10 11 43

Happy to close mine. Sorry I didn't see #45657 before starting.

@priethor priethor removed the [Status] In Progress Tracking issues with work in progress label Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Global Styles Anything related to the broader Global Styles efforts, including Styles Engine and theme.json [Type] Enhancement A suggestion for improvement.
Projects
None yet
4 participants