Skip to content

Themes - #3

Merged
Nechja merged 16 commits into
mainfrom
feature/theme-consolidation
Feb 8, 2026
Merged

Themes#3
Nechja merged 16 commits into
mainfrom
feature/theme-consolidation

Conversation

@Nechja

@Nechja Nechja commented Feb 8, 2026

Copy link
Copy Markdown
Owner

Added theming and removed hard coded colors.

Robot junk follows:
Theme system enhancements:

  • Added HighContrastDark and Soft theme modes to the ThemeMode enum, and updated theme switching logic in ThemeService to support them. [1] [2]
  • Defined extensive color palettes for both new themes in ThemeColors.cs, including primary, accent, background, text, state, input, and navigation colors.
  • Updated the resource assignment logic in ThemeService to handle all new color properties for "HighContrastDark" and "Soft" themes, as well as expanded semantic colors for existing themes. [1] [2] [3]

UI styling improvements:

  • Added shared button styles in App.axaml for action, danger, restart, accent, subtle, and outlined buttons, including pointer-over states for visual feedback.

Navigation and property updates:

  • Modified navigation text color logic in MainWindowViewModel to use the new NavigationSelectedColor for active sections, and added properties for new theme modes to support UI binding and state updates. [1] [2] [3]

Expanded semantic colors:

  • Added new semantic colors (danger, restart, action accent, input, contrast text, network accent, logs, navigation) to ThemeColors.Dark and ThemeColors.Light for richer UI feedback and accessibility.

These changes collectively make the application's UI more flexible, visually consistent, and accessible across a wider range of user preferences and needs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Introduces an expanded theming system (new theme modes + semantic color resources) and refactors UI XAML to consume theme resources instead of hard-coded colors, aiming for more consistent styling and accessibility.

Changes:

  • Added HighContrastDark and Soft to ThemeMode, with theme switching commands/bindings and palette definitions.
  • Centralized color usage via DynamicResource keys (background/surface/text/semantic action colors) across multiple views/components.
  • Added shared button styles (action/danger/restart/accent/subtle/outlined) in App.axaml and updated components to use class-based styling.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
Views/MainWindow.axaml Removes local button style (moved to app-level shared styles).
Views/LogsWindow.axaml Replaces hard-coded colors/gradient with theme resources and shared button styles.
Views/Dialogs/CreateContainerDialog.axaml Migrates dialog styling to theme resources and shared outlined button style.
Views/Components/VolumeCard.axaml Uses shared danger button class instead of hard-coded styling.
Views/Components/StackGroup.axaml Applies shared button classes and introduces theme resource usage for “Remove All”.
Views/Components/SettingsView.axaml Adds radio options/bindings for new theme modes.
Views/Components/NetworkCard.axaml Uses NetworkAccentColor resource and shared danger button class.
Views/Components/ImageCard.axaml Migrates action buttons to shared classes and theme resources.
Views/Components/ContainerCard.axaml Switches action foreground colors to semantic theme resources.
ViewModels/MainWindowViewModel.cs Adds new theme state properties/commands and updates navigation selected color logic.
Services/ThemeService.cs Extends theme variant application and populates additional semantic resource keys for all themes.
Models/Enums.cs Adds HighContrastDark and Soft to ThemeMode.
Configuration/ThemeColors.cs Defines new palettes + adds semantic colors for Dark/Light.
App.axaml Adds global shared button styles used across views/components.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ViewModels/MainWindowViewModel.cs Outdated
Comment thread ViewModels/MainWindowViewModel.cs Outdated
Comment thread ViewModels/MainWindowViewModel.cs
Comment thread ViewModels/MainWindowViewModel.cs
Comment thread ViewModels/MainWindowViewModel.cs
Comment thread ViewModels/MainWindowViewModel.cs Outdated
Comment thread App.axaml
Comment thread Views/Components/StackGroup.axaml Outdated
Comment thread Views/Components/ImageCard.axaml Outdated
Comment thread ViewModels/MainWindowViewModel.cs
@Nechja
Nechja merged commit 39afc67 into main Feb 8, 2026
1 check passed
@Nechja
Nechja deleted the feature/theme-consolidation branch February 8, 2026 05:22
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.

2 participants