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] 사용자 정보등록 모달 구현 #70

Merged
21 commits merged into from
Aug 6, 2024
Merged

[feat] 사용자 정보등록 모달 구현 #70

21 commits merged into from
Aug 6, 2024

Conversation

lybell-art
Copy link
Collaborator

#️⃣ 연관 이슈

📝 작업 내용

사용자 정보등록 모달을 추가했습니다.

  • AuthModal 컴포넌트입니다. 이 모달이 닫히고 사용자가 정보등록을 완료했을 때 실행시킬 콜백은 onComplete props로 넘겨주면 됩니다.
  • onComplete props에 들어갈 함수는 신규 사용자인지 확인하는 boolean형 매개변수를 1개 받을 수 있습니다.

타이틀을 반응형으로 바꿨습니다.

  • 타이틀은 이제 반응형입니다. 360px일 때 타이틀은 이제 2줄이 됩니다.
  • 타이틀의 position 속성을 sticky로 바꿔서, 모달이 띄워질 때 타이틀이 표시되지 않도록 했습니다.

GNB를 반응형으로 바꿨습니다.

  • GNB의 현재 스크롤 보여주는 검은색 바는 이제 자바스크립트에서 계산된 --pos가 아니라 currentSection 그 자체인 --section 값을 css로 넘겨주며, css에서 요소의 위치와 간격을 지정하고 반응형으로 계산하도록 했습니다.
    • 선언적 반응형 애니메이션에 효과적입니다.
  • 아직까지는 태블릿 상태만 반응되게 했습니다. 모바일에서는 햄버거 목록을 띄워줄 생각입니다.

💬 리뷰 요구사항

현재는 사용자가 정보등록을 완료하면 localStorage로 토큰은 저장하지만, 별도의 상태관리는 하지 않고 있습니다.
저희가 사용자 정보등록 완료했을 때 상단 헤더의 정보등록 렌더링을 바꿔야 하며, 그러기 위해서는 어딘가의 전역 상태관리 라이브러리에 사용자의 이름 정보를 넘겨주어야 합니다.
그걸 서버측에서 어떻게 받아오는 게 좋으려나요...

[refactor] Auth Modal이 더 이상 고정된 width/height값을 가지지 않고 페이지 자식의 height값을 갖도록 수정
[comment] 디자이너에 대한 분노 표출
[design] 타이틀 텍스트 반응형으로 수정
[fix] 모달이 띄워질 때 타이틀 텍스트가 표시되는 버그 수정
@lybell-art lybell-art added the feat 기능 구현 label Aug 6, 2024
@lybell-art lybell-art requested a review from darkdulgi August 6, 2024 07:15
@lybell-art lybell-art self-assigned this Aug 6, 2024
@win-luck win-luck linked an issue Aug 6, 2024 that may be closed by this pull request
19 tasks
@darkdulgi darkdulgi closed this pull request by merging all changes into dev in a8c5d69 Aug 6, 2024
@lybell-art lybell-art deleted the feature/17-auth branch August 17, 2024 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 메인-모달-정보등록
2 participants