Update the drawing position of the Treeview control text

[LeafShi1]

Fixes #12601

Proposed changes

• Update the drawing position of the Treeview control text

Customer Impact

• The check box of the TreeView control can be fully displayed

Regression?

• No

Risk

• Minimal

Screenshots

Before

Create a form, add a TreevView with CheckBoxes=true and DrawMode=OwnerDrawText
In OnDrawNode event only set DrawDefault=true
Show the form: checkboxes are shown corrupted on the right border

After

The checkbox of the Treeview control can be fully displayed

Test methodology

• Manually

Test environment(s)

• .net 10.0.0-alpha.1.24611.1

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

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

Project coverage is 76.01356%. Comparing base (c9c06a5) to head (e9204ba).
Report is 29 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #12638         +/-   ##
===================================================
- Coverage   76.01450%   76.01356%   -0.00094%     
===================================================
  Files           3176        3176                 
  Lines         638840      638840                 
  Branches       47169       47169                 
===================================================
- Hits          485611      485605          -6     
- Misses        149718      149733         +15     
+ Partials        3511        3502          -9     

Flag	Coverage Δ
Debug	76.01356% <0.00000%> (-0.00094%)	⬇️
integration	18.13630% <0.00000%> (-0.00777%)	⬇️
production	49.79352% <0.00000%> (-0.00212%)	⬇️
test	97.03765% <ø> (ø)
unit	47.01537% <0.00000%> (+0.00210%)	⬆️

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

[ricardobossan]

I tested the PR, and the reported issue appears to be resolved:

However, when DrawMode is set to OwnerDrawText, the node names disappear. Is this the intended behavior?

Other than that, the code looks good to me.

[Tanya-Solyanik]

@LeafShi1 @ricardobossan - had you tested this fix in different DPI settings?

[LeafShi1]

@LeafShi1 @ricardobossan - had you tested this fix in different DPI settings?

Tested on 225% DPI (The main screen is 225% DPI, the secondary screen is 100% DPI)

[Tanya-Solyanik]

@LeafShi1 - this might be a good issue to make conditional on an AppContext switch, we'll have to discuss it in a team meeting.
@merriemcgaw - we are changing layout by 1 pixel, it might be visible if customer had specific background to the tree nodes. DO you think this warrants an opt-out?