Skip to content

Conversation

@dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Oct 12, 2021

Related PRs

Description

Fixes #4100. When navigating to a Help section child screen, the parent screen would remain visible outside of the container. Hiding the overflow of the containing element fixes this.

The reason this occurred for the first time now is likely because it is the first time we have utilized the default navigation transitions for the platforms, which is provided by React Navigation. This means the screen animates from right to left on iOS, rather than a cross-fade.

To test: See WordPress/gutenberg#35552.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@dcalhoun dcalhoun added the [Type] Bug Something isn't working label Oct 12, 2021
@dcalhoun dcalhoun self-assigned this Oct 12, 2021
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Oct 12, 2021

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

@twstokes twstokes self-requested a review October 12, 2021 15:05
@twstokes
Copy link
Contributor

LGTM. Tests passed for me @dcalhoun!

@dcalhoun dcalhoun enabled auto-merge October 12, 2021 16:52
@dcalhoun dcalhoun merged commit c503a0c into develop Oct 12, 2021
@dcalhoun dcalhoun deleted the bug/help-section-overflow branch October 12, 2021 17:28
@dcalhoun dcalhoun mentioned this pull request Oct 12, 2021
5 tasks
@dcalhoun dcalhoun added this to the 1.63.1 (18.4) milestone Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Help view erroneously reveals parent screen when navigating to child sheet

3 participants