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

Colour Consistency Issues when switching themes #43832

Open
alanjacobmathew opened this issue Sep 3, 2022 · 8 comments
Open

Colour Consistency Issues when switching themes #43832

alanjacobmathew opened this issue Sep 3, 2022 · 8 comments
Labels
[Feature] Colors Color management [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable

Comments

@alanjacobmathew
Copy link
Contributor

Description

When switching b/w themes the custom colour given to certain blocks changes automatically in the post Editor. This was observed for Heading, Para, List. The issues is mainly for background colour for these fields.

Themes tested:
TT2
TT1-block
T21
T20
T19

Test were done only for post editor and not the FSE editor. For all the blocks, text colour was black, and background colour was red. Few images below how each results were. Below tests were done without GB plugin installed, but even when it's installed there were issues with colour

1: TT2
image

2: TT1 Blocks
image

3: TT1
image

4: T20
image

5: T19
image

Step-by-step reproduction instructions

1:Add para block, heading block, list block
2: Add text color and background color.
3: Changelthemes between TT2, TT1-block, TT1, T20, T19

Screenshots, screen recording, code snippet

No response

Environment info

WP 6.0.2
php: 7.4
Browser Brave 1.43.88(chromium)
Win 10

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

@carolinan
Copy link
Contributor

Can you provide more detailed, step by step instructions of how you set up the colors and also what your expectations where?

@alanjacobmathew
Copy link
Contributor Author

I never did anything more than just switching themes.

Steps:

  • Install T22 and T20 themes.
  • Activate T20
  • Open post editor
  • Add a list block
  • insert content with red(FF0000) as text color and background color as black(000000)
  • Switch theme to T22.

Now the background color for the list block in my case became #1A4548. I noticed it happening for para and heading too.

Now change the background color in T20 to #1a4548, and change theme, the background color becomes black. This doesn't happen to other colors I tested.

I tested the cases with themes from T22 upto T19, all with the background color black. and for each theme the result differs as given in the images above. The test cases data were in the following form
a: Current theme installed
b: Text color: red
c: Background color: black

The text outside the images were the output results for that specific theme

@carolinan
Copy link
Contributor

To confirm, are the colors you are selecting from the theme color palette, the WordPress default color palette, or custom from the color picker?

The theme color palettes are theme specific, the colors are intended to change when you switch themes. These colors are what gives each theme a unique look.

@alanjacobmathew
Copy link
Contributor Author

image

Default color paletter

@carolinan
Copy link
Contributor

If you look carefully on the picture, you can see that the dark green color, which is selected, is in the theme palette, not the default.

@cbirdsong
Copy link

This seems like it's possibly related to the issues discussed in #38998 and #39372?

@annezazu
Copy link
Contributor

annezazu commented Sep 9, 2022

Agreed with @cbirdsong - I'd prefer we close this out and defer to those.

@alanjacobmathew
Copy link
Contributor Author

I picked the colour from the color picker and not from the theme palette. But after retesting it again, even if the color is picked individually, if it is a part of theme palette, it automatically assigns it to as Theme's primary color.
Not sure, but it may affect users if it directly converts one Theme's primary to another theme's primary color set.

If I set an entire page in accordance with a similar color code and if I change the theme the entire color structure is changed. It may not affect those who simply write content without any color styling, but for those who make landing pages it might have an impact.

Colour Cons

@annezazu annezazu added [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. [Feature] Colors Color management [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable labels Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Colors Color management [Feature] Themes Questions or issues with incorporating or styling blocks in a theme. [Type] Feedback Issues that relate purely to feedback on a feature that isn't necessarily actionable
Projects
None yet
Development

No branches or pull requests

4 participants