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

refactor: rename data-ds-color-mode to data-color-scheme #2798

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

unekinn
Copy link
Contributor

@unekinn unekinn commented Nov 20, 2024

This does not currently affect the design tokens, which still contain the color scheme modes in the group "Mode". This will be changed later.

Closes #2806

Copy link

changeset-bot bot commented Nov 20, 2024

🦋 Changeset detected

Latest commit: a5351f8

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

This PR includes changesets to release 4 packages
Name Type
@digdir/designsystemet-react Major
@digdir/designsystemet-theme Major
@digdir/designsystemet-css Major
@digdir/designsystemet Major

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 Nov 20, 2024

Preview deployments for this pull request:

Storybook - 20. Nov 2024 - 15:27

Storefront - 20. Nov 2024 - 15:28

Theme - 20. Nov 2024 - 15:27

Copy link
Contributor

github-actions bot commented Nov 20, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 55.01% 3188 / 5795
🔵 Statements 55.01% 3188 / 5795
🔵 Functions 87.14% 183 / 210
🔵 Branches 76.84% 521 / 678
File CoverageNo changed files found.
Generated in workflow #1015 for commit a5351f8 by the Vitest Coverage Report Action

@unekinn unekinn force-pushed the refactor/data-ds-color-mode branch from 1fe9ea2 to 4ccbe68 Compare November 20, 2024 12:38
@unekinn unekinn force-pushed the refactor/data-ds-color-mode branch 2 times, most recently from dbe1701 to 9fc31da Compare November 20, 2024 14:21
This does not currently affect the design tokens, which still contain
the color scheme modes in the group "Mode". This will be changed later.

Closes #2806
@unekinn unekinn force-pushed the refactor/data-ds-color-mode branch from 9fc31da to a5351f8 Compare November 20, 2024 14:25
Copy link
Member

@Barsnes Barsnes left a comment

Choose a reason for hiding this comment

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

🔧

@unekinn unekinn merged commit 6998d4b into next Nov 21, 2024
13 checks passed
@unekinn unekinn deleted the refactor/data-ds-color-mode branch November 21, 2024 09:06
mimarz pushed a commit that referenced this pull request Nov 21, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to next, this PR will
be updated.

⚠️⚠️⚠️⚠️⚠️⚠️

`next` is currently in **pre mode** so this branch has prereleases
rather than normal releases. If you want to exit prereleases, run
`changeset pre exit` on `next`.

⚠️⚠️⚠️⚠️⚠️⚠️

# Releases
## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

### Minor Changes

- `data-color` prop now supports type safety based on the token
structure which was used to generate the theme CSS. Instructions for
enabling this has been added to the README. An optional
`react-types.d.ts` has also been added, which adds type hints for
`data-color` and `data-size` to all HTML elements.
([#2795](#2795))

## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

### Minor Changes

- CLI: `tokens build` command now generates a `colors.d.ts` file which
enables type safety for the `data-color` attribute when included in your
`tsconfig.json`. The `@digdir/designsystemet-theme` package has been
updated to include types for those themes.
([#2795](#2795))

## @digdir/[email protected]

### Minor Changes

- Changed name of Tokens Studio theme "Mode" to "Color scheme", and
updated the path in the token structure from `primitives/modes/colors`
to `primitives/modes/color-scheme`
([#2808](#2808))

- CLI: `tokens build` command now generates a `colors.d.ts` file which
enables type safety for the `data-color` attribute when included in your
`tsconfig.json`. The `@digdir/designsystemet-theme` package has been
updated to include types for those themes.
([#2795](#2795))

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Barsnes pushed a commit that referenced this pull request Nov 22, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to next, this PR will
be updated.

⚠️⚠️⚠️⚠️⚠️⚠️

`next` is currently in **pre mode** so this branch has prereleases
rather than normal releases. If you want to exit prereleases, run
`changeset pre exit` on `next`.

⚠️⚠️⚠️⚠️⚠️⚠️

# Releases
## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

### Minor Changes

- `data-color` prop now supports type safety based on the token
structure which was used to generate the theme CSS. Instructions for
enabling this has been added to the README. An optional
`react-types.d.ts` has also been added, which adds type hints for
`data-color` and `data-size` to all HTML elements.
([#2795](#2795))

## @digdir/[email protected]

### Major Changes

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

### Minor Changes

- CLI: `tokens build` command now generates a `colors.d.ts` file which
enables type safety for the `data-color` attribute when included in your
`tsconfig.json`. The `@digdir/designsystemet-theme` package has been
updated to include types for those themes.
([#2795](#2795))

## @digdir/designsystemet@1.0.0-next.37

### Minor Changes

- Changed name of Tokens Studio theme "Mode" to "Color scheme", and
updated the path in the token structure from `primitives/modes/colors`
to `primitives/modes/color-scheme`
([#2808](#2808))

- CLI: `tokens build` command now generates a `colors.d.ts` file which
enables type safety for the `data-color` attribute when included in your
`tsconfig.json`. The `@digdir/designsystemet-theme` package has been
updated to include types for those themes.
([#2795](#2795))

- **BREAKING CHANGE**: The attribute / prop `data-ds-color-mode` has
been renamed to `data-color-scheme`
([#2798](#2798))

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename data-ds-color-mode to data-color-scheme
2 participants