#52702 closed defect (bug) (fixed)
Twenty Twenty-One: With WP5.7 RC2, the text color for the dark background color palette is dark instead of white.
Reported by: | bduclos | Owned by: | desrosj |
---|---|---|---|
Milestone: | 5.7 | Priority: | normal |
Severity: | normal | Version: | 5.7 |
Component: | Bundled Theme | Keywords: | has-patch commit dev-reviewed |
Focuses: | css | Cc: |
Description
In WP 5.6, when the user chooses “Black", "Dark Gray or "Gray" as the background-color for the Media & Text, Cover, Paragraph and Heading blocks, the text is by default white. With 5.7 RC2, the text is now dark.
Steps to reproduce the issue with WP 5.7 RC2:
- Insert a Media & Text block.
- Set its background color as Black using the first color palette item.
- Insert a paragraph inside the block.
- Observe that by default, the text is dark. In WP 5.6 the text was white.
The following code in 5.6 has disappeared from 5.7:
:not(.has-text-color).has-black-background-color[class], :not(.has-text-color).has-gray-background-color[class], :not(.has-text-color).has-dark-gray-background-color[class] { color: var(--global--color-white); }
Attachments (5)
Change History (28)
This ticket was mentioned in Slack in #core-themes by hellofromtonya. View the logs.
4 years ago
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
4 years ago
This ticket was mentioned in Slack in #core-themes by hellofromtonya. View the logs.
4 years ago
#7
@
4 years ago
Also seeing the problem in the Columns block with paragraph blocks in each column, i.e. see image.
This ticket was mentioned in PR #1070 on WordPress/wordpress-develop by hellofromtonya.
4 years ago
#8
- Keywords has-patch added; needs-patch removed
Trac ticket: https://core.trac.wordpress.org/ticket/52702
#10
@
4 years ago
- Keywords needs-testing added
To test the patch:
- In terminal:
npm run grunt patch:https://github.com/WordPress/wordpress-develop/pull/1070
to pull the patch down to your machine - In a post, add the Media & Text block.
- Set a image and add text.
- Set the background color to a dark color by going to Color settings > Background Color.
- Is the text white?
- Add a Columns blocks.
- Set the background color to a dark color by going to Color settings > Background Color.
- Set to 2-columns
- Add a Paragraph block into each column with some text
- Observe the text color. Is it white?
#11
@
4 years ago
I did a test with dark mode on and off and with different color combinations and I believe the PR solves the issue.
This ticket was mentioned in Slack in #core-themes by hellofromtonya. View the logs.
4 years ago
#13
@
4 years ago
Testing.
WordPress 5.7 RC2.
Theme: Twenty Twenty One.
Active plugins: WordPress Beta Tester.
Initial test.
Added Media & Text block.
Added black background.
In paragraph block inserted some text. Default is a very dark color.
Testing patch.
http://localhost:8889/
Twenty Twenty One
WordPress 5.7 alpha.
In terminal:
npm run grunt patch:https://github.com/WordPress/wordpress-develop/pull/1070
In a post added Media & Text block.
Changed the background color to black.
Added some text to the paragraph block.
Default text is now white.
Added a Columns block with 3 columns.
Changed the background color to black.
Added paragraph block into all three columns.
Added text into the first column.
Default text is now white.
Additional test.
Added a Heading block.
Changed the background color to black.
Added some text to the Heading block.
Default text is now white.
Additional test.
Wrote some text that defaults to the paragraph block.
Changed the background color to black.
Default text automatically turned white.
@
4 years ago
Screenshot of multiple block types where the background color is black. Text is automatically changed to white.
#14
@
4 years ago
- Keywords commit dev-feedback added
Looks good! briefly tested it myself but it seems well-covered by everyone else :) Tagging for commit.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
4 years ago
#17
@
4 years ago
- Owner set to desrosj
- Resolution set to fixed
- Status changed from new to closed
In 50493:
#18
@
4 years ago
- Keywords fixed-major added
- Resolution fixed deleted
- Status changed from closed to reopened
This ticket was mentioned in Slack in #core by monikarao. View the logs.
4 years ago
hellofromtonya commented on PR #1070:
3 years ago
#24
Committed via https://core.trac.wordpress.org/changeset/50493.
Show the issue in action with 5.7 RC1