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

🐛 Verkeerde design tokens voor alert-icon in React- en webcomponenten #360

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

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Jan 17, 2025

Omschrijving

Momenteel wordt in de React-alert (Utrecht) de token lux.alert.icon.size gebruikt, terwijl dit utrecht.alert.icon.size zou moeten zijn.
Ook in de webcomponent-alert wordt de verkeerde token gebruikt: lux.alert.icon.inset-block-start in plaats van utrecht.alert.icon.inset-block-start.

Dit zorgt voor inconsistentie in de styling en moet worden aangepast.

Verwacht gedrag

  • De Utrecht-componenten gebruiken de correcte Utrecht-tokens.
  • De verkeerde/onnodige lux-tokens worden verwijderd uit de JSON.

Acceptatiecriteria

  • In de React-alert wordt utrecht.alert.icon.size gebruikt.
  • In de webcomponent-alert wordt utrecht.alert.icon.inset-block-start gebruikt.
  • Lux-tokens voor deze componenten worden verwijderd uit de JSON.
  • Er zijn geen visuele regressies na de wijziging.
@AlineNap AlineNap moved this to Backlog in LUX componenten Jan 17, 2025
@AlineNap AlineNap changed the title Onnodig(?) lux component tokens toegepast op community component 🐛 Bug Report: Verkeerde design tokens voor alert-icon in React- en webcomponenten Feb 4, 2025
@AlineNap AlineNap changed the title 🐛 Bug Report: Verkeerde design tokens voor alert-icon in React- en webcomponenten 🐛 Verkeerde design tokens voor alert-icon in React- en webcomponenten Feb 4, 2025
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