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

USWDS-Site: Replaced deprecated xlink:href code with recommended use of href #2926

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RachelCorsino
Copy link
Contributor

@RachelCorsino RachelCorsino commented Oct 30, 2024

Summary

All xlink:href instances have be replaced with href within the repo in order to account for the deprecation of xlink:href.

Related issue

Closes USWDS - Bug: xlink:href deprecated #6088 from USWDS repo. This issue extends to fixes needed in USWDS-Site.

Preview link

Preview link:
USWDS Icon Component Page

Problem statement

The USWDS-Site repository currently contains numerous instances of xlink:href, which is a deprecated attribute that may cause compatibility issues/errors.

Solution

The USWDS-Site repository has been updated to use the href attribute instead of the deprecated xlink:href attribute.

Testing and review

  1. Ensure local version of site is not using xlink:href.

cathybaptista

This comment was marked as resolved.

Copy link
Contributor

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! One potential image regression but this should be good to go.

Also noting there are references to xmlns:xlink in case we are able to remove these as well!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue: This image is failing to find and saying it's an invalid source. Can you look into this? It's appearing on develop

Copy link
Contributor

@mejiaj mejiaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks @RachelCorsino

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.

USWDS - Bug: xlink:href deprecated
4 participants