-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feat: external link icon #1540
Feat: external link icon #1540
Conversation
uses CSS keyword 'currentColor' to achieve hover and visited styles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking great! Small comment below and some questions out to Design, but this seems like a clean way to do it.
benefits/static/css/styles.css
Outdated
@@ -142,6 +142,25 @@ a:visited:not(.btn) { | |||
color: var(--selected-color); | |||
} | |||
|
|||
[target="_blank"]::after { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think at a minimum we want to add the element here:
[target="_blank"]::after { | |
a[target="_blank"]::after { |
Is targeting all links with _blank
enough? Too much? Is it consistent practice to make external links open in new tabs?
Asking Sarah some related questions on the Figma board:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Closes #1471
Also closes #1478 , closes #1484
This PR assumes that all external links have an attribute of
target="_blank"
and adds an icon after the link.Footer links
Eligibility index
Help page