Make WordPress Core

Opened 3 years ago

Closed 2 years ago

#57414 closed defect (bug) (fixed)

Twenty Ten: The color of the plus icon should be white in Add Block button

Reported by: umesh84's profile umesh84 Owned by: audrasjb's profile audrasjb
Milestone: 6.3 Priority: normal
Severity: normal Version: 6.1.1
Component: Bundled Theme Keywords: has-patch has-screenshots
Focuses: ui, css Cc:

Description

In Twenty Ten Theme, The color of the plus icon should be white in Add Block button

Attachments (5)

CleanShot 2023-01-03 at 16.07.16@2x.png (546.6 KB) - added by umesh84 3 years ago.
For better understanding I have attached a screenshot
#57414.patch (477 bytes) - added by kajalgohel 3 years ago.
patch added
57414.2.patch (484 bytes) - added by sabernhardt 3 years ago.
removes extra spaces and uses 3-character color code
7a087379e6469e3c4f58574e0c0c7968.gif (20.1 KB) - added by audrasjb 2 years ago.
Before patch :'(
f6c73d262fd3a9e58112b0723323efdd.gif (19.5 KB) - added by audrasjb 2 years ago.
After patch \o/

Download all attachments as: .zip

Change History (12)

@umesh84
3 years ago

For better understanding I have attached a screenshot

@kajalgohel
3 years ago

patch added

#1 @naeemhaque
3 years ago

  • Keywords has-patch has-screenshots added; needs-patch removed

@sabernhardt
3 years ago

removes extra spaces and uses 3-character color code

#2 @sabernhardt
3 years ago

  • Milestone changed from Awaiting Review to 6.3

To meet coding standards, I updated the patch for Twenty Ten.

However, it may be better to add the color to all child elements of the toggle button within the editor for any theme (probably after the :hover state).

		& * {
			color: $white;
		}
Last edited 3 years ago by sabernhardt (previous) (diff)

#3 @pooja1210
3 years ago

Hi,

Tested the patch and it's working fine. With Twenty Ten Theme, the color of the Add Block button(plus icon) is appearing white after the fix.

Test Environment:
WordPress: 6.2
Theme: Twenty Ten
OS: MacOS
Browser: Chrome

Issue Before Patch: https://prnt.sc/lukTTPINSX9O

Fix After Patch Added: https://prnt.sc/bJTolmDvwz8W

Thanks!

#4 @amin7
3 years ago

Test Report


Patch tested: https://core.trac.wordpress.org/attachment/ticket/57414/%2357414.patch

Environment

OS: Windows 10 version 22H2
Web Server: Nginx
PHP: 8.1.9
WordPress: 6.2
Browser: Chrome 109.0.5414.87
Theme: Twenty Ten
Active Plugins: No plugins activated.

Actual Results


✅ The color of the plus icon from add block button is now white

Supplemental Artifacts

Before Patch: https://d.pr/i/KtPEbA
After Patch: https://d.pr/i/mcDz5x

#5 @sabernhardt
3 years ago

GB46568 may correct this for multiple themes, but I'll leave this open for now.

#6 @audrasjb
2 years ago

Given the related PR on Gutenberg was closed, I'd say it's probably better (and doesn't hurt) to fix it on Twenty Ten anyway…
See https://github.com/WordPress/gutenberg/pull/46636

@audrasjb
2 years ago

Before patch :'(

@audrasjb
2 years ago

After patch \o/

#7 @audrasjb
2 years ago

  • Owner set to audrasjb
  • Resolution set to fixed
  • Status changed from new to closed

In 56081:

Twenty Ten: Prevent Block Inserter icon color override.

This changeset prevents the likes of body * { color: #000; } in custom styles to override the Block Inserter icon styles.

Props umesh84, kajalgohel, sabernhardt, pooja1210, amin7.
Fixes #57414.

Note: See TracTickets for help on using tickets.