Skip to content

Fix AppBar Text coloring#337

Merged
TheAxelander merged 2 commits intomasterfrom
pre-release
Dec 17, 2025
Merged

Fix AppBar Text coloring#337
TheAxelander merged 2 commits intomasterfrom
pre-release

Conversation

@TheAxelander
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 17, 2025 11:56
@TheAxelander TheAxelander merged commit 61c4c61 into master Dec 17, 2025
7 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes text and icon coloring in the AppBar to ensure proper contrast and consistency across theme modes by changing from Color.Surface to Color.Inherit.

Key changes:

  • Changed icon color from Color.Surface to Color.Inherit for three separator icons in the AppBar
  • Converted HTML <a> tag to MudBlazor <MudLink> component with Color.Inherit for the Change Log link
  • Updated release date in CHANGELOG.md from "2025-xx-xx" to "2025-12-17"

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
OpenBudgeteer.Blazor/Shared/MainLayout.razor Updated AppBar separator icons and Change Log link to use Color.Inherit instead of Color.Surface, ensuring proper color inheritance from the AppBar component
CHANGELOG.md Updated version 1.11 release date to 2025-12-17

The changes are consistent with the existing AppBar styling patterns where other interactive elements (MudIconButton components) already use Color.Inherit. The use of Color.Inherit is appropriate for AppBar elements as it ensures they inherit the proper text color from the AppBar theme, providing correct contrast in both light and dark modes.


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

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.

1 participant