Skip to content

Fix Settings page text not adopting High Contrast Aquatic/Desert themes#2463

Merged
emmche merged 1 commit into
microsoft:mainfrom
emmche:emmche/settings-theme-text
May 21, 2026
Merged

Fix Settings page text not adopting High Contrast Aquatic/Desert themes#2463
emmche merged 1 commit into
microsoft:mainfrom
emmche:emmche/settings-theme-text

Conversation

@emmche
Copy link
Copy Markdown
Contributor

@emmche emmche commented May 12, 2026

Fixes

Text are not adopting the high contrast aquatic and desert mode in settings page.

Description of the changes

Add HighContrast theme dictionary to Settings.xaml that overrides TextFillColorPrimaryBrush and TextFillColorSecondaryBrush to use SystemColorWindowTextColor. Also fix SettingsRichTextBlockStyle using TextFillColorPrimary (a Color) instead of TextFillColorPrimaryBrush (a Brush) for the Foreground property.

How changes were validated

Manual testing with accessibility themes
Before
aquatic_before

desert_before

After
aquatic_after
desert_after

Add HighContrast theme dictionary to Settings.xaml that overrides
TextFillColorPrimaryBrush and TextFillColorSecondaryBrush to use
SystemColorWindowTextColor. Also fix SettingsRichTextBlockStyle using
TextFillColorPrimary (a Color) instead of TextFillColorPrimaryBrush
(a Brush) for the Foreground property.

Fixes AB#40625114

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@emmche emmche marked this pull request as ready for review May 12, 2026 07:33
@emmche emmche requested a review from Copilot May 12, 2026 07:33
Copy link
Copy Markdown
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 Settings page text colors not updating correctly when Windows High Contrast themes (e.g., Aquatic/Desert) are enabled, by adding HighContrast theme-specific resource overrides and correcting an incorrect RichTextBlock foreground resource type.

Changes:

  • Added a HighContrast ThemeDictionary in Settings.xaml overriding TextFillColorPrimaryBrush / TextFillColorSecondaryBrush to use SystemColorWindowTextColor.
  • Fixed SettingsRichTextBlockStyle to use TextFillColorPrimaryBrush (Brush) instead of TextFillColorPrimary (Color) for Foreground.

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

@emmche emmche merged commit c1da26f into microsoft:main May 21, 2026
9 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.

6 participants