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

Settings: Use documented NavigationView adaptive behavior and always set title bar #2628

Merged
merged 1 commit into from
Dec 17, 2020

Conversation

lukeblevins
Copy link
Contributor

@lukeblevins lukeblevins commented Dec 17, 2020

  • Remove AdaptiveTrigger for window width in favor of using the PaneDisplayMode="Auto" setting with CompactModeThresholdWidth="0" for identical behavior
  • Fix bug where the titlebar (custom drag region) was only set when the Settings page is first created (fixes Can't drag the window when in Settings page #1498)

@lukeblevins lukeblevins requested a review from yaira2 December 17, 2020 17:26
@lukeblevins lukeblevins linked an issue Dec 17, 2020 that may be closed by this pull request
@yaira2 yaira2 added the ready to merge Pull requests that are approved and ready to merge label Dec 17, 2020
@yaira2 yaira2 merged commit 73dba2b into master Dec 17, 2020
@yaira2 yaira2 deleted the settings-titlebar branch December 17, 2020 18:21
@yaira2
Copy link
Member

yaira2 commented Dec 17, 2020

@duke7553 Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't drag the window when in Settings page
2 participants