navikt / aksel

NAVs designsystem og Aksel-portalen
https://aksel.nav.no
MIT License
158 stars 42 forks source link

[Darkside] Tag CSS update #3402

Open larseirikhansen opened 5 days ago

changeset-bot[bot] commented 5 days ago

⚠️ No Changeset found

Latest commit: 5f98494cf509796635f14bc36ca81f167b3c2c15

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

github-actions[bot] commented 5 days ago

Storybook demo / Chromatic

📝 Endringer til review: 114

1acba80a0 | 91 komponenter | 135 stories

larseirikhansen commented 5 days ago

Kan jeg åpne en diskusjon om å ha kommentarer i CSSen vår som "overskrifter"? Ikke overalt kanskje, men jeg synes det gjør CSSen mer oversiktlig, og at det er lettere óg går raskere å finne frem når man ikke er lommekjent med en CSS-fil. Jeg har gjort det i denne pull requesten og i https://github.com/navikt/aksel/pull/3400.

For å unngå at det blir støyete kan man være streng med kun å bruke det som overskrifter, f.eks. som i en artikkel.

KenAJoh commented 2 days ago

Kan jeg åpne en diskusjon om å ha kommentarer i CSSen vår som "overskrifter"? Ikke overalt kanskje, men jeg synes det gjør CSSen mer oversiktlig, og at det er lettere óg går raskere å finne frem når man ikke er lommekjent med en CSS-fil. Jeg har gjort det i denne pull requesten og i #3400.

For å unngå at det blir støyete kan man være streng med kun å bruke det som overskrifter, f.eks. som i en artikkel.

Go for it! We decided to try this a few weeks ago, but only where it is necessary since section-comments might be redundant some places. Can see for example the new ActionMenu CSS for reference