Skip to content

WinUI: Reduce navigation bar item size to fit long translations#4741

Merged
Marc-André Moreau (mamoreau-devolutions) merged 1 commit into
mainfrom
NavBarSize
May 11, 2026
Merged

WinUI: Reduce navigation bar item size to fit long translations#4741
Marc-André Moreau (mamoreau-devolutions) merged 1 commit into
mainfrom
NavBarSize

Conversation

@GabrielDuf
Copy link
Copy Markdown
Contributor

This pull request focuses on refining the UI by reducing the size and spacing of several navigation and avatar elements to create a more compact and visually consistent interface. The main changes involve adjusting icon and control sizes, margins, and font sizes across navigation items and user avatar components.

UI Compactness and Visual Consistency:

  • Reduced the icon size in CustomNavViewItem from 24px to 20px and adjusted the progress ring's size and margin to match. The height of the navigation item was also decreased from 54px to 48px. (src/UniGetUI/Controls/CustomNavViewItem.cs)
  • Decreased the CompactPaneLength of the main navigation view from 68px to 52px, making the collapsed navigation pane narrower. (src/UniGetUI/Pages/MainView.xaml)
  • Reduced the font size of all main navigation buttons from 16px to 14px for a more streamlined appearance. (src/UniGetUI/Pages/MainView.xaml)
  • Adjusted the margin of the UserAvatar control in the navigation pane footer to be smaller, aligning it with the new compact design. (src/UniGetUI/Pages/MainView.xaml)
  • Updated the PersonPicture avatar size in both login and logout controls from 36px to 32px for consistency with other UI elements. (src/UniGetUI/Services/UserAvatar.cs)
    **

issue #4738

@GabrielDuf Gabriel Dufresne (GabrielDuf) changed the title Reduce navigation bar item size to fit long translations WinUI: Reduce navigation bar item size to fit long translations May 11, 2026
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mamoreau-devolutions Marc-André Moreau (mamoreau-devolutions) merged commit 2601ac6 into main May 11, 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

Development

Successfully merging this pull request may close these issues.

3 participants