-
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
[WIP] Initial attempt to leave a pattern block wrapped around a pattern #49967
Conversation
Size Change: +1.08 kB (0%) Total Size: 1.37 MB
ℹ️ View Unchanged
|
Flaky tests detected in e9e9308. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4856952236
|
@aaronrobertshaw this change seems to fix the duplicated block controls issue |
… appear on pattern block (#50118) --------- Co-authored-by: Aaron Robertshaw <[email protected]>
…ning the inner blocks
60edbdf
to
f4105c7
Compare
@@ -6,13 +6,29 @@ | |||
"category": "theme", | |||
"description": "Show a block pattern.", | |||
"supports": { | |||
"html": false, | |||
"inserter": false |
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.
Curious to know why we want to allow the pattern block to be shown in the inserter? Moreover, should it be allowed to create a pattern block without the slug
attribute? What's the difference between it and a group block?
Closing this in favour of one that focuses on just adding the patterns wrapper and will circle back on the syncing options later after that is in |
What?
Experimenting with leaving a Pattern block wrapper around a pattern
Why?
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast