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

State: Assign default for all allowed blockTypes #4707

Merged
merged 1 commit into from
Jan 30, 2018

Conversation

aduth
Copy link
Member

@aduth aduth commented Jan 27, 2018

Related: #3745

This pull request seeks to define a default value for the EditorProvider blockTypes setting. As implemented, this is only respected if explicitly provided. This is true for the top-level editor, passed from the server-side client-assets.php script enqueuing, but as discovered in the original implementation of #3745 with nested editor providers, would result in an error if not provided. The changes here provide this default which, while not currently used, reduces fragility of the provider usage.

Testing instructions:

There should be no changes in behavior, specifically note lack of regressions in behavior of allowed block types.

Explicitly passed from top-level editor, but should be inferred from nested providers
@aduth aduth added the Framework Issues related to broader framework topics, especially as it relates to javascript label Jan 27, 2018
@aduth aduth merged commit 0dbab97 into master Jan 30, 2018
@aduth aduth deleted the add/editor-provider-default-block-types branch January 30, 2018 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues related to broader framework topics, especially as it relates to javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant