AttributedLabel Accessibility Links #537
Conversation
RoyalPineapple
changed the title
| @@ -389,7 +364,7 @@ extension AttributedLabel { | |||
| } | |||
|
|
|||
| assert( | |||
| alignments.count == 1, | |||
There was a problem hiding this comment.
This was causing a crash in the demo app.
Someone went through the trouble of writing the nil case below, it'd be a shame never to use it.
RoyalPineapple
changed the title
RoyalPineapple
force-pushed
the
alex/RotorAdditions
branch
2 times, most recently
from
e5ccc90 to
91e208b
Compare
RoyalPineapple
force-pushed
the
alex/RotorAdditions
branch
from
91e208b to
173630b
Compare
| private var accessibilityLinkIndex = -1 | ||
|
|
||
| /// These elements need to be retained by the view, and cannot be created inside the | ||
| /// `accessibilityCustomRotors` getter. |
There was a problem hiding this comment.
This concern about the elements being retained is still valid. We cannot pass back unretained elements created within the itemSearchBlock.
The trick here is the elements are now retained in an array which is itself captured by the itemSearchBlock.
Once VoiceOver releases the rotor these elements all get deallocated and then will be recreated the next time VoiceOver requests the rotor.
RoyalPineapple
force-pushed
the
alex/RotorAdditions
branch
from
ece74e1 to
8ff1979
Compare
RoyalPineapple
force-pushed
the
alex/RotorAdditions
branch
from
8ff1979 to
881411f
Compare
| extension BidirectionalCollection where Element: Equatable, Element: NSObjectProtocol { | ||
| private func itemSearch(_ predicate: UIAccessibilityCustomRotorSearchPredicate) -> UIAccessibilityCustomRotorItemResult? { | ||
| guard let first else { return nil } | ||
| guard let currentItem = predicate.currentItem.targetElement as? Element, |
There was a problem hiding this comment.
Thinking outloud, im surprised to see both the generic constraint and this dynamic check; are both required?
This PR migrates
AttributedLabel's link rotor from a stateful to functional pattern, eliminating the need for two variables and preventing voiceover from getting out of sync.I also fixed a bug in
links(at location: CGPoint)that could cause a crash if theAttributedStringlacked a specifiedNSTextAlignment.