Skip to content

Add sticky header support and modernize MainWindow UI infrastructure and safer elevation handling#2

Merged
Maxhy merged 2 commits into
mainfrom
develop
Jun 5, 2026
Merged

Add sticky header support and modernize MainWindow UI infrastructure and safer elevation handling#2
Maxhy merged 2 commits into
mainfrom
develop

Conversation

@aurelm-w

@aurelm-w aurelm-w commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

This PR introduces sticky header infrastructure and refactors for the MainWindow to improve maintainability, UX and robustness.
It also improved the elevation flow to be more robust. There are also some bug fixes and refactors to improve code stability and reliability.

  • MainWindow improvements and sticky header overlay support in MainWindow.xaml
  • Refactored the top application bar layout with a Grid instead of manual positioning
  • Added StickyHeaderSupport and StickyHeader properties
  • Sticky header support and reusable sticky headers across pages
  • Added IStickyHeaderSupport integration to NavItem
  • Reorganized the content area to support sticky header overlays
  • Elevation handling : hardened administrator detection with exception handling and improved UAC elevation flow (fix App crashes when canceling elevation request during settings access #1 )
  • Improved responsiveness and removed UI positioning workarounds
  • Refactored code for stability and optimization

@codacy-production

codacy-production Bot commented Jun 5, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 5 complexity · 0 duplication

Metric Results
Complexity 5
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Maxhy Maxhy self-requested a review June 5, 2026 11:04
@Maxhy Maxhy merged commit 9e1dfda into main Jun 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App crashes when canceling elevation request during settings access

2 participants