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: Alert component (2) #353

Open
AlineNap opened this issue Jan 7, 2025 · 0 comments
Open

🎨👀 Design review: Alert component (2) #353

AlineNap opened this issue Jan 7, 2025 · 0 comments

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Jan 7, 2025

Bevindingen

1. Icoon uitlijning

  • Het icoon graag centreren met de heading.
  • Als er geen heading is, dan het icoon centreren met de eerste regel van de paragraaf.
  • De token utrecht.alert.icon.inset-block-start lijkt te ontbreken voor het correct positioneren van het icoon.

2. Typography tokens voor heading toepassen

De titel moet de LUX-component tokens krijgen zoals besproken in Alert #237 (comment).

Toe te passen tokens:

  • lux.alert.heading.font-family
  • lux.alert.heading.font-size
  • lux.alert.heading.font-weight
  • lux.alert.heading.line-height

3. Thema NLdoc controle

  • Bij het inschakelen van het NLdoc-thema wordt de font-size van de titel groter.
  • Dit wordt waarschijnlijk opgelost door het correct toepassen van de Typography tokens voor de heading (zie punt 3).

4. Visual regressiontest uitbreiden

In de huidige visual regressiontest ontbreekt de heading. Graag een variant toevoegen met heading. Hiermee kunnen we de positionering van het icoon en de styling van de titel goed testen.

@AlineNap AlineNap converted this from a draft issue Jan 7, 2025
@AlineNap AlineNap changed the title Design review alert (2) 🎨👀 Design review alert (2) Feb 4, 2025
@AlineNap AlineNap changed the title 🎨👀 Design review alert (2) 🎨👀 Design review: Alert component (2) 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