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

Color Settings in Customizer Do Not Apply When Comments Enabled #84132

Closed
mgozdis opened this issue Nov 12, 2023 · 5 comments
Closed

Color Settings in Customizer Do Not Apply When Comments Enabled #84132

mgozdis opened this issue Nov 12, 2023 · 5 comments
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Customizer The site customizer for traditional, non-block themes. [Feature Group] Appearance & Themes Features related to the appearance of sites. Needs triage Ticket needs to be triaged [Platform] Simple [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Type] Bug

Comments

@mgozdis
Copy link

mgozdis commented Nov 12, 2023

Quick summary

Customizer settings for colors are defaulting to the theme's default colors despite custom colors set. This only happens when comments are enabled on a page. If comments are disabled on the page, custom colors work as expected.

7280454-zd-a8c

Steps to reproduce

  1. Install a classic theme such as Dara, Radcliffe 2, or Baskerville 2 on a Simple site.
  2. Create a page and enable comments in Discussion Settings: 6o4zIV.png
  3. Set custom Colors under Appearance > Customize > Colors & Backgrounds and Save (notice they don't apply in preview and theme's defaults show instead): IIHKp3.png
  4. View page and see the theme's default colors are applied instead of custom colors: 915lrE.png
  5. Disable comments in Discussion settings for the page, view the page, see correct colors applied: yMAYMx.png

What you expected to happen

The correct customizer color settings should apply regardless if comments are enabled or disabled.

What actually happened

Colors only display correctly if comments are disabled. When comments are enabled, CSS is loaded in a different order, causing the theme's default color settings to override the custom colors set in the Customizer. When hard refreshing, you will see the custom colors display briefly before being overridden by theme's default colors.

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

Simple

Logs or notes

This was tested and reproducible on Simple sites with various Classic Themes including Dara, Radcliffe 2, and Baskerville 2. Unknown if AT sites are affected.

@mgozdis mgozdis added [Type] Bug [Feature Group] Appearance & Themes Features related to the appearance of sites. [Feature] Customizer The site customizer for traditional, non-block themes. Needs triage Ticket needs to be triaged [Product] WordPress.com All features accessible on and related to WordPress.com. labels Nov 12, 2023
@github-actions github-actions bot added [Platform] Simple [Pri] Low Address when resources are available. labels Nov 12, 2023
@inaikem
Copy link
Contributor

inaikem commented Nov 12, 2023

I encountered the error here, too: 7280459-zd-a8c

I've advised the user they can fix this temporarily by deactivating comments on affected pages.

@alshakero, I'm not 100% certain but it feels like this might be related to #83982 (at least in spirit).

Copy link

github-actions bot commented Nov 12, 2023

Support References

This comment is automatically generated. Please do not edit it.

  • 7280454-zen
  • 7280459-zen

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Nov 12, 2023
@alshakero
Copy link
Member

alshakero commented Nov 12, 2023

Hi! It is related, can you confirm this happens when you're not proxied? We pulled this version, and now it only affects proxied users.

@mgozdis
Copy link
Author

mgozdis commented Nov 14, 2023

@alshakero Yes, this was happening when not proxied as users were reporting it. Currently, I am only seeing the issue when proxied now so it seems fixed for users. Can you confirm?

@alshakero
Copy link
Member

Yes, it was fixed here: p1699612343673279/1699464090.252049-slack-C02FMH4G8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Customizer The site customizer for traditional, non-block themes. [Feature Group] Appearance & Themes Features related to the appearance of sites. Needs triage Ticket needs to be triaged [Platform] Simple [Pri] Low Address when resources are available. [Product] WordPress.com All features accessible on and related to WordPress.com. [Type] Bug
Development

No branches or pull requests

3 participants