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

[Darkside] Implement className-switch #3586

Merged
merged 42 commits into from
Mar 3, 2025
Merged

Conversation

KenAJoh
Copy link
Collaborator

@KenAJoh KenAJoh commented Feb 13, 2025

Description

Component Checklist 📝

  • JSDoc
  • Examples
  • Documentation / Decision Records
  • Storybook
  • Style mappings (@navikt/core/css/config/_mappings.js)
  • Component tokens (@navikt/core/css/tokens.json)
  • CSS class deprecations (@navikt/aksel-stylelint/src/deprecations.ts)
  • Exports (@navikt/core/react/src/index.ts and @navikt/core/react/package.json)
  • New component? CSS import (@navikt/core/css/index.css)
  • Breaking change? Update migration guide. Consider codemod.
  • Changeset (Format: <Component>: <gitmoji?> <Text>. E.g. "Button: ✨ Add feature xyz.")

Copy link

changeset-bot bot commented Feb 13, 2025

🦋 Changeset detected

Latest commit: d30f8f9

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

Copy link
Contributor

github-actions bot commented Feb 13, 2025

Storybook demo / Chromatic

📝 Changes to review: 1

6af1b120e | 91 components | 135 stories

@KenAJoh KenAJoh added the ignore-chromatic Hindrer kode i å bli testet med chromatic label Feb 13, 2025
{icon}
<span className="sr-only">
<span className={cn("sr-only")}>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, maybe i was a bit overzealous in this semi-automatic process... though it shouldn't matter much (renaming does nothing if it doesn't match?... will waste a few more CPU cycles though... I think it might be easier to scan later for changes that hit classNames where there isn't any navds to rename and remove them then? 🤔 )

KenAJoh and others added 2 commits February 17, 2025 14:46
* 🚚 Rename all .navds prefixes to .aksel

* 📝 Changeset

* 📝 MD-docs
@KenAJoh KenAJoh marked this pull request as ready for review February 19, 2025 11:44
@KenAJoh KenAJoh merged commit 96b9b8e into main Mar 3, 2025
5 checks passed
@KenAJoh KenAJoh deleted the darkside-classname-update branch March 3, 2025 10:30
@github-actions github-actions bot mentioned this pull request Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-chromatic Hindrer kode i å bli testet med chromatic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants