winforms
winforms copied to clipboard
fix issue #13648 DemoConsole app: The extra number displayed on Name & Text Propeties for the ToolStrip control
Fixes #13648
Proposed changes
- We need to match the exact type name shouldn't just use StartsWith since we removed type check in pr13573
while (i < cc.Count)
{
- if (cc[i] is Component comp && comp.GetType() == type)
+ if (cc[i] is Component comp)
{
string name = comp.Site.Name;
if (name.StartsWith(type.Name, StringComparison.Ordinal))
Screenshots
Before
After
Test methodology
- manual
Microsoft Reviewers: Open in CodeFlow
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 76.69700%. Comparing base (
285f3b4) to head (1a8e5c7). Report is 2 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #13650 +/- ##
====================================================
- Coverage 97.41503% 76.69700% -20.71803%
====================================================
Files 1192 3255 +2063
Lines 353853 641604 +287751
Branches 5435 47491 +42056
====================================================
+ Hits 344706 492091 +147385
- Misses 8378 145857 +137479
- Partials 769 3656 +2887
| Flag | Coverage Δ | |
|---|---|---|
| Debug | 76.69700% <ø> (-20.71803%) |
:arrow_down: |
| integration | 18.99943% <ø> (?) |
|
| production | 51.21963% <ø> (?) |
|
| test | 97.41503% <ø> (ø) |
|
| unit | 48.59479% <ø> (?) |
Flags with carried forward coverage won't be shown. Click here to find out more.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.