Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #12663 [Accessibility][High Contrast] ToolStrip grip icon colour contrast is less than 4.5:1 in Normal Mode and HC Desert mode #12682

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Epica3055
Copy link
Member

@Epica3055 Epica3055 commented Dec 25, 2024

Fixes #12663

root cause

dot color is not contrasting enough compared to the background color.

Proposed changes

  • Change the color to a more contrasting color compared to the background color.

Regression?

  • No

Risk

  • low

Screenshots

Before

Image

HC Aquatic
Image

HC Desert
Image

HC Dusk
Image

HC Night Sky
Image

After

image

Test methodology

  • manual,
Microsoft Reviewers: Open in CodeFlow

…olour contrast is less than 4.5:1 in Normal Mode and HC Desert mode
@Epica3055 Epica3055 requested a review from a team as a code owner December 25, 2024 08:43
Copy link

codecov bot commented Dec 25, 2024

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 76.03546%. Comparing base (58d5c74) to head (174a974).
Report is 15 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #12682         +/-   ##
===================================================
+ Coverage   76.03484%   76.03546%   +0.00061%     
===================================================
  Files           3180        3181          +1     
  Lines         639637      639670         +33     
  Branches       47213       47215          +2     
===================================================
+ Hits          486347      486376         +29     
- Misses        149773      149781          +8     
+ Partials        3517        3513          -4     
Flag Coverage Δ
Debug 76.03546% <66.66667%> (+0.00061%) ⬆️
integration 18.17487% <0.00000%> (+0.00967%) ⬆️
production 49.83006% <66.66667%> (+0.00387%) ⬆️
test 97.02915% <ø> (-0.00209%) ⬇️
unit 47.05204% <66.66667%> (+0.00365%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ricardobossan
Copy link
Member

ricardobossan commented Dec 26, 2024

All LGTM!

None

none

Night Sky

Night sky

Aquatic

Aquatic

Desert

desert

Dusk

dusk

Copy link
Member

@LeafShi1 LeafShi1 left a comment

Choose a reason for hiding this comment

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

It seems that the ToolStripProfessionalRenderer has been changed. Can you confirm whether this problem also exists in other rendering modes?

@LeafShi1 LeafShi1 added the waiting-review This item is waiting on review by one or more members of team label Dec 27, 2024
@Tanya-Solyanik Tanya-Solyanik added 📭 waiting-author-feedback The team requires more information from the author 📬 waiting-for-testing The PR is awaiting manual testing by the primary team; no action is yet required from the author(s) and removed waiting-review This item is waiting on review by one or more members of team labels Dec 27, 2024
@Tanya-Solyanik
Copy link
Member

Please get this fix tested on Win10 themes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬 waiting-for-testing The PR is awaiting manual testing by the primary team; no action is yet required from the author(s) 📭 waiting-author-feedback The team requires more information from the author
Projects
None yet
4 participants