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

[Gutenberg] Gallery refactor as nested image blocks #14769

Merged
merged 13 commits into from
Jul 7, 2021

Conversation

mkevins
Copy link
Contributor

@mkevins mkevins commented Jun 3, 2021

Related PRs:

gutenberg: WordPress/gutenberg#32382
gutenberg-mobile: wordpress-mobile/gutenberg-mobile#3579
WordPress-iOS: wordpress-mobile/WordPress-iOS#16699

Description

This PR brings in the refactored Gallery block with inner image blocks.

To test:

This has already been tested via the only "sub PR". Originally, this PR was going to include other changes to bring in the flag from editor settings as well, but that can be handled independently in a subsequent PR.

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

mkevins added 6 commits June 1, 2021 20:38
This detects the new format via the presence of the `imageCount`
property in the block's json and defers processing to the image block
processor (recursively through the `processContent` method).
…rnmobile/refactor/gallery-as-nested-image-blocks-update-processor
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 3, 2021

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 3, 2021

You can test the changes on this Pull Request by downloading the APKs:

…lery-as-nested-image-blocks-update-processor

[Gutenberg] Gallery refactor - Update upload completion processor
@mkevins mkevins marked this pull request as ready for review July 7, 2021 06:00
@peril-wordpress-mobile
Copy link

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

Copy link
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

Great work @mkevins 👍
All the changes have already been approved via the sub-PR 🎉

@mkevins mkevins merged commit 5c1cf69 into develop Jul 7, 2021
@mkevins mkevins deleted the rnmobile/refactor/gallery-as-nested-image-blocks branch July 7, 2021 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants