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

chore: token migration guide #614

Draft
wants to merge 19 commits into
base: naporitan/replace-token1
Choose a base branch
from

Conversation

naporin0624
Copy link
Member

@naporin0624 naporin0624 commented Jul 31, 2024

やったこと

  • 新トークン移行のためのマイグレーションガイドを作成
  • link2 surface5 surface10 が figma 上に存在しないため移行を作成できていない。

動作確認環境

  • storybook

チェックリスト

不要なチェック項目は消して構いません

  • 破壊的変更がある場合には、対象のパッケージのメジャーバージョンが上がっていることを確認した
  • 追加したコンポーネントが index.ts から再 export されている
  • README やドキュメントに影響があることを確認した

@naporin0624 naporin0624 changed the base branch from main to v4.0.0 July 31, 2024 06:48
@naporin0624 naporin0624 self-assigned this Jul 31, 2024
@naporin0624 naporin0624 changed the base branch from v4.0.0 to naporitan/replace-token1 July 31, 2024 06:48
Comment on lines +126 to +148
### カラーの上書き

ライトモードの場合は `:root`、ダークモードの場合は `:root[data-theme='dark']` セレクターに対応しています。`@charcoal-react/dist/index.css` の後に記述することで、トークンを上書きできます。

#### ライトモード

**例**: ブランドカラーを変更する場合

```css
:root {
--charcoal-color-container-primary-default: #0096fa;
}
```

#### ダークモード

**例**: ブランドカラーを変更する場合

```css
:root[data-theme='dark'] {
--charcoal-color-container-primary-default: #0096fa;
}
```
Copy link
Member Author

Choose a reason for hiding this comment

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

新トークン上書きはここから

@naporin0624 naporin0624 marked this pull request as ready for review August 15, 2024 08:02
@naporin0624 naporin0624 marked this pull request as draft August 28, 2024 07:23
@naporin0624
Copy link
Member Author

多分いらないきがする

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants