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

fix: diasbled button #435

Merged
merged 1 commit into from
Nov 22, 2023
Merged

fix: diasbled button #435

merged 1 commit into from
Nov 22, 2023

Conversation

oganessone718
Copy link
Contributor

요약 *

fixes #241
너무 많은 게 바뀐 것 같아서... 새로 올립니당...

스크린샷

image

이후 Task *

  • 없음

Copy link

netlify bot commented Nov 2, 2023

Deploy Preview for biseo-preview ready!

Name Link
🔨 Latest commit d22f94e
🔍 Latest deploy log https://app.netlify.com/sites/biseo-preview/deploys/6543769a0d95fa00086f1579
😎 Deploy Preview https://deploy-preview-435--biseo-preview.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.

@oganessone718 oganessone718 self-assigned this Nov 7, 2023
Copy link
Member

@SnowSuno SnowSuno left a comment

Choose a reason for hiding this comment

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

👍 깔끔하게 잘 작동하네요
코멘트만 참고해서 나중에 다른 비슷한 부분 작업하실때 참고하시면 좋을 것 같아요

<Text variant="boldtitle3" color="blue600">
<Text
variant="boldtitle3"
color={validated ? "blue600" : "gray300"}
Copy link
Member

Choose a reason for hiding this comment

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

color css 속성은 inherit이 되니까 Button 컴포넌트에서 조정하는 것이 더 좋을 것 같아요!

그리고 atomic css 토큰을 적용하면 사실 아예 <Text /> 컴포넌트 자체가 필요 없어지긴 할것 같아요

Copy link
Member

@SnowSuno SnowSuno left a comment

Choose a reason for hiding this comment

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

👍 깔끔하게 잘 작동하네요
코멘트만 참고해서 나중에 다른 비슷한 부분 작업하실때 참고하시면 좋을 것 같아요

@SnowSuno SnowSuno merged commit f81a811 into main Nov 22, 2023
4 checks passed
@SnowSuno SnowSuno deleted the fix-394-new branch November 22, 2023 11:24
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.

[UI] disabled button ui
2 participants