Skip to content

bug fixed: Transition.downToUp have a black background container #1560#2864

Open
Leobuaa wants to merge 3 commits intojonataslaw:masterfrom
Leobuaa:master
Open

bug fixed: Transition.downToUp have a black background container #1560#2864
Leobuaa wants to merge 3 commits intojonataslaw:masterfrom
Leobuaa:master

Conversation

@Leobuaa
Copy link
Copy Markdown

@Leobuaa Leobuaa commented Aug 7, 2023

  • I add a parameter oldPage on api GetNavigationExt.to(), oldPage is the previous page when you push a new page, if you use a custom transition, we need to change the old page exit transition, otherwise it will show black container in some cases, it fixes the bug:
    transition: Transition.downToUp have a black background container #1560

  • When we use Get.to(() => NextPage(), transition: Transition.downToUp), we should change the previous page's exit transition to SlideTopTransition, now use new api: Get.to(() => NextPage(), transition: Transition.downToUp, oldPage: currentPage)

before:

before.mp4

after:

after.mp4

Every PR must update the corresponding documentation in the code, and also the readme in english with the following changes.

Pre-launch Checklist

  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
  • All existing and new tests are passing.

@Leobuaa
Copy link
Copy Markdown
Author

Leobuaa commented Aug 9, 2023

@jonataslaw

@Leobuaa Leobuaa marked this pull request as draft August 21, 2023 08:22
@Leobuaa Leobuaa marked this pull request as ready for review August 21, 2023 08:23
@jonataslaw
Copy link
Copy Markdown
Owner

Thank you very much for this PR!
I'm just wondering if there's a way to do this dynamically, without having to pass a page by parameter.

In fact, I think this problem only occurs on iOS, due to the cupertino animation, but this can be disabled.

If it only happens on iOS, we could disable the previous page animation when the animation is vertical.

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