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

Retrieve color data correctly in ListView #13062

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

JeremyKuhne
Copy link
Member

@JeremyKuhne JeremyKuhne commented Mar 3, 2025

We accidentally dropped the call to ColorTranslator, which properly handles alpha when converting COLORREF. COLORREF's implicit conversion does this correctly.

Fixes #13048

Microsoft Reviewers: Open in CodeFlow

We accidentally dropped the call to ColorTranslator, which properly handles alpha when converting COLORREF. COLORREF's implicit conversion does this correctly.

Fixes dotnet#13048
@JeremyKuhne JeremyKuhne requested a review from a team as a code owner March 3, 2025 18:27
Copy link

codecov bot commented Mar 3, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 76.10329%. Comparing base (4bcc8a9) to head (b418251).
Report is 2 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                  @@
##                main      #13062          +/-   ##
====================================================
- Coverage   96.95044%   76.10329%   -20.84715%     
====================================================
  Files           1225        3275        +2050     
  Lines         357134      643804      +286670     
  Branches        5537       47445       +41908     
====================================================
+ Hits          346243      489956      +143713     
- Misses         10049      150291      +140242     
- Partials         842        3557        +2715     
Flag Coverage Δ
Debug 76.10329% <0.00000%> (-20.84715%) ⬇️
integration 18.04141% <0.00000%> (?)
production 50.13186% <0.00000%> (?)
test 96.95044% <ø> (ø)
unit 47.53828% <0.00000%> (?)

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

@JeremyKuhne JeremyKuhne merged commit 9488747 into dotnet:main Mar 3, 2025
8 checks passed
@JeremyKuhne JeremyKuhne deleted the colorreffix branch March 3, 2025 19:55
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0 Preview3 milestone Mar 3, 2025
LeafShi1 pushed a commit to LeafShi1/winforms that referenced this pull request Mar 6, 2025
We accidentally dropped the call to ColorTranslator, which properly handles alpha when converting COLORREF. COLORREF's implicit conversion does this correctly.

Fixes dotnet#13048
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.

BackColor and ForeColor in DrawListViewColumnHeaderEventArgs have alpha component set to 0
2 participants