-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Patterns: Apply white background to the preview as a fallback #54534
Conversation
Size Change: +41 B (0%) Total Size: 1.62 MB
ℹ️ View Unchanged
|
Flaky tests detected in 78cf864. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6218346811
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aaronrobertshaw Thanks for the review! |
Fixes #54533
What?
This PR applies a white background to the pattern page preview when the currently activated theme does not have a background color.
Why?
This is because if the currently active theme does not have a background color, it will be affected by the black canvas background behind the preview.
How?
If the background color obtained via the
useGlobalStyle
hook isundefined
, the default white color is passed to the BlockPreview component.Before this approach, I also tried a simple CSS solution as follows.
However, this is not a problem for themes that do not define a background color, but for themes that do have a background color, the following flickering occurs:
Screencast
864f979559cbeda0e27e2f83aa177805.mp4
Testing Instructions
header
template is white.