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

Feat: add Toggle Button(set up tailwind in monorepo) #15

Merged
merged 7 commits into from
Jun 18, 2023

Conversation

poiu694
Copy link
Member

@poiu694 poiu694 commented Jun 17, 2023

Related Issue #14

  • Toggle Button을 생성했습니다.

  • workspace가 service-frontend, ui로 나누어져있어, tailwind.config.js를 두 개를 같이 수정해야 했어서 tailwind-config workspace를 통해 관리 포인트를 하나로 줄였습니다.

  • 기존 service-frontend에서 ui 내 컴포넌트를 가져왔을 경우 transpile이 되어있지 않아 문제가 있던 부분을 해결했습니다.

  • color token이 적용되지 않아서 임시로 헥사값을 그대로 넣었습니다!

  • 컬러 디자인 시스템 적용하고 올릴까 했는데, monorepo 설정으로 인한 오류를 겪게될 것 같아서 미리 올립니당

Preview

In service

image

Storybook

image

Animation

toggle-example

크롬에서는 나쁘지 않게 보이는 것 같은데..>! 캡쳐하니까 묘하게 끊겨보이네요

Copy link
Collaborator

@yungjurick yungjurick left a comment

Choose a reason for hiding this comment

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

머지하고 그 위에 tailwind 세팅하는게 좋을 것 같아 보이네! 수고했어!!

Copy link
Member

@swimjiy swimjiy left a comment

Choose a reason for hiding this comment

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

애니메이션 정말 귀엽다ㅋㅋㅋ고생 많았어!
eslint import order PR 조금 전에 머지해서, 요거 풀 받고 에러 없는지 확인만 해주면 될 듯!

@poiu694
Copy link
Member Author

poiu694 commented Jun 18, 2023

ffcc19a 린트 확인했고 머지하겠습니당~

@poiu694 poiu694 merged commit 419987d into master Jun 18, 2023
@poiu694 poiu694 deleted the feature/toggle-button branch June 18, 2023 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants