Repository navigation
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
🤖 PR meta 🤖📦 Bundle sizeSize Change: -52 B (0%) Total Size: 8 MB 📦 View Changed
⚡ PerformanceShow the resultsClient side metrics exclude the server response time. front-end-block-theme
front-end-classic-theme
media-processing
media-upload
post-editor
site-editor
|
color so overrides reach stroke-based icons| .edit-site-patterns__pattern-icon { | ||
| fill: var(--wp-block-synced-color); | ||
| flex-shrink: 0; | ||
| } |
There was a problem hiding this comment.
Removed rather than updated to color: this class no longer exists in any source file, so the rule is dead CSS.
Co-Authored-By: Claude <noreply@anthropic.com>
| // Keep `fill` as a fallback for custom block icons that do not use `currentColor`. | ||
| fill: CanvasText; | ||
| color: CanvasText; |
There was a problem hiding this comment.
Block icons in the admin UI, in High Contrast mode.
| Before | After |
|---|---|
| // Keep `fill` as a fallback for custom block icons that do not use `currentColor`. | ||
| fill: CanvasText; | ||
| color: CanvasText; |
There was a problem hiding this comment.
Block icons in the list view, in High Contrast mode.
| Before | After |
|---|---|
| // Keep `fill` as a fallback for custom icons that do not use `currentColor`. | ||
| fill: CanvasText; | ||
| color: CanvasText; |
There was a problem hiding this comment.
Icons rendered inside a Button, in High Contrast mode.
| Before | After |
|---|---|
| // Keep `fill` as a fallback for custom icons that do not use `currentColor`. | ||
| fill: CanvasText; | ||
| color: CanvasText; |
There was a problem hiding this comment.
The icon in a placeholder label, in High Contrast mode.
| Before | After |
|---|---|
| // Keep `fill` as a fallback for custom block icons that do not use `currentColor`. | ||
| fill: CanvasText; | ||
| color: CanvasText; |
There was a problem hiding this comment.
Block icons in the canvas, in High Contrast mode.
packages/block-editor/src/components/block-icon/content.scss is the canvas copy of packages/block-editor/src/components/block-icon/style.scss. The two files are identical and are changed together.
| flex-shrink: 0; | ||
| margin-right: $grid-unit-15; | ||
| fill: $gray-900; | ||
| color: $gray-900; |
There was a problem hiding this comment.
The lock / unlock icon next to each checkbox in the block lock modal.
lock and unlock are not stroke-based today, so this is a preventive change.
|
|
||
| .block-editor-patterns__pattern-icon { | ||
| fill: var(--wp-block-synced-color); | ||
| color: var(--wp-block-synced-color); |
There was a problem hiding this comment.
The icon on a synced pattern in the patterns list.
symbol is not stroke-based today, so this is a preventive change.
|
|
||
| &-icon--checked { | ||
| fill: $gray-300; | ||
| color: $gray-300; |
There was a problem hiding this comment.
The viewport icon of a checked viewport in the block visibility modal.
desktop, tablet and mobile are not stroke-based today, so this is a preventive change.
| &:hover, | ||
| &:focus-visible { | ||
| fill: var(--wpds-color-foreground-interactive-brand); | ||
| color: var(--wpds-color-foreground-interactive-brand); |
| pointer-events: none; | ||
| svg { | ||
| fill: $gray-600; | ||
| color: $gray-600; |
|
@t-hamano did you intend to close this PR? |
|
@ciampo While working on this PR, I realized that the |


What?
This PR makes SVG icons colorable through the
colorproperty so that color overrides reach stroke-based icons.Why?
Since #78808, stroke-based icons render an inline
fill: noneand paint themselves withstroke="currentColor":Any styling that colors an icon via
filltherefore stops working:How?
fillwithcolorwhere the rule exists to color an icon.currentColor, keepfillas a fallback and addcoloralongside it.@wordpress/block-editorand@wordpress/componentsget a CHANGELOG entry: they are the packages where an icon is visually fixed. The other packages only receive the preventive change.Testing Instructions
I have left an inline comment on each change describing where to look.
Screenshots or screencast
Screenshots are attached to the inline comments, next to the change they belong to.
Use of AI Tools
Claude Code was used to locate the affected styles and to draft this change. I reviewed every change and take responsibility for it.