-
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
Letter spacing: update label copy #36385
Conversation
…control labels are uniform Organize imports alphabetically Remove hyphen from label
Size Change: +7.4 kB (+1%) Total Size: 1.09 MB
ℹ️ View Unchanged
|
packages/block-editor/src/components/letter-spacing-control/style.scss
Outdated
Show resolved
Hide resolved
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.
The copy change LGTM 👍
I left one small comment regarding an unneeded class name. Once that is removed this should be good to go.
packages/block-editor/src/components/letter-spacing-control/index.js
Outdated
Show resolved
Hide resolved
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.
After the classname removal, this is ready to ship
✅ Copy looks good
✅ Reordered stylesheets are still imported correctly
Description
This PR:
Overwrites default component view label styles, forcing an 8px bottom margin on the letter spacing typography control so that the Typography control labels are uniformToolsPanel: Standardize input control label margin #36387 will take care of this.Testing
Insert a core text block – the paragraph block is an excellent choice. Select your block.
In the Typography Tools Panel in the right-hand sidebar, add a Letter spacing control.
The label margins should match and the label, for the Tools Panel options drop down and for the control itself should read
'Letter spacing'
, without the hyphen.Types of changes
Copy
Checklist:
*.native.js
files for terms that need renaming or removal).