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

Fixes #3663 #3679

Merged
merged 1 commit into from Jul 31, 2020
Merged

Fixes #3663 #3679

merged 1 commit into from Jul 31, 2020

Conversation

Tanya-Solyanik
Copy link
Member

@Tanya-Solyanik Tanya-Solyanik commented Jul 31, 2020

Added accessible name property to ToolStrip combo box in the test app

Fixes #3663

Microsoft Reviewers: Open in CodeFlow

@Tanya-Solyanik Tanya-Solyanik requested a review from a team as a code owner July 31, 2020 01:47
@ghost ghost assigned Tanya-Solyanik Jul 31, 2020
@RussKie RussKie added the test-enhancement Improvements of test source code label Jul 31, 2020
@codecov
Copy link

codecov bot commented Jul 31, 2020

Codecov Report

Merging #3679 into master will decrease coverage by 31.43968%.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##              master       #3679          +/-   ##
====================================================
- Coverage   67.14224%   35.70256%   -31.43969%     
====================================================
  Files           1380         909         -471     
  Lines         503181      249758      -253423     
  Branches       40771       36559        -4212     
====================================================
- Hits          337847       89170      -248677     
+ Misses        159609      155586        -4023     
+ Partials        5725        5002         -723     
Flag Coverage Δ
#Debug 35.70256% <ø> (-31.43969%) ⬇️
#production 35.70256% <ø> (+0.00920%) ⬆️
#test ?

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

@Tanya-Solyanik Tanya-Solyanik merged commit 5ef592f into dotnet:master Jul 31, 2020
@ghost ghost added this to the 5.0 RC1 milestone Jul 31, 2020
@@ -388,12 +388,13 @@ private void InitializeComponent()
//
// toolStripComboBox1
//
this.toolStripComboBox1.AccessibleName = "Custom accessible name for combo box";
Copy link
Member

Choose a reason for hiding this comment

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

This custom accessible name is not proper, it is also failed.
image

@@ -583,7 +584,7 @@ private void InitializeComponent()
//
// toolStripProgressBar1
//
this.toolStripProgressBar1.AccessibleName = "Proressbar";
this.toolStripProgressBar1.AccessibleName = "Custom accessible name for progress bar";
Copy link
Member

Choose a reason for hiding this comment

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

The result of progressBar is same as comboBox:
image

@Tanya-Solyanik Tanya-Solyanik deleted the bug-3663 branch September 2, 2020 04:09
@dotnet dotnet locked as resolved and limited conversation to collaborators Jan 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Accessibility] ToolStripComboBox/ToolStripTextBox items' name property cannot be null
3 participants