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

[Design System] Button #97

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

[Design System] Button #97

wants to merge 16 commits into from

Conversation

ckdwns9121
Copy link
Member

개요 🔍

작업 내용 📝

  • outlineButton, textButton, ContainButton,IconButton 작업을 진행했습니다.
  • 기존에 합쳤던 컴포넌트를 개별로 분리해서 작업하자는 의견이 나와 모두 쪼개었습니다.

기타 사항 🙋‍♂️

  • 아직 Icon + Button에 대한 작업을 어떻게 할 지 잘 모르겠습니다. 이는 리뷰를 통해 의견을 전달해주시면 감사합니다.
  • 단일 Button일 때는 정확한 Size가 명시되었지만 Icon이 추가되었을 시는 버튼의 전체 사이즈가 명시되어있지 않습니다..
  • selected , disabled와 같은 옵션이 추가되었을 시 텍스트 컬러도 변경되는데 SVG color도 바뀌어야 하는지 알 수 없습니다.
  • loading 상태일 때는 SVG Icon을 렌더링 해야하는지 알 수 없습니다.
  • 버튼 크기가 변경된다면 Icon 크기는 어떻게 변경되는지 알 수 없습니다.
  • 버튼의 크기는 default, small, large, x-large 총 4가지인데 Icon의 크기는 16 | 20 | 24 3가지만 주어집니다.

@netlify
Copy link

netlify bot commented Oct 30, 2022

Deploy Preview for ids-storybook ready!

Name Link
🔨 Latest commit bba70b6
🔍 Latest deploy log https://app.netlify.com/sites/ids-storybook/deploys/6370d129fe87ab00098bc29a
😎 Deploy Preview https://deploy-preview-97--ids-storybook.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 settings.

@ckdwns9121 ckdwns9121 added Developer 개발자 이슈 Process 작업 단위(디자인 + 개발) labels Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Developer 개발자 이슈 Process 작업 단위(디자인 + 개발)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Process] Component - 버튼 작업 진행
1 participant