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

Fixes color picker segmented control rendering and scrolling issues #16366

Closed
wants to merge 1 commit into from

Conversation

antonis
Copy link

@antonis antonis commented Apr 23, 2021

Fixes wordpress-mobile/gutenberg-mobile#3104

gutenberg: WordPress/gutenberg#30994
gutenberg-mobile: wordpress-mobile/gutenberg-mobile#3394
WordPress-Android: wordpress-mobile/WordPress-Android#14492

To test:
Follow the steps on the gutenberg pr

Regression Notes

  1. Potential unintended areas of impact
    None identified

  2. What I did to test those areas of impact (or what existing automated tests I relied on)
    Followed testing steps on the gutenberg pr

  3. What automated tests I added (or what prevented me from doing so)
    This PR only updates the gutenberg-mobile version and does not introduce iOS native code.

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • 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 Apr 23, 2021

You can trigger an installable build for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Apr 23, 2021

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

@antonis
Copy link
Author

antonis commented Apr 27, 2021

Build 47260 is available on AppCenter

@antonis antonis marked this pull request as ready for review April 27, 2021 11:07
@antonis antonis added this to the 17.3 milestone Apr 27, 2021
@antonis antonis self-assigned this Apr 27, 2021
@antonis antonis requested review from dcalhoun and geriux April 27, 2021 11:07
@antonis antonis modified the milestones: 17.3, 17.4 Apr 29, 2021
@antonis antonis force-pushed the gutenberg/issue/30878-colorpicker branch from 31055ff to 8f64b90 Compare May 6, 2021 11:54
Copy link
Member

@dcalhoun dcalhoun left a comment

Choose a reason for hiding this comment

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

@antonis
Copy link
Author

antonis commented May 7, 2021

Closing this since it was meant for testing WordPress/gutenberg#30994

@antonis antonis closed this May 7, 2021
@peril-wordpress-mobile
Copy link

Fails
🚫 Podfile: reference to a commit hash

Generated by 🚫 dangerJS

@jkmassel jkmassel deleted the gutenberg/issue/30878-colorpicker branch July 26, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants