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

Add visually hidden text to summary list examples #233

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

edwardhorsford
Copy link
Contributor

Change links in summary lists need visually hidden text so that the links make sense when read in isolation or as a list of links.

In the native design system component you pass arguments for this. Until this is supported, this shows how to manually add the accessible text using the component and updates the docs to mention it needs to be done.

Screenshot 2024-06-11 at 10 39 56

Partially fixes #217

Copy link

sonarcloud bot commented Jun 11, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@Tomdango Tomdango merged commit 4cdd2f5 into main Jun 11, 2024
2 checks passed
@Tomdango Tomdango deleted the add-visually-hidden-text-summary-list branch June 11, 2024 12:37
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.

Summary lists are missing accessible change link text
2 participants