Fix ToolStripStatusLabel text visibility in High Contrast mode with System RenderMode#14110
Fix ToolStripStatusLabel text visibility in High Contrast mode with System RenderMode#14110SimonZhao888 wants to merge 2 commits intodotnet:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR fixes a visibility issue with ToolStripStatusLabel text in High Contrast mode when using System RenderMode. The fix ensures that text is properly highlighted against the background, making it readable for users who rely on high contrast accessibility features.
Key Changes:
- Moved
textRectdeclaration earlier in the method to enable its use in the high contrast highlight logic - Added highlight background rendering (
FillRectangle) for pressed and selected states in high contrast mode
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #14110 +/- ##
===================================================
- Coverage 77.15789% 77.15384% -0.00405%
===================================================
Files 3279 3279
Lines 645317 645325 +8
Branches 47718 47720 +2
===================================================
- Hits 497913 497893 -20
- Misses 143717 143745 +28
Partials 3687 3687
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Hi, a couple of observations on this change: 1. Text color may not contrast with the Highlight background (ToolStripLabel.cs, Pressed/Selected branches): The fix correctly adds 2. Duplicated High Contrast logic (lines ~363-366 and ~371-374): The identical (Copilot co-authored on Klaus' behalf.) |
Fixes #14098
Proposed changes
Customer Impact
Regression?
Risk
Screenshots
Before
RuntimeIssue.mp4
After
ToolStripLabel.mp4
Test methodology
Microsoft Reviewers: Open in CodeFlow