-
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
Separator: Fix block CSS classes in the editor #42769
Conversation
Size Change: +87 B (0%) Total Size: 1.26 MB
ℹ️ View Unchanged
|
@@ -37,7 +37,7 @@ export default function SeparatorEdit( { attributes, setAttributes } ) { | |||
'has-css-opacity': opacity === 'css', | |||
'has-alpha-channel-opacity': opacity === 'alpha-channel', | |||
}, | |||
colorProps.classname | |||
colorProps.className |
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.
@glendaviesnz am I onto something here, that this was only a small typo, or am I missing something intentional?
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.
Definitely a typo, thanks for sorting, tested well for me and this brings it back into line with the save.js method.
@@ -37,7 +37,7 @@ export default function SeparatorEdit( { attributes, setAttributes } ) { | |||
'has-css-opacity': opacity === 'css', | |||
'has-alpha-channel-opacity': opacity === 'alpha-channel', | |||
}, | |||
colorProps.classname | |||
colorProps.className |
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.
Definitely a typo, thanks for sorting, tested well for me and this brings it back into line with the save.js method.
I pushed a fix for the failing unit tests |
Thank you. I will continue to test to see if there are block validation issues. |
Not seeing any block validation errors for the class changes. |
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.
Works for me.
This is only the edit.js, not the save.js, so there should be no error.
Thank you very much for a fix. I've just noticed the color inconsistency (WordPress 6.0.1) between the editor and front end when using a color from color options (for example, vivid red). Then I noticed that the editor adds only Looking forward for this fix be included in a core :) |
What?
Partial for #41194
Fixes a typo (?) in the
SeparatorEdit
function whereclassname
was used instead ofclassName
.The problem lead to missing CSS class names, which meant that the separator block did not show the correct colors in the editor.
Why?
The separator block did not show the correct colors in the editor.
How?
Fixes the typo so that the correct classes are output for the block, depending on the selected color.
Before:
block-editor-block-list__block wp-block has-alpha-channel-opacity is-selected wp-block-separator
After:
block-editor-block-list__block wp-block has-alpha-channel-opacity has-purple-to-red-gradient-background has-background wp-block-separator
Testing Instructions
Please test with your chosen theme, classic or FSE, which has support for gradients, and with Twenty Twenty-One.
Each variation should have one each of the color options: a color from the palette, a custom color using the color picker, and a gradient.
Screenshots or screencast
Before, the selected color was not visible.
After: It is still difficult to see the color when the block is selected, but it is used: