-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Blocks: Dropping the controls property from blocks #1262
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.
Looks Great Riad! The gallery controls as well as the quote controls still work, so I do not believe any regressions occurred. Been a while since I looked at the blocks, and I like the new pattern where the control views are defined in the block instead of higher up.
blocks/library/gallery/index.js
Outdated
return [ | ||
controls, | ||
focus && images.length > 1 && ( | ||
<InspectorControls> |
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 will need a key attribute set for it I believe.
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.
Good catch 👍
ca0517f
to
317827c
Compare
Thanks for the review @BE-Webdesign |
In this PR, I'm updating all the blocks to use
BlockControls
inedit
instead of thecontrols
property.related #1205