Skip to content
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

Disable alignment for innerBlocks of Nav Block #27365

Merged

Conversation

jordesign
Copy link
Contributor

Adding the Experimental Layout property to disable alignment settings for InnerBlocks of the Navigation Block.

Done in the same method as outlined for the Buttons block here:
#26380

This addresses issue #27179 - where setting an alignment on the Social Links block breaks the display of the Navigation Block.

@talldan talldan added [Block] Navigation Affects the Navigation Block [Type] Enhancement A suggestion for improvement. labels Jan 20, 2021
Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jordesign, this is working well for me, it would be great to get it merged.

I notice the branch has conflicts, they look easy to resolve, should just be able to accept both sets of changes.

@jordesign
Copy link
Contributor Author

Hey @talldan I think that conflict is resolved now (first time doing so - so I hope it worked)

Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks @jordesign.

@talldan talldan merged commit 7a2b5a2 into WordPress:master Jan 21, 2021
@github-actions github-actions bot added this to the Gutenberg 9.9 milestone Jan 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants