-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Gallery block: enable the new gallery block by default if running in core #37134
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Size Change: 0 B Total Size: 1.11 MB ℹ️ View Unchanged
|
noisysocks
added
[Block] Gallery
Affects the Gallery Block - used to display groups of images
Backport to WP 6.7 Beta/RC
Pull request that needs to be backported to the WordPress major release that's currently in beta
Gutenberg Plugin
Issues or PRs related to Gutenberg Plugin management related efforts
labels
Dec 6, 2021
noisysocks
approved these changes
Dec 6, 2021
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.
Code LGTM! Thanks for this, makes sense IMO to not have unnecessary feature gating code in Core.
noisysocks
force-pushed
the
update/gallery-version-check-for-core
branch
from
December 6, 2021 01:19
83ecd12
to
9f780a7
Compare
I rebased this to (hopefully) fix failing tests. |
noisysocks
removed
the
Backport to WP 6.7 Beta/RC
Pull request that needs to be backported to the WordPress major release that's currently in beta
label
Dec 6, 2021
noisysocks
pushed a commit
that referenced
this pull request
Dec 6, 2021
…core (#37134) * Enable the new gallery block by default if running in core and not plugin * Add explanatory comment Co-authored-by: Glen Davies <glen.davies@a8c.com>
This was referenced May 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
[Block] Gallery
Affects the Gallery Block - used to display groups of images
Gutenberg Plugin
Issues or PRs related to Gutenberg Plugin management related efforts
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Add a check to see if running in plugin in core, and only check Gallery block version compatibility if in plugin, otherwise enable the new version by default
How has this been tested?
Add a v1 gallery block to a site with use_balanceTags enabled
Check out this PR and make sure the block remains in v1 format and new blocks added are v1
Turn off the use_balanceTags option and check that the v1 blocks migrate to v2 and new blocks are v2 format