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: markup vote detail page and add common button style in chakra #31

Merged
merged 13 commits into from
Jan 7, 2024

Conversation

SKY-PEY
Copy link
Contributor

@SKY-PEY SKY-PEY commented Jan 5, 2024

개요

투표 만들기 모달, 투표 상세페이지 1차 마크업 완료
차크라에 공통 버튼 테마 추가

스크린샷

image

image

주요 내용

  • 투표 만들기 모달 마크업
  • 투표 상세페이지 마크업 1차
  • 차크라 공통 버튼 테마 추가

차크라에 추가된 내용은 다음과 같습니다.

1. 모든 차크라 Button에 disabled 디자인 추가 (별도 적용해야 하는 거 없음)

image

2. 하단 고정 CTA 버튼, 기본 파란색 버튼 추가

image

2-1. CTA버튼 적용 화면

image

2-2. 적용 방법

image
차크라 Button에 variant="CTAButton" 만 적으면 됩니다, 텍스트 디자인(button)까지 다 들어가 있어요

고민한 점, 질문거리

피그마 보면서 최대한 다른 분들에게 영향안가게 했는데 만약 disabled 디자인이 다르면 알려주세요~!

  • 디테일 페이지 마크업만 하고 pr올리려 했는데 생각보다 차크라 배우는거에 시간을 많이 뺏겨서ㅠㅠ 마크업 1차로 먼저 올립니다!

발행된 이슈 연결 후 머지 후 자동 클로즈, 브랜치 삭제 연결
close #11 #30

@SKY-PEY SKY-PEY self-assigned this Jan 5, 2024
@SKY-PEY SKY-PEY linked an issue Jan 5, 2024 that may be closed by this pull request
1 task
Copy link

vercel bot commented Jan 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
trip-vote ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 7, 2024 7:52am

@SKY-PEY SKY-PEY changed the base branch from main to dev January 5, 2024 19:15
@SKY-PEY
Copy link
Contributor Author

SKY-PEY commented Jan 5, 2024

Deployedment failed 떴는데 actions에 안 뜨네요
아마 안 쓰는 import랑 주석이 걸린건가 싶은데 혹시 성후님 보실때도 actions에 안 떠있으면 어디서 fail인지 알려주실 수 있을까요?
@HOOOO98

아 근데 어차피 저는 다른분들 다 머지하고 마지막에 풀 받고 다시 올려야겠네요ㅋㅋ

@HOOOO98
Copy link
Contributor

HOOOO98 commented Jan 6, 2024

Deployedment failed 떴는데 actions에 안 뜨네요 아마 안 쓰는 import랑 주석이 걸린건가 싶은데 혹시 성후님 보실때도 actions에 안 떠있으면 어디서 fail인지 알려주실 수 있을까요? @HOOOO98

아 근데 어차피 저는 다른분들 다 머지하고 마지막에 풀 받고 다시 올려야겠네요ㅋㅋ

@SKY-PEY

/src/components/Vote/VoteDetailsBottomButton/VoteDetailsBottomButton.tsx(3,1): error TS6133: 'styles' is declared but its value is never read.

이렇게 오류가 나왔네요~
처음에 메인으로 pr 올리셔서 깃헙 액션이 감지 못한 거 같아요,,!
다른 Pr들 머지 되면 pull 받아보시고 push 하면 될 거 같아요~!

Yamyam-code
Yamyam-code previously approved these changes Jan 7, 2024
NamgungJongMin
NamgungJongMin previously approved these changes Jan 7, 2024
JSH99
JSH99 previously approved these changes Jan 7, 2024
Copy link
Member

@JSH99 JSH99 left a comment

Choose a reason for hiding this comment

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

CTA 버튼 테마 만들어주셔서 감사합니다👍🏻

@SKY-PEY SKY-PEY merged commit 2ab2df7 into dev Jan 7, 2024
3 checks passed
@SKY-PEY SKY-PEY deleted the 11-feat-create-vote branch January 7, 2024 08:12
HOOOO98 pushed a commit that referenced this pull request Jan 9, 2024
Feat: markup vote detail page and add common button style in chakra
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Feat : create vote detail page markup
6 participants