-
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
Fix BlockSwitcher checks for showing a Dropdown menu or not #57047
Conversation
@@ -218,54 +214,48 @@ export const BlockSwitcherDropdownMenu = ( { clientIds, blocks } ) => { | |||
} } | |||
menuProps={ { orientation: 'both' } } | |||
> | |||
{ ( { onClose } ) => | |||
showDropDown && ( |
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.
All these are whitespace changes because I removed the showDropDown
check, since we know we should render a Dropdown.
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.
Sure! Could we make lighter has*
selectors though? 😄
Size Change: -8 B (0%) Total Size: 1.71 MB
ℹ️ View Unchanged
|
Flaky tests detected in 06b0f1a. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7206568112
|
What?
This PR fixes a bug I noticed while reviewing this PR. Right now if a block has no styles or block transformations, but has pattern transformations we don't render the Dropdown.
Testing Instructions