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

デフォルトのカラーを白にする #346

Merged
merged 7 commits into from
Jan 1, 2024
Merged

Conversation

takashi0602
Copy link
Member

@takashi0602 takashi0602 commented Jan 1, 2024

ref #305

概要

Figmaのデザイン変更に伴って文字色のカラーを白ベースにしました。また、使用しなくなったカラーを削除しました。

※ 別PRでカラーのkey名は修正します🙏

スナップショット

主要な変更以外は割愛します🙏

Badge

背景色をなくしました(背景色は親側からclassNameを渡すことで付与する。TechnologyBadgeを参照)

Storybook上に表示されたBadgeコンポーネント

Text

ベースを白に変更。かつ、使用するカラーを追加・修正しました。

Storybook上に表示されたTextコンポーネント

Accordion

ベースを白に変更。かつ、アイコンのカラーも修正しました。

Storybook上に表示されたAccordionコンポーネント

@takashi0602 takashi0602 self-assigned this Jan 1, 2024
@uyupunpopunpo
Copy link
Contributor

@tyokinuhata
Copy link
Member

また、使用しなくなったカラーを削除しました。

purple消えてないけどこれは使われてるって認識で合ってる?

@tyokinuhata
Copy link
Member

Badge
背景色をなくしました(背景色は親側からclassNameを渡すことで付与する。TechnologyBadgeを参照)

これってデフォルトではバッジは透過されてるっていう認識で合ってる?

@tyokinuhata
Copy link
Member

Text
ベースを白に変更。

「ベース」ってデフォルト的な意味で解釈してるけど合ってる?

@takashi0602
Copy link
Member Author

purple消えてないけどこれは使われてるって認識で合ってる?
使われてなかったので(パープル以外も)削除しました🙇‍♂️

@takashi0602
Copy link
Member Author

これってデフォルトではバッジは透過されてるっていう認識で合ってる?
認識の通りで、透過されてます🫡

@takashi0602
Copy link
Member Author

「ベース」ってデフォルト的な意味で解釈してるけど合ってる?
デフォルト的な意味であってます🙆‍♂️

Copy link
Member

@tyokinuhata tyokinuhata left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@takashi0602 takashi0602 merged commit 9b662ec into main Jan 1, 2024
6 checks passed
@takashi0602 takashi0602 deleted the fix/305_font-color branch January 1, 2024 17:31
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.

3 participants