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

Fix when multiple media selection adds only one image or video block #1383

Conversation

marecar3
Copy link
Contributor

Fixes #1346 . Additional details and testing instructions are in the description of the related gutenberg PR.

Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

Point to appropriate gb ref
@marecar3 marecar3 added this to the 1.13 milestone Sep 25, 2019
@marecar3 marecar3 requested review from Tug and mchowning September 25, 2019 14:33
@marecar3 marecar3 changed the title Update release notes Fix when multiple media selection adds only one image or video block Sep 25, 2019
@hypest
Copy link
Contributor

hypest commented Sep 30, 2019

👋 @marecar3 and @dratwas , it's not clear to me and unless you two have already coordinated, can you both make sure the works in this PR and #1331 are compatible? They seem to touch similar functionality so, just wanted to make sure. Thanks!

@marecar3
Copy link
Contributor Author

Hey @hypest, thanks for the ping.

As I can see, It seems that this PR is introducing multiple select on a native level and this PR is fixing existing support (for multiple select) on JS side.

We can wait for @dratwas to confirm.

cc: @pinarol @mkevins

@dratwas
Copy link
Contributor

dratwas commented Sep 30, 2019

Hey, in my PR I added the possibility to select how we will handle the multiple select. If multiple is set to false - we will use the old implementation and few blocks with image/video should be added by sending event through the bridge. If multiple is set to true, all selected media will be passed to the JS callback instead of sending them in event one by one.

Copy link
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Looks good! Approved via WordPress/gutenberg#17579 (review)

I added a comment on the release notes. I think we should specify the platform on these fixes.

RELEASE-NOTES.txt Outdated Show resolved Hide resolved
@etoledom etoledom modified the milestones: 1.13, 1.15 Oct 2, 2019
@etoledom
Copy link
Contributor

etoledom commented Oct 2, 2019

Milestone moved to 1.15

@marecar3 marecar3 merged commit f942586 into develop Oct 2, 2019
@marecar3 marecar3 deleted the issue/1346_Multiple-media-selection-adds-only-one-image-or-video-block branch October 2, 2019 12:26
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.

[Android] Multiple media selection adds only one image/video block
4 participants