Recalculate the node position of the TreeView when DrawMode = TreeViewDrawMode.OwnerDrawText

[LeafShi1]

Fixes #12681

Root Cause

When treeView.DrawMode = TreeViewDrawMode.OwnerDrawText, the node's FillRectangle and FocusRectangle positions are calculated incorrectly

Proposed changes

• Update function CustomDraw of the TreeView.cs, When RightToLeft == RightToLeft.Yes && RightToLeftLayout reverses the X drawing coordinates of FillRectangle and FocusRectangle

Customer Impact

• When setting RightToLeft = Yes and RightToLeftLayout = True, the selected node box is fully rendered

Regression?

• No

Risk

• Minimal

Screenshots

Before

After

When setting RightToLeft = Yes, RightToLeftLayout = True and .DrawMode = TreeViewDrawMode.OwnerDrawText, the selected node box is fully rendered

Test methodology

• Manually

Test environment(s)

• .net 10.0.0-alpha.1.24628.1

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

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

Project coverage is 76.13939%. Comparing base (3b08ae7) to head (abe7f92).
Report is 24 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #12698         +/-   ##
===================================================
+ Coverage   76.02967%   76.13939%   +0.10972%     
===================================================
  Files           3181        3189          +8     
  Lines         639670      640126        +456     
  Branches       47215       47239         +24     
===================================================
+ Hits          486339      487388       +1049     
+ Misses        149819      149211        -608     
- Partials        3512        3527         +15     

Flag	Coverage Δ
Debug	76.13939% <0.00000%> (+0.10972%)	⬆️
integration	18.17291% <0.00000%> (+0.01002%)	⬆️
production	50.03550% <0.00000%> (+0.21632%)	⬆️
test	97.02825% <ø> (+0.00078%)	⬆️
unit	47.26545% <0.00000%> (+0.21446%)	⬆️

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

[ricardobossan]

I proceeded to test the PR and the issue seems to have been solved:

The code LGTM!

[Tanya-Solyanik]

Had you tested this change at different scaling factors and in different level nodes in a tree view? For example 3rd level child?
It will be helpful to add code comments that describe what the constants are for.