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

Revert changes in PR11529 and PR11761 #12737

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

LeafShi1
Copy link
Member

@LeafShi1 LeafShi1 commented Jan 9, 2025

Fixes #12590

Root Cause

In order to fix the flickering issue#2053861, I modified the drawing logic of ComboBox in #11529, this leads to the current issue and issue #11760.

Proposed changes

Customer Impact

  • ComboBox dropdown button can be shown normally when switching RightToLeft property, or recreating the combobox for any other reason.

Regression?

  • Yes

Risk

  • Minimal

Before

When change the RightToLeft property after expanding the Popup FlatStyle comboBox, the dropdown button on the comboBox disappeared
Image

After

The dropdown button on the comboBox can be show normally
AfterChanges

Test methodology

  • Manually

Test environment(s)

  • .net 10.0.0-alpha.1.24605.1
Microsoft Reviewers: Open in CodeFlow

Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 62.50000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 76.15371%. Comparing base (a54a4a9) to head (bbe67a5).
Report is 8 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #12737         +/-   ##
===================================================
+ Coverage   76.12836%   76.15371%   +0.02535%     
===================================================
  Files           3188        3191          +3     
  Lines         640086      640087          +1     
  Branches       47233       47230          -3     
===================================================
+ Hits          487287      487450        +163     
+ Misses        149269      149120        -149     
+ Partials        3530        3517         -13     
Flag Coverage Δ
Debug 76.15371% <62.50000%> (+0.02535%) ⬆️
integration 18.16891% <0.00000%> (+0.00183%) ⬆️
production 50.06555% <62.50000%> (+0.05324%) ⬆️
test 97.02819% <ø> (-0.00009%) ⬇️
unit 47.27376% <62.50000%> (+0.03053%) ⬆️

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

@Tanya-Solyanik Tanya-Solyanik added the 💥 regression-release Regression from a public release label Jan 9, 2025
Copy link
Member

@Tanya-Solyanik Tanya-Solyanik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Please re-activate the associated bug and investigate an alternative fix. We'll service this fix if we receive customer reports.

@LeafShi1 LeafShi1 merged commit f5fef93 into dotnet:main Jan 9, 2025
8 checks passed
@LeafShi1 LeafShi1 deleted the Revert_changes_in_PR11529 branch January 9, 2025 05:40
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0 Preview1 milestone Jan 9, 2025
@ricardobossan
Copy link
Member

/backport to release/9.0

Copy link
Contributor

Started backporting to release/9.0: https://github.com/dotnet/winforms/actions/runs/12768052150

Copy link
Contributor

@ricardobossan backporting to release/9.0 failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Revert 11761
Using index info to reconstruct a base tree...
M	src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.FlatComboAdapter.cs
M	src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.cs
M	src/System.Windows.Forms/src/System/Windows/Forms/Controls/ToolStrips/ToolStripComboBox.ToolStripComboBoxControl.ToolStripComboBoxFlatComboAdapter.cs
Falling back to patching base and 3-way merge...
Auto-merging src/System.Windows.Forms/src/System/Windows/Forms/Controls/ToolStrips/ToolStripComboBox.ToolStripComboBoxControl.ToolStripComboBoxFlatComboAdapter.cs
Auto-merging src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.cs
Auto-merging src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.FlatComboAdapter.cs
Applying: Revert PR 11529
Using index info to reconstruct a base tree...
M	src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.FlatComboAdapter.cs
M	src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.cs
Falling back to patching base and 3-way merge...
Auto-merging src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.cs
Auto-merging src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.FlatComboAdapter.cs
CONFLICT (content): Merge conflict in src/System.Windows.Forms/src/System/Windows/Forms/Controls/ComboBox/ComboBox.FlatComboAdapter.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0002 Revert PR 11529
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💥 regression-release Regression from a public release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ComboBox dropdown button disappear when switching RightToLeft property
3 participants