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

NavView Minimal Pane Sometimes Transparent OnLoad / OnWindowResize #5531

Merged
merged 5 commits into from
Jul 28, 2021

Conversation

karkarl
Copy link
Contributor

@karkarl karkarl commented Jul 21, 2021

Description

NavigationView sometimes appear as transparent when in NavigationViewDisplayMode::Minimal OnLoad and OnWindowResize. Turns out this is due to SplitView and NavigationView becoming out of sync, causing SplitView VisualState to evaluate to OpenOverlayLeft while NavigationView pane is closed while minimal.

Motivation and Context

Fixes #5342

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Jul 21, 2021
@karkarl
Copy link
Contributor Author

karkarl commented Jul 21, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@beervoley beervoley added area-NavigationView NavView control team-Controls Issue for the Controls team labels Jul 22, 2021
@StephenLPeters StephenLPeters removed the needs-triage Issue needs to be triaged by the area owners label Jul 22, 2021
@beervoley
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@karkarl
Copy link
Contributor Author

karkarl commented Jul 26, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@karkarl
Copy link
Contributor Author

karkarl commented Jul 27, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@karkarl karkarl force-pushed the user/karenlai/NavViewTransparentPaneFix branch from 72df4c3 to 8f0b8af Compare July 28, 2021 00:32
@karkarl
Copy link
Contributor Author

karkarl commented Jul 28, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@karkarl karkarl merged commit ef92e9a into main Jul 28, 2021
@karkarl karkarl deleted the user/karenlai/NavViewTransparentPaneFix branch July 28, 2021 18:44
@ranjeshj ranjeshj added the needs-cherrypicktorelease PR tagged for cherry-pick to the current release branch (but not yet picked) label Aug 4, 2021
kmahone pushed a commit that referenced this pull request Aug 10, 2021
…5531)

* force close pane for minimal

* changes from comments

* add minWidth to compactModeThresholdWidth

* update verification files

* update verification files
ranjeshj added a commit that referenced this pull request Aug 18, 2021
@ghost
Copy link

ghost commented Aug 19, 2021

🎉Microsoft.UI.Xaml v2.6.2-prerelease.210818003 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-NavigationView NavView control team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigation view pane background can appear transparent after resizing window while pane is in overlay state
4 participants