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

Flyout with FlyoutBehavior="Locked" unlocks (reverts to Flyout) after navigation. #27337

Closed
lendres opened this issue Jan 24, 2025 · 4 comments · Fixed by #27379
Closed

Flyout with FlyoutBehavior="Locked" unlocks (reverts to Flyout) after navigation. #27337

lendres opened this issue Jan 24, 2025 · 4 comments · Fixed by #27379
Assignees
Labels
i/regression This issue described a confirmed regression on a currently supported version p/0 Work that we can't release without partner/syncfusion Issues / PR's with Syncfusion collaboration platform/windows 🪟 regressed-in-9.0.30 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@lendres
Copy link

lendres commented Jan 24, 2025

Description

Flyouts set to locked (FlyoutBehavior="Locked") seem to switch to behavior associated with FlyoutBehavior="Flyout" after navigation is performed.

  • At application startup, the behavior seems correct.
  • After navigating to another page and back, the flyout behavior seems to have changed.
  • This behavior seems associated with Maui Controls 9.0.30. The behavior is correct in Maui Controls 9.0.14.

Steps to Reproduce

  1. Download the example repository and build the branch maui-controls-9.0.30. Be sure to select this branch, it is not the default.
  2. Observed that the flyout is lock.
    Image
  3. Navigate to the second page.
    Image
  4. Navigate back to the first page and observe that the hamburger button is present and the flyout is not locked.
    Image
  5. Observe that the hamburger icon can be clicked to change the flyout to collapsed.
    Image

Note

Use the branch maui-controls-9.0.14 of the example repository and retest to observe that this behavior was correct in version 9.0.14.

Link to public reproduction project repository

https://github.com/lendres/Maui-Flyout-Unlocking

Version with bug

9.0.30 SR3

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

9.0.14 SR1.4

Affected platforms

Windows

Affected platform versions

10.0.22631 Build 22631

Did you find any workaround?

None is known.

Relevant log output

@lendres lendres added the t/bug Something isn't working label Jan 24, 2025
@lendres lendres changed the title Flyout with FlyoutBehavior="Locked" _unlocks_ (reverts to Flyout) after navigation. Flyout with FlyoutBehavior="Locked" unlocks (reverts to Flyout) after navigation. Jan 24, 2025
@kubaflo
Copy link
Contributor

kubaflo commented Jan 24, 2025

It works fine on Android and iOS, so it is probably Windows only issue

@Zhanglirong-Winnie Zhanglirong-Winnie added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed i/regression This issue described a confirmed regression on a currently supported version labels Jan 26, 2025
@Zhanglirong-Winnie
Copy link

This issue has been verified Visual Studio 17.13 Preview 3(9.0.30 ). Can repro this issue on Windows platform. On 9.0.22 works fine.

@hwsmo
Copy link

hwsmo commented Jan 26, 2025

Please solve the problem promptly, as no workaround can be found so far :(

@vishnumenon2684 vishnumenon2684 added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Jan 27, 2025
@vishnumenon2684 vishnumenon2684 self-assigned this Jan 27, 2025
@PureWeen PureWeen added this to the .NET 9 SR4 milestone Jan 28, 2025
@PureWeen PureWeen moved this from Todo to In Progress in MAUI SDK Ongoing Jan 28, 2025
@PureWeen PureWeen added the p/0 Work that we can't release without label Jan 28, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in MAUI SDK Ongoing Feb 5, 2025
@hwsmo
Copy link

hwsmo commented Feb 15, 2025

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:

Image

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.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
i/regression This issue described a confirmed regression on a currently supported version p/0 Work that we can't release without partner/syncfusion Issues / PR's with Syncfusion collaboration platform/windows 🪟 regressed-in-9.0.30 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Status: Done
6 participants