skku-comit / comit-website

CoMit HomePage Frontend
https://comit-website.vercel.app
2 stars 1 forks source link

회원가입 페이지 개선 + clubroom 탭에서 footer 위치 수정 #128

Closed Imjungjuna closed 3 months ago

Imjungjuna commented 3 months ago

Description

회원가입 페이지 개선하였습니다.

Collapsible 기능 완성했습니다. 이메일 전체 주소 입력하는 걸로 수정했습니다. rounded되도록 수정했습니다. 반응형으로 수정하였습니다. Collapsible ui 완성했습니다. (DropDown ui는 원래 없어서 어떤 걸 말씀하셨는지 모르겠어요ㅠ)

추가 수정사항: 전체적인 패딩과 간격 수정, 버튼 호버 시 opacity 변경, validation 오류시 input border 빨간색으로 변경, password 영문 숫자 적어도 한 글자씩은 포함하게 수정, 에러 메시지 동시에 display되는 문제 해결

close #122

Additional context

clubroom 페이지에서 footer가 나올때까지 여백 생기는 문제 해결: footer 위치 화면에 맞게 변경해놓았습니다. 나중에 layout.tsx 구조를 바꾸면 좋을 것 같습니다.

image -> 이런 식으로 구성해서 항상 footer 위치가 정해지게 하는 것은 어떤가요?

vercel[bot] commented 3 months ago

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

Name Status Preview Comments Updated (UTC)
comit-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2024 8:04am
jihostudy commented 3 months ago

수고하셨습니다! 몇가지 수정사항 요청입니다.

[Form]

  1. collapseButton은 FormData에 필요하지 않은 데이터로 보입니다.
    • FormData는 유저가 해당 페이지에서 제공해주는 데이터입니다.
    • collapseButton은 토글 다운/업을 하기 위한 필드로 보이는데, 이는 useState<boolean>(false)로 관리하고, FormData에서는 제외하는 것이 좋아 보입니다.

[반응형]

Imjungjuna commented 3 months ago

네 두가지 다 수정해보겠습니다.

Imjungjuna commented 3 months ago

말씀주신 대로 toggle 옵션은 FormData에서 제외하였습니다. 화면이 짧을 때 Header + Content + Footer가 h-screen이 되도록 하는 이슈는 곧 해결하겠습니다.