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

Cover block - Allow media editing #2403

Merged
merged 13 commits into from
Jul 24, 2020
Merged

Cover block - Allow media editing #2403

merged 13 commits into from
Jul 24, 2020

Conversation

geriux
Copy link
Contributor

@geriux geriux commented Jun 18, 2020

Fixes #2388

Gutenberg PR -> WordPress/gutenberg#23280
WordPress-iOS PR -> wordpress-mobile/WordPress-iOS#14420
WordPress-Android PR -> wordpress-mobile/WordPress-Android#12356

This PR adds Media editing functionality to the Cover block. With the recent changes and refactor we can reuse the same functionality as in Image and Media & Text block.

To test check the Gutenberg PR description.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 18, 2020

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@geriux geriux force-pushed the feature/cover-media-editing branch from 902cf1f to 472a11a Compare July 7, 2020 15:26
@geriux geriux changed the title Cover block - Allow media editing [TEST] Cover block - Allow media editing Jul 7, 2020
@geriux geriux removed the Blocks label Jul 7, 2020
@geriux geriux force-pushed the feature/cover-media-editing branch from 472a11a to 6539050 Compare July 8, 2020 16:58
@geriux geriux force-pushed the feature/cover-media-editing branch from 6539050 to 0a18c5f Compare July 15, 2020 11:10
@geriux geriux force-pushed the feature/cover-media-editing branch from 0a18c5f to a190fb0 Compare July 15, 2020 13:50
@geriux geriux force-pushed the feature/cover-media-editing branch from a190fb0 to a318d84 Compare July 20, 2020 12:29
@geriux geriux changed the title [TEST] Cover block - Allow media editing Cover block - Allow media editing Jul 20, 2020
@geriux geriux requested a review from mkevins July 20, 2020 12:32
@geriux geriux added this to the 1.34 milestone Jul 20, 2020
@geriux geriux added the Media label Jul 20, 2020
@geriux geriux marked this pull request as ready for review July 20, 2020 12:33
@geriux geriux force-pushed the feature/cover-media-editing branch from a318d84 to 05d6917 Compare July 20, 2020 13:29
@geriux geriux force-pushed the feature/cover-media-editing branch from 05d6917 to 9d87b97 Compare July 20, 2020 13:43
@geriux geriux force-pushed the feature/cover-media-editing branch from 9d87b97 to 5379d4c Compare July 22, 2020 10:39
Copy link
Contributor

@mkevins mkevins left a comment

Choose a reason for hiding this comment

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

Approved via: WordPress/gutenberg#23280 (review) - just have to update the submodule reference. LGTM!

@geriux geriux force-pushed the feature/cover-media-editing branch from 482a18a to 225f509 Compare July 24, 2020 07:27
@geriux geriux merged commit 3fb5eee into develop Jul 24, 2020
@geriux geriux deleted the feature/cover-media-editing branch July 24, 2020 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cover block: allow editing image
2 participants