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

[Mobile] - Fix regression with the Color hook and ColorPanel #5683

Conversation

wpmobilebot
Copy link
Collaborator

Related PRs

Description

This PR is generated by version-toolkit to downstream the changes for gutenberg submodule.

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Apr 19, 2023

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

@geriux geriux force-pushed the version-toolkit/gutenberg/rnmobile/fix/color-customization-for-blocks branch from 59759c0 to eaf2810 Compare April 19, 2023 14:22
@geriux geriux force-pushed the version-toolkit/gutenberg/rnmobile/fix/color-customization-for-blocks branch from 89954dd to 05b2fb1 Compare April 19, 2023 16:09
@geriux geriux self-assigned this Apr 19, 2023
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.

@dcalhoun
Copy link
Member

The two failing CI tasks appear to be the Columns block slider visual snapshot. I have found this particular test is quite sensitive to the distance the bottom sheet is scrolled. I.e. if it is scrolled too far or too little, it will cause a test failure due to the elements moving slightly. Scroll inertia can also come into play on iOS, where the "bounce" animation may have not yet completed before the snapshot is taken.

We may need to merely update the snapshots or the scroll distance may need to be tweaked on each platform.

@geriux geriux merged commit ff6441a into trunk Apr 20, 2023
@geriux geriux deleted the version-toolkit/gutenberg/rnmobile/fix/color-customization-for-blocks branch April 20, 2023 16:31
@geriux geriux mentioned this pull request Apr 21, 2023
4 tasks
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.

3 participants