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
Fix CupertinoSliverNavigationBar's alwaysShowMiddle not working properly during page transition #120895
Conversation
@DavideDessi319 there's been some infrastructure issues with the tests. Could you please rebase onto the latest master branch? |
…nBar page transition if alwaysShowMiddle is set to false
d3db779
to
e78242d
Compare
@MitchellGoodwin apologies for the delay on the rebase, it should be done now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for taking the time to fix this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a few style guide nits!
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
…king properly during page transition (flutter/flutter#120895)
This change fixes the CupertinoSliverNavigationBar's
middle
Widget showing for an instant during a page transition when the nav bar is not collapsed andalwaysShowMiddle
is set to false. The PR adds a check on the transition'shasUserMiddle
property based on thealwaysShowMiddle
and the collapsed/expanded state of the nav bar.Fixes #119522
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.