storefront icon indicating copy to clipboard operation
storefront copied to clipboard

Custom text color in blocks is broken in editor - customizer styles not consistently applied in editor

Open Aljullu opened this issue 5 years ago • 11 comments

Filling it so we can track this for following releases, even though the issue only affects the last version of Gutenberg master so it might make sense to wait until an official release is done.

To Reproduce

  1. In the editor, change the text color of a paragraph block.
  2. Notice changes are not applied. That's because it doesn't have a style attribute anymore and one of our CSS rules (.editor-styles-wrapper .block-editor-block-list__block) overrides the base styles.

Screenshots

Peek 2020-04-28 08-47

Expected behavior

Changes in the text color should have an effect in the editor.

Aljullu avatar Apr 28 '20 06:04 Aljullu

An update on this: Gutenberg 8.0.0 was released and this issue is reproducible.

I also tested, and links are not correctly styled either.

Aljullu avatar May 06 '20 16:05 Aljullu

Review this comment in the Gutenberg issue: https://github.com/WordPress/gutenberg/issues/21931#issuecomment-624814695

GlennMartin1 avatar May 06 '20 18:05 GlennMartin1

Thanks for tracking that down @GlennMartin1 !

I've pulled the relevant part of the comment here for clarity:

The issue here is that Twenty Nineteen was not loading its color palette stylesheet on the editor.

Possibly we have the same issue in Storefront.

haszari avatar May 06 '20 19:05 haszari

Could this be milestoned for 2.5.7?

GlennMartin1 avatar May 08 '20 16:05 GlennMartin1

Added to milestone for now – still deciding what we're focusing on this release but would be good to tidy this up.

haszari avatar May 08 '20 23:05 haszari

This issue's title is incorrect and should be revised.

The problem is not Gutenberg. Instead, the problem is that Storefront is (presumably) not loading the color palette stylesheet on the editor.

GlennMartin1 avatar May 14 '20 11:05 GlennMartin1

The problem is not Gutenberg. Instead, the problem is that Storefront is (presumably) not loading the color palette stylesheet on the editor.

Thanks Glenn - I've updated the title. Personally I prefer issues that are focused on the user-facing problem, not a particular code solution. Hopefully I've captured the problem in the new title!

haszari avatar May 14 '20 20:05 haszari

Let's hold off on this for now – I see the discussion is still ongoing in Gutenberg, and there's a now a Gutenberg PR open to address the issue.

It looks like this is still being worked out - this affects many themes. When Gutenberg is released and there's guidance on the best fix, we can implement in Storefront.

haszari avatar May 19 '20 00:05 haszari

Could be fixed in Gutenberg: https://github.com/WordPress/gutenberg/issues/21931#event-3514907525

haszari avatar Jul 07 '20 06:07 haszari

Another instance in 3883197-zen

ericamQA avatar Apr 08 '21 03:04 ericamQA

An instance of this here: 23525996-hc

KirkwallDay avatar Jun 29 '21 21:06 KirkwallDay