-
Notifications
You must be signed in to change notification settings - Fork 3
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
진입 페이지 UI 구현 (스페이스 입장 동작 및 API 동작 제외) #83
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다..! LGTM * 100
<span className="text-[64px]">Honey Flow</span> | ||
<span className="text-[16px]"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변경 제안은 아니고, 이런 텍스트 크기 같은 부분은 나중에 저희끼리 더 구체적으로 변수로 정해서 넣으면 될 것 같습니다. (text-sm
이런 식으로)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 좋습니다 !
저도 |
className="mt-1 block w-full border rounded p-2" | ||
onChange={(e) => { | ||
setSpaceName(e.target.value); | ||
}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
찾아보니 shadcn에도 Label, Input 컴포넌트가 있는 것 같은데, 일반 태그를 사용하신 이유가 궁금합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shadcn에 해당 컴포넌트의 존재 여부를 몰라서 일반 태그를 커스텀하여 사용하고 있었습니다 ! ㅎㅎ
알려주셔서 감사합니다! 수정 후 커밋 작성했습니다.
<DialogTrigger asChild> | ||
<Button>스페이스 생성</Button> | ||
</DialogTrigger> | ||
<DialogContent className="sm:max-w-[425px]"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이게 tailwind의 반응형 문법이군요...👍 저도 tailwind는 처음이라 이런 것들을 최대한 많이 눈에 익혀둬야할 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tailwind가 반응형 지원을 아주 간편하게 설정할 수 있었습니다 ㅎㅎ.
다만 조금 더 디테일한 반응형 스타일에 대해서 저도 학습하고 수정을 거쳐야할 것 같아요 !
@hoqn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 이게 그 뒤에 커밋이 이뤄져서 Approve가 풀렸었네요. 아마 이 부분도 전체 설정으로 새로운 커밋이 생겨도 유지되도록 할 수 있었던 것 같은데 한 번 확인해볼게요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 번거롭게 해드려서 죄송합니다 ㅜㅜ. 커밋이 추가 작성되면 approve가 다 풀리는군요 |
✏️ 한 줄 설명
✅ 작업 내용
🏷️ 관련 이슈
📸 스크린샷/영상
📌 리뷰 진행 시 참고 사항
shadcn
,tailwind css
를 사용해보는 것은 처음인데 코드 작성방식이 괜찮은 건지? 스스로 판단이 잘 되지 않습니다.Home.tsx
구현 코드를 중점적으로 검토해주시면 감사하겠습니다!나는 보통 이렇게 한다
이런 느낌으로 편히 말씀해주세요 . ㅎㅎ 적극 반영하겠습니다.