-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Navigation: Add content only prototype #73431
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
base: trunk
Are you sure you want to change the base?
Conversation
|
Size Change: +1.09 kB (+0.04%) Total Size: 2.54 MB
ℹ️ View Unchanged
|
|
Flaky tests detected in cefbe0c. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/19508159715
|
| if ( getBlockName( selectedBlockClientId ) === 'core/navigation' ) { | ||
| // If the selected block is navigation, exclude all its descendants | ||
| descendants.forEach( ( childId ) => | ||
| navigationChildren.add( childId ) | ||
| ); | ||
| } |
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 makes it so if your only item within the pattern is a navigation block, nothing shows in your inspector sidebar. I don't think this code is necessary. What was it for?
16c5aa4 to
54bc434
Compare
| @@ -0,0 +1,201 @@ | |||
| /** | |||
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 whole file is copy/pasted
What?
Closes
Why?
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast