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

AttributedLabel Link Accessibility #459

Merged
merged 2 commits into from
Jun 11, 2024
Merged

Conversation

RoyalPineapple
Copy link
Collaborator

@RoyalPineapple RoyalPineapple commented May 30, 2023

Updating attributed label's links to more closely mirror the SwiftUI accessibility experience

SwiftUI.Link.mp4

This is our current implementation, which doesn't announce links properly.

WithoutLinks.mov

Here is the new version, announcing links as they are read.

WithLinks.mp4

@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch from 8a97a0e to 0344a7f Compare May 30, 2023 21:02
@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch 2 times, most recently from bc23c22 to e6457d0 Compare June 8, 2023 17:00
@RoyalPineapple RoyalPineapple marked this pull request as ready for review June 29, 2023 17:34
@RoyalPineapple RoyalPineapple requested a review from a team as a code owner June 29, 2023 17:34
@RoyalPineapple RoyalPineapple marked this pull request as draft November 28, 2023 16:54
@kyleve
Copy link
Collaborator

kyleve commented Nov 29, 2023

It looks like there's some failing tests:

Failing tests:

	AttributedLabelTests.test_linkAttribute()
	AttributedLabelTests.test_textContainerRects()

@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch 2 times, most recently from ec73157 to 736cd9e Compare November 30, 2023 17:28
@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch 4 times, most recently from 7bfd79a to 88a7dca Compare November 30, 2023 22:17
@RoyalPineapple RoyalPineapple marked this pull request as ready for review November 30, 2023 22:47
@RoyalPineapple RoyalPineapple requested a review from n8chur December 5, 2023 16:50
@RoyalPineapple
Copy link
Collaborator Author

RoyalPineapple commented Dec 5, 2023

Replaced with #473

@robmaceachern
Copy link
Member

@RoyalPineapple Moving to draft because I think this was addressed elsewhere but feel free to reopen if needed!

@RoyalPineapple RoyalPineapple marked this pull request as ready for review May 30, 2024 18:19
…f the .link string attribute which we strip to allow our own styling
@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch from d12564c to 599f360 Compare May 30, 2024 18:31
@RoyalPineapple RoyalPineapple force-pushed the alex/LinkAccessibility branch from e20a350 to d62afc3 Compare May 30, 2024 19:57
@RoyalPineapple RoyalPineapple merged commit 6e97efa into main Jun 11, 2024
10 checks passed
@RoyalPineapple RoyalPineapple deleted the alex/LinkAccessibility branch June 11, 2024 16:38
RoyalPineapple added a commit that referenced this pull request Jun 13, 2024
Bump to 4.1.0 to include new link accessibility fixes
#459
maxg-square added a commit that referenced this pull request Nov 4, 2024
* tag '4.3.0':
  Prepare version 4.3.0 (#513)
  Add passThroughTouches to BlueprintView, PassthroughView (#511)
  Bump checkout and upload-artifacts (#512)
  Bump rexml from 3.2.8 to 3.3.6
  Unnest Box.CornerStyle (#508)
  Prepare 4.2.1 (#507)
  Make public the UIBezierPath helper using Box.CornerStyle
  Prepare 4.2.0 (#505)
  Label and AttributedLabel now support accessibilityValue (#504)
  Prepare 4.1.2 Release (#503)
  Replace newlines in a11y labels with spaces (#502)
  Fix string index out of bounds crash. (#501)
  bumping version to 4.1.0 (#499)
  AttributedLabel Link Accessibility (#459)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants