Skip to content

Dark mode (g) fix combo box renderer #13502

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

Open
wants to merge 63 commits into
base: klaus/preview5/F
Choose a base branch
from

Conversation

KlausLoeffelmann
Copy link
Member

@KlausLoeffelmann KlausLoeffelmann commented May 21, 2025

Fixes dark mode issues around the ComboBox.

Microsoft Reviewers: Open in CodeFlow

@github-actions github-actions bot added the area-DarkMode Issues relating to Dark Mode feature label May 21, 2025
@KlausLoeffelmann KlausLoeffelmann changed the base branch from main to klaus/preview5/F May 21, 2025 21:29
@dotnet-policy-service dotnet-policy-service bot added the draft draft PR label May 22, 2025
LeafShi1 and others added 3 commits May 22, 2025 09:03
Update Pre-Release Iteration and Resource Management Milestone to
preview6
###### Microsoft Reviewers: [Open in
CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/dotnet/winforms/pull/13483)
[main] Source code updates from dotnet/dotnet
[main] Source code updates from dotnet/dotnet
@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch from 87e4d23 to 9ef2994 Compare May 23, 2025 14:33
[main] Source code updates from dotnet/dotnet
@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch 3 times, most recently from 203787d to 9ef2994 Compare May 24, 2025 23:42
[main] Source code updates from dotnet/dotnet
@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch from 9ef2994 to 80e1794 Compare May 26, 2025 07:48
@KlausLoeffelmann KlausLoeffelmann changed the title Dark mode g fix combo box renderer Dark mode (g) fix combo box renderer May 26, 2025
@KlausLoeffelmann KlausLoeffelmann changed the base branch from klaus/preview5/F to main May 26, 2025 07:51
@KlausLoeffelmann KlausLoeffelmann marked this pull request as ready for review May 26, 2025 07:52
@KlausLoeffelmann KlausLoeffelmann requested a review from a team as a code owner May 26, 2025 07:52
@KlausLoeffelmann KlausLoeffelmann removed the draft draft PR label May 26, 2025
Copy link

codecov bot commented May 26, 2025

Codecov Report

Attention: Patch coverage is 28.82704% with 1432 lines in your changes missing coverage. Please review.

Project coverage is 76.47392%. Comparing base (8484b55) to head (992a23a).
Report is 8 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #13502         +/-   ##
===================================================
- Coverage   76.59415%   76.47392%   -0.12023%     
===================================================
  Files           3230        3240         +10     
  Lines         639165      640298       +1133     
  Branches       47297       47415        +118     
===================================================
+ Hits          489563      489661         +98     
- Misses        146025      147060       +1035     
  Partials        3577        3577                 
Flag Coverage Δ
Debug 76.47392% <28.82704%> (-0.12023%) ⬇️
integration 18.73558% <17.14712%> (-0.04734%) ⬇️
production 50.82904% <28.82704%> (-0.16676%) ⬇️
test 97.40411% <ø> (ø)
unit 48.20863% <26.88867%> (-0.19267%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch from 80e1794 to 325a438 Compare May 26, 2025 20:52
dotnet-maestro bot and others added 2 commits May 27, 2025 02:45
[main] Source code updates from dotnet/dotnet
…he value to a new text box (dotnet#13526)

I think we forgot to update *pdwEffect after _owner.OnDragDrop(dragEvent);
@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch from 325a438 to 992a23a Compare May 27, 2025 18:42
@KlausLoeffelmann KlausLoeffelmann changed the base branch from main to klaus/preview5/F May 27, 2025 22:50
KlausLoeffelmann and others added 29 commits May 28, 2025 20:20
* CheckBoxes,
* RadioButtons
* Tabs.

Also:

* Move ApplyThemingImplicitly flag definition to CreateParams.
* Update approach to set ApplyThemingImplicitly control style.
* Delegate FlatStyle.Standard Button rendering to the System due to HighDpi issues in the VisualStyle-based renderers.
* By that, implement new architecture for extendeble DarkMode/Style renderers.
@KlausLoeffelmann KlausLoeffelmann force-pushed the DarkMode_g_FixComboBoxRenderer branch from 992a23a to 716c1f6 Compare May 29, 2025 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-DarkMode Issues relating to Dark Mode feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants