-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[Windows] - Resolved FlyoutBehavior "Locked" State Reset Issue After Navigation #27379
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
Conversation
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
App.Tap("SecondPageButton"); | ||
App.WaitForElement("GoBackButton"); | ||
App.Tap("GoBackButton"); | ||
VerifyScreenshot(); |
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.
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.
@jsuarezruiz, Thank you for highlighting this! In the latest commit, the test case was updated by replacing VerifyScreenShot with WaitForNoElement to ensure the absence of the Flyout icon. This modification eliminates the need to add snapshots for the test.
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/rebase |
9c4d1d6
to
da0ea0a
Compare
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
|
Hi! Unfortunately, this change was not a good solution. Reason: In the delegate "getDefaultNavBarIsVisible" the current FlyoutBehavior value of the _shell is no longer accessed, but rather the previously determined value. After navigating to another page in the app and navigating back again, the visibility of the drawer toggle button is no longer updated correctly when the FlyoutBehavior is changed from the state "Locked" to "Flyout" This is exactly the case I have in my app now. |
Root Cause
The visibility of the FlyoutIcon is determined by the DrawerToggleVisible value. In ShellView, the condition for DrawerToggleVisible was solely based on the StackCount and did not account for scenarios where the Shell was in the Flyout state. This caused the DrawerToggleVisible to return true incorrectly, leading to the FlyoutIcon displayed after back navigation.
Description of Change
Updated the DrawerToggleVisible value to account for FlyoutBehavior, ensuring that the FlyoutIcon visibility accurately reflects the Shell's Flyout state. This prevents the FlyoutIcon from being incorrectly displayed after navigation when the Shell is in Disabled and Locked mode.
Issues Fixed
Fixes #27337
Tested the behaviour in the following platforms
Ouput
BeforeFix.mp4
AfterFix.mp4