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

🎨👀 Design Review: Link Component #369

Open
4 tasks
AlineNap opened this issue Feb 4, 2025 · 0 comments
Open
4 tasks

🎨👀 Design Review: Link Component #369

AlineNap opened this issue Feb 4, 2025 · 0 comments

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Feb 4, 2025

Omschrijving

Tijdens de design review van 18 december zijn er een aantal bevindingen over de implementatie van de link-component.

Openstaande punten

  • In Storybook ontbreekt een voorbeeld van de visited state. Deze graag toevoegen.
  • De token 'utrecht.link.active.text-decoration' staat op none, maar er is toch een underline zichtbaar.
    • Bestaat deze token niet?
    • Waarom wordt deze instelling niet toegepast? Dit is geen dealbreaker, maar wel iets om te onderzoeken.
  • De positie van het icoon is nu niet goed te testen omdat het icoon-component niet wordt gebruikt. Nu we een icon-component hebben, kan dit worden opgepakt.
  • De token utrecht.link.column-gap wordt niet toegepast.

Acties

  • Visited state toevoegen aan Storybook.
  • Onderzoeken waarom utrecht.link.active.text-decoration niet wordt toegepast.
  • Controleren of de icoon-positie goed is met het nieuwe icon-component.
  • Nagaan waarom utrecht.link.column-gap niet wordt toegepast.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To do
Development

No branches or pull requests

1 participant