-
Notifications
You must be signed in to change notification settings - Fork 675
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
[ProgressRing] Remove ProgressRing from UIA tree when it is not active #5930
[ProgressRing] Remove ProgressRing from UIA tree when it is not active #5930
Conversation
@@ -341,7 +341,8 @@ void ProgressRing::UpdateStates() | |||
// Swap player source to determinate. | |||
SetAnimatedVisualPlayerSource(); | |||
UpdateLottieProgress(); | |||
} | |||
} | |||
winrt::AutomationProperties::SetAccessibilityView(*this, winrt::AccessibilityView::Content); |
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.
We have to do this here since VSS setters can't set this on the "root" control
@@ -25,6 +25,7 @@ | |||
<VisualState x:Name="Inactive"> | |||
<VisualState.Setters> | |||
<Setter Target="LayoutRoot.Opacity" Value="0" /> | |||
<Setter Target="LottiePlayer.(AutomationProperties.AccessibilityView)" Value="Raw" /> |
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.
Setting this since we don't want the animation to still exist in the UIA tree.
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.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
🎉 Handy links: |
🎉 Handy links: |
🎉 Handy links: |
🎉 Handy links: |
Description
Removing ProgressRing from UIA tree when it is not active.
Motivation and Context
Closes #2741
How Has This Been Tested?
Tested with narrator, control does not get announced anymore.
Screenshots (if appropriate):