Skip to content

Conversation

@M-Lipin
Copy link
Contributor

@M-Lipin M-Lipin commented May 4, 2020

Adding unit tests.

Fixes #3206

Proposed changes

  • Moving LabelAccessibleObject to a separate file.
  • Ensured UIA accessibility support.
  • Adding unit tests.

Customer Impact

  • Improving development experience for Label control accessibility.

Regression?

  • No

Risk

  • Minimal

Screenshots

Before

image

After

image

Test methodology

  • Manual testing;
  • Unit testing;
  • UIA automation.

Accessibility testing

Test environment(s)

.NET Core 5.0

Version: 5.0.100-alpha.1.20073.10
Commit: 29f4d693a9

Runtime Environment:
OS Name: Windows
OS Version: 10.0.18363
OS Platform: Windows
RID: win10-x64
Base Path: C:\Program Files\dotnet\sdk\5.0.100-alpha1-05536\

Host (useful for support):
Version: 5.0.0-alpha.1.20072.3
Commit: c3dc1fdfdc

Microsoft Reviewers: Open in CodeFlow

@M-Lipin M-Lipin requested a review from a team as a code owner May 4, 2020 19:40
@ghost ghost assigned M-Lipin May 4, 2020
@M-Lipin
Copy link
Contributor Author

M-Lipin commented May 4, 2020

This PR is related to #2164 and #3037

@ghost ghost added the waiting-author-feedback The team requires more information from the author label May 6, 2020
@RussKie RussKie marked this pull request as draft May 6, 2020 06:04
@codecov
Copy link

codecov bot commented May 12, 2020

Codecov Report

Merging #3208 into master will decrease coverage by 29.09856%.
The diff coverage is 51.02041%.

@@                 Coverage Diff                  @@
##              master       #3208          +/-   ##
====================================================
- Coverage   62.14766%   33.04910%   -29.09856%     
====================================================
  Files           1257         868         -389     
  Lines         449428      254013      -195415     
  Branches       39227       36752        -2475     
====================================================
- Hits          279309       83949      -195360     
- Misses        164638      165250         +612     
+ Partials        5481        4814         -667     
Flag Coverage Δ
#Debug 33.04910% <51.02041%> (-29.09856%) ⬇️
#production 33.04910% <51.02041%> (-0.29283%) ⬇️
#test ?

@Ryuugamine Ryuugamine force-pushed the Issue_3206_Label_control_UIA_accessibility branch from c043254 to 86274a4 Compare June 18, 2020 15:32
@RussKie RussKie marked this pull request as ready for review June 22, 2020 12:42
@RussKie
Copy link
Contributor

RussKie commented Jun 22, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@vladimir-krestov vladimir-krestov left a comment

Choose a reason for hiding this comment

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

LGTM

@Ryuugamine Ryuugamine force-pushed the Issue_3206_Label_control_UIA_accessibility branch from 1d4c811 to aad4969 Compare July 2, 2020 12:02
@dotnet dotnet deleted a comment from codecov bot Jul 6, 2020
@Ryuugamine Ryuugamine force-pushed the Issue_3206_Label_control_UIA_accessibility branch from a406564 to 733648a Compare July 6, 2020 08:36
@RussKie RussKie merged commit 6f85a5d into dotnet:master Jul 6, 2020
@ghost ghost added this to the 5.0 Preview8 milestone Jul 6, 2020
@RussKie RussKie deleted the Issue_3206_Label_control_UIA_accessibility branch July 6, 2020 09:28
@RussKie RussKie removed the waiting-author-feedback The team requires more information from the author label Jul 6, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Feb 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Label control does not support UIA accessiblility

5 participants