-
Notifications
You must be signed in to change notification settings - Fork 985
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 issue 12187 Extra Border Color in DataGridView with RightToLeft. #12210
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #12210 +/- ##
===================================================
+ Coverage 75.16050% 75.39022% +0.22972%
===================================================
Files 3070 3100 +30
Lines 632557 634378 +1821
Branches 46787 46865 +78
===================================================
+ Hits 475433 478259 +2826
+ Misses 153740 152710 -1030
- Partials 3384 3409 +25
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Maybe we need further testing by our test team to see if the fix is good under different type mode settings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All LGTM
I agree with Leaf, this should be tested thoroughly. @Syareel-Sukeri - could you please test the PR? |
src/System.Windows.Forms/src/System/Windows/Forms/Controls/DataGridView/DataGridViewRow.cs
Outdated
Show resolved
Hide resolved
@Epica3055 This PR changes cell size compared to NET8. Could you please investigate if we can adjust color value here or elsewhere - winforms/src/System.Windows.Forms/src/System/Windows/Forms/Controls/DataGridView/DataGridViewCell.cs Line 3154 in 7c9f958
|
e3ea34c
to
f7107b4
Compare
Fixes #5961
Proposed changes
Regression?
Screenshots
Before
After
When no color applied
Test methodology
Test environment(s)
Microsoft Reviewers: Open in CodeFlow