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

temporary fix for post meta attributes bug in Gutenberg 6.6.0 #35

Merged
merged 12 commits into from
Oct 4, 2019

Conversation

razwan
Copy link
Member

@razwan razwan commented Oct 4, 2019

Description
The 6.6.0 update seems to cause an issue where the editor breaks when blocks that use post meta as source for one of their attributes are added to the content. see WordPress/gutenberg#17767

This is a temporary solution to avoid crashing the editor by relying on the NOVABLOCKS_USE_POST_META_ATTRIBUTES constant to display just a height setting for each hero block instead of using page-level controls that rely on post meta

@razwan razwan changed the title temporary fix for post meta attributes temporary fix for post meta attributes bug in Gutenberg 6.6.0 Oct 4, 2019
@razwan
Copy link
Member Author

razwan commented Oct 4, 2019

Also, in the fallback version, I enabled the position indicators by default, and will display the scroll indicator if the hero block is the first block in the content and it is set to full height. I guess we're good to go.

@vladolaru vladolaru merged commit 8a5ee10 into master Oct 4, 2019
@vladolaru vladolaru deleted the meta-attributes-temp-fix branch October 4, 2019 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants