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 gallery size labels - 1.22.0 #11236

Conversation

mkevins
Copy link
Contributor

@mkevins mkevins commented Feb 5, 2020

Related PR:

gutenberg: WordPress/gutenberg#20045
gutenberg-mobile: wordpress-mobile/gutenberg-mobile#1856
WordPress-iOS: wordpress-mobile/WordPress-iOS#13375

Description

This brings the fix from WordPress/gutenberg#19800 to the release branch for 1.22.0 and updates the release notes to reflect the new feature.

How has this been tested?

Tested via steps here: WordPress/gutenberg#19800 (comment)

PR submission checklist:

  • I have considered adding unit tests where possible.
  • 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.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Feb 5, 2020

You can test the changes on this Pull Request by downloading the APK here.

@hypest
Copy link
Contributor

hypest commented Feb 5, 2020

Tried out the Peril-built WordPress-pr-11236-build-35907.apk and I see empty Image size options on Android 10, Pixel 2XL, when having a gallery with 2 images from the WP Media Library.

(FWIW, works OK for me on WPiOS, Peril build 21887)

screenshot-1580905861262

@hypest
Copy link
Contributor

hypest commented Feb 6, 2020

Tried again, installing both a locally built vanillaRelease and the same Peril build as before and I see no issue with "empty dialog" 🎉. There must have been some issue with how the Peril build was installed before.

Copy link
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

LGTM!

@peril-wordpress-mobile
Copy link

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@mkevins mkevins merged commit 319ff54 into gutenberg/integrate_release_1.22.0 Feb 6, 2020
@mkevins mkevins deleted the gutenberg/integrate_release_1.22.0-fix-gallery-size-labels branch February 6, 2020 14:28
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