Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

[terra-hyperlink] Implement Accessibility changes (Part-1) #3605

Merged
merged 9 commits into from
Apr 11, 2022

Conversation

saket2403
Copy link
Contributor

Summary

Marked isUnderlineHidden and isDisabled props as private. Markup changes for Disabled Hyperlink for better accessibility. Replaced children with text prop.

Closes #

Deployment Link

https://terra-core-deployed-pr-#.herokuapp.com/

Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

@saket2403 saket2403 self-assigned this Apr 5, 2022
@saket2403 saket2403 requested a review from a team as a code owner April 5, 2022 09:55
@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 5, 2022 09:55 Inactive
@saket2403 saket2403 changed the base branch from main to A11y-MVB April 5, 2022 11:10
@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 5, 2022 11:19 Inactive
@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 5, 2022 11:39 Inactive
@cerner cerner deleted a comment from terra-bot Apr 5, 2022
@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 8, 2022 06:35 Inactive
@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 8, 2022 11:10 Inactive
@sdadn
Copy link
Contributor

sdadn commented Apr 8, 2022

Is there a tech design for this story?

@benbcai benbcai temporarily deployed to terra-core-hyperlink-a1-vdnsyt April 11, 2022 04:52 Inactive
@saket2403 saket2403 merged commit 62dedec into A11y-MVB Apr 11, 2022
@saket2403 saket2403 deleted the hyperlink-a11y branch April 11, 2022 09:28
sdadn pushed a commit that referenced this pull request May 11, 2022
* Pilot

* Docs

* Comments

* Remove Badge

* Minor Changes
sdadn pushed a commit that referenced this pull request May 11, 2022
* Pilot

* Docs

* Comments

* Remove Badge

* Minor Changes
sdadn pushed a commit that referenced this pull request Sep 30, 2022
* Pilot

* Docs

* Comments

* Remove Badge

* Minor Changes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants