-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Dimensions Panel: Fix resetting of axial spacing controls #50654
Conversation
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.
Detailed PR with clear testing instructions and screencasts! I'll just trust you on this! Code LGTM 👍
Size Change: +1 B (0%) Total Size: 1.39 MB
ℹ️ View Unchanged
|
Flaky tests detected in ccb69b6. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4989184275
|
Related:
What?
Fixes regression introduced in #48070 which prevents resetting spacing controls for blocks with axial sides support only e.g. Button block padding.
Why?
Without this fix, any block specifying horizontal or vertical side support will crash when choosing to reset the relevant individual control in the Dimension panel.
How?
Skips filtering the values per side if there are no values.
Testing Instructions
Screenshots or screencast
Screen.Recording.2023-05-16.at.5.32.46.pm.mp4
Screen.Recording.2023-05-16.at.5.35.37.pm.mp4