OER-WEB-TEAM / design-system--drupal-theme

1 stars 0 forks source link

External Link - Module vs. Manual Discussion #62

Closed emsaiz closed 5 days ago

emsaiz commented 1 month ago

Just wanted to keep you posed about some of the discussions that have design system implications!

https://styleguide.od.nih.gov/components/link#external-link

Current guidance about how to create/add external link icon: "All you need to do is make sure to include the target="_blank" attribute to your links."

Update:

Let us know how you feel about this! I can make changes to the external link section depending on what we decide.

emsaiz commented 1 month ago

Should "image as link" get it's own H2? It's overkill for the code but I feel like it follows the pattern better image

AlexVanK commented 5 days ago

For the first point, this does not affect the Design System. It's a logistic problem on grants, that needs to be handled there. I would open a related ticked on the grants repo.

For the second point, I can't split the link page into an extra section as each component has only 3 variations (default + 2). Technically image is not a variation as an image could be all 3 variations too (normal, external, anchor).