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

update checkbox style #1706

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

update checkbox style #1706

wants to merge 12 commits into from

Conversation

geckotang
Copy link
Collaborator

@geckotang geckotang commented Feb 3, 2025

やったこと

  • Checkboxのサイズにsmallとmediumを追加。(mediumがデフォルト)
  • disabledのスタイルが適用されていなかったため、disabledを渡すように。
  • SVGアイコンの差し替え
  • ドロップシャドウやインナーシャドウの変更

変更したコンポーネント

Copy link

changeset-bot bot commented Feb 3, 2025

🦋 Changeset detected

Latest commit: 4b2c024

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

This PR includes changesets to release 1 package
Name Type
ingred-ui Minor

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

@geckotang geckotang self-assigned this Feb 3, 2025
Copy link

netlify bot commented Feb 3, 2025

Deploy Preview for ingred-ui ready!

Name Link
🔨 Latest commit 190ad79
🔍 Latest deploy log https://app.netlify.com/sites/ingred-ui/deploys/67a19c44c7b21b0008d5a142
😎 Deploy Preview https://deploy-preview-1706--ingred-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@geckotang geckotang changed the title update checkbox style [WIP] update checkbox style Feb 3, 2025
@noronaoki
Copy link
Contributor

@geckotang
対応ありがとうございます!

今回のデザイン修正では主に下記を変更しております。(事前に共有できておらずすみません)

  • サイズの追加
  • チェックマークなどの中身のイラスト(主に太さなど)
  • ドロップシャドウやインナーシャドウ
    • 現状よりも薄くしています

拝見したところシャドウ系が未修正でしたのでこちらも対応お願いいたします。
また、下記のパターンにおいてチェックマークイラスト差し変わっていないように見えたのでご確認お願いします。

image

ラジオボタンも同様の修正を加えております。
それと今回の修正でシャドウ系は全般的に変更を加えておりますので、もしコード内で共通化されている場合は一括での変更をお願いしたいです。
例えばチェックボックスやラジオボタン、ボタンなどの立体的に見せているコンポーネントにはFigmaで共通のドロップシャドウ・インナーシャドウを用いています。

@geckotang geckotang changed the title [WIP] update checkbox style update checkbox style Feb 4, 2025
@geckotang
Copy link
Collaborator Author

@noronaoki 修正しましたので確認お願いします。

拝見したところシャドウ系が未修正でしたのでこちらも対応お願いいたします。
また、下記のパターンにおいてチェックマークイラスト差し変わっていないように見えたのでご確認お願いします。

修正しました。

それと今回の修正でシャドウ系は全般的に変更を加えておりますので、もしコード内で共通化されている場合は一括での変更をお願いしたいです。

他の部分は、後追いのPRでやりたいと思います。

@noronaoki
Copy link
Contributor

@geckotang
拝見しましたが、やはり横棒の太さが太いように見受けられます。
今一度ご確認ください。
image
image

それとチェックのDisabledパターンが載っていなかったのでこちらも追加でお願いいたします。
image

@geckotang
Copy link
Collaborator Author

@noronaoki 修正しました。

https://ingred-ui.netlify.app/?path=/story/components-inputs-checkbox--design-samples
元々のものを見ていて気づいたのですが、 上記のDesign Sampleの「Checked(disabled)」は、indeterminateのアイコンがでていたようで、その修正も含めています。

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.

2 participants