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

[FE][Feat] #14 : bottomsheet 관련 로그인, 회원가입 #273

Merged
merged 12 commits into from
Nov 26, 2024

Conversation

juwon5272
Copy link
Collaborator

📝 PR 개요

  • x버튼 클릭시 바텀시트 최소 높이로 변경
  • 바텀시트 사용하여 로그인
  • 로그인 되었을 경우 채널 목록 나타남
  • 로그인 회원가입에서 잘못된 입력인 경우 빨간 글씨로 안내
  • 바텀시트 로직 수정

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#14

📷 스크린샷 및 동영상 (선택 사항)

2024-11-26.4.51.19.mov

@juwon5272 juwon5272 added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 26, 2024
@juwon5272 juwon5272 self-assigned this Nov 26, 2024
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

혹시 바텀시트를 중간 정도 높이까지 올렸을 때, 잘 멈추나요? 제 현재 상태에서는 끝까지 올려야만 올라가는 것 같아서요!

@juwon5272
Copy link
Collaborator Author

2024-11-26.5.52.32.mov

네 여기 영상처럼 잘 멈춥니다

Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@juwon5272 juwon5272 merged commit b3f79c1 into frontend Nov 26, 2024
4 checks passed
@juwon5272 juwon5272 deleted the feature/fe/#14-bottomsheet-content branch December 5, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants