-
Notifications
You must be signed in to change notification settings - Fork 45
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
AccessibilityBlocker aggressively blocking. #483
Conversation
… isBlocking is false
@@ -33,7 +33,7 @@ public struct AccessibilityBlocker: Element { | |||
|
|||
public func backingViewDescription(with context: ViewDescriptionContext) -> ViewDescription? { | |||
UIView.describe { config in | |||
config[\.isAccessibilityElement] = !isBlocking | |||
config[\.isAccessibilityElement] = false |
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.
Ahh, since this is just a container, right?
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.
exactly
@@ -33,7 +33,7 @@ public struct AccessibilityBlocker: Element { | |||
|
|||
public func backingViewDescription(with context: ViewDescriptionContext) -> ViewDescription? { | |||
UIView.describe { config in | |||
config[\.isAccessibilityElement] = !isBlocking |
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.
This UIView should never be an accessibilityElement because it's always wrapping another element.
if this gets set to true (when isBlocking is false) it also blocks the accessibility because voiceover never asks for its children.
…ller * origin/main: (62 commits) AccessibilityBlocker aggressively blocking. (#483) Bumping version to 3.0.0 (#482) Allow for customization of the preview name (#478) Update CHANGELOG for AttributedLabel fixes (#480) Fix link detection for stretched labels (#476) chore: Updated minimum deployment target from iOS 14 to iOS 15 [UI-5185] (#479) chore(ios): Bump to Xcode 15.1 and Ruby 3.2.2 [UI-5184] (#477) AXCustomContent Support (#471) Bumping versions to 2.2.0 (#470) Update concatenation logic and unit tests update changelog never cache subelements optionally do not cache subelements Feature: add TintAdjustmentMode, modifiers, and tests Add to CHANGELOG Add tintAdjustmentMode to Image Bumping versions to 2.1.0 (#466) Resolved a Swift 5.9 compilation warning (#465) Update KeyboardObserver (#463) Bump activesupport from 7.0.4.3 to 7.0.7.2 ...
Fixed a bug where
AccessibilityBlocker
would block accessibility whenisBlocking
isfalse