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

Theme Style Variations: Changing style variations, or choosing a new color preset, clears user created custom colors #62232

Open
ndiego opened this issue Jun 3, 2024 · 3 comments
Labels
[Feature] Theme Style Variations Related to style variations provided by block themes Needs Dev Ready for, and needs developer efforts [Type] Bug An existing feature does not function as intended

Comments

@ndiego
Copy link
Member

ndiego commented Jun 3, 2024

Description

If you have created custom colors in the Styles panel, as soon as you select a different style variation, or select a color preset, the custom colors are removed. This behavior is the same in 6.5 and with Gutenberg. See the videos below for comparison.

It is not clear to me why user created custom colors should be tied to the style variation that has been chosen. While there may be a technical reason, this is not intuitive. Furthermore, since Gutenberg (and soon 6.6) adds color presets to the Style panel, it is very easy to click on a preset and then accidentally wipe all your custom colors. If a user doesn't know that they can click undo, there is no way to get their colors back outside of refreshing the Site Editor without saving.

Step-by-step reproduction instructions

  1. Using TT4 and Gutenberg, open the Styles panel in the Site Editor and add a few custom colors
  2. Select a color preset and see that the custom colors are removed

Screenshots, screen recording, code snippet

WordPress 6.5

custom-colors-6-5.mp4

WordPress 6.5 + Gutenberg

custom-colors-6-5-gutenberg.mp4

Environment info

  • WordPress 6.5
  • Gutenberg lastest

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

@ndiego ndiego added [Type] Bug An existing feature does not function as intended [Feature] Theme Style Variations Related to style variations provided by block themes labels Jun 3, 2024
@mgawe
Copy link

mgawe commented Jun 3, 2024

Related: #48407

@colorful-tones colorful-tones added the Needs Dev Ready for, and needs developer efforts label Jun 3, 2024
@richtabor
Copy link
Member

It does feel strange that a user's custom color would be discarded. We treat them separately in the UI, feels they should be resolved separately as well.

@scruffian
Copy link
Contributor

I think this is part of the change proposed in #62661. This is not a small fix, so I'll move it off the 6.6 board.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Theme Style Variations Related to style variations provided by block themes Needs Dev Ready for, and needs developer efforts [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

5 participants