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

perf: Reduce inner blocks tree depth #5746

Merged

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 May 8, 2023

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

@dcalhoun dcalhoun requested a review from geriux May 10, 2023 23:33
wpmobilebot and others added 4 commits May 11, 2023 21:23
…-depth' of github.com:wordpress-mobile/gutenberg-mobile into version-toolkit/gutenberg/perf/reduce-inner-blocks-tree-depth
@geriux
Copy link
Contributor

geriux commented May 11, 2023

@dcalhoun It looks like there's a visual test that keeps failing 🤔

dcalhoun added 6 commits May 11, 2023 16:30
The block list refactor modified the Xpath for this element.
Now that rich text fields can be selected without selecting the parent,
the test query happened to select the Button block instead of the
parent Buttons block. We now navigate upwards on Android to address
this.
Android fails to display the keyboard at times, which can cause waiting
for the keyboard to hide to finish prematurely.
…into version-toolkit/gutenberg/perf/reduce-inner-blocks-tree-depth
This was erroneously included during a merge conflict resolution.
@dcalhoun
Copy link
Member

@geriux CI tasks are now passing. 🎉

Copy link
Contributor

@geriux geriux left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

dcalhoun added 2 commits May 12, 2023 08:06
…into version-toolkit/gutenberg/perf/reduce-inner-blocks-tree-depth
@dcalhoun dcalhoun enabled auto-merge May 12, 2023 12:08
…into version-toolkit/gutenberg/perf/reduce-inner-blocks-tree-depth
@dcalhoun dcalhoun merged commit b65e5a9 into trunk May 12, 2023
@dcalhoun dcalhoun deleted the version-toolkit/gutenberg/perf/reduce-inner-blocks-tree-depth branch May 12, 2023 13:42
@dcalhoun dcalhoun self-assigned this May 12, 2023
@dcalhoun dcalhoun added this to the 1.96.0 (22.5) milestone May 12, 2023
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.

3 participants