-
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
Spacing presets: fix bug with unlinked not applying if non int slugs used #44136
Conversation
…tandard 10,20,30 slugs
Size Change: -10 B (0%) Total Size: 1.25 MB
ℹ️ View Unchanged
|
Works like a charm, thanks for the quick fix @glendaviesnz 🙌 |
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.
This is testing nicely for me, too.
✅ Still tests well with the generated spacing scale
✅ With provided theme.json
settings, works in post editor and site editor controls.
LGTM!
@cbravobernal I have cherry-picked this into the 14.1 release branch |
cc @c4rl0sbr4v0 |
…tandard 10,20,30 slugs (#44136)
I just cherry-picked this PR to the wp/6.1 branch to get it included in the next release: 2e2cce6 |
What?
Removes the check for digits from the check to see it spacing values are set and just checks for a defined value
Why?
Fixes: #44133 - if a theme uses string only slugs for spacing sizes the link/unlink option doesn't work as expected - control shows linked when returning to block even if unlinked values are set
How?
Removes the regex check for
\d
Testing Instructions
settings.spacing
section:Screenshots or screencast
Before:
linked-before.mp4
After:
linked-after.mp4