Skip to content

Conversation

@dayannne
Copy link
Contributor

@dayannne dayannne commented Dec 31, 2024

📌 관련 이슈번호


체크리스트

  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (yarn build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

작업한 QA 리스트

  • 뷰포트 높이 기준 수정 및 모바일 반응형 스타일 적용
  • 캐러셀 현재 인덱스 표시 버그
  • 온보딩 유효성 검사 - 40까지 입력 제한 / 체크 클릭 시 인풋 입력내용 삭제
  • 메인 예외처리 주차 0일 때 0 그대로 뜸
  • 수정 페이지 네브바 제거
  • 모름일 때 임신 ??주차로 표시
  1. 뷰포트 높이 기준 / 레이아웃 수정
    로컬에선 이상 없는데 한 번 main 배포 후 확인 필요할 것 같습니다!
  2. 네비게이션 바 하단 상태바 공간 추가
  3. 모바일 반응형 적용
    • 너비 최대 430px 최소 375px 기준으로 변경
    • 버튼, 인풋 등 반응형에 맞춰 스타일 수정 적용
  4. 온보딩 주차 입력 유효성 검사
  5. 임신 주차 예외처리 수정
  6. 캐러셀 현재 인덱스 표시 버그 수정

📢 To Reviewers


📸 스크린샷 or 실행영상

- 왼쪽 스크롤 시 현재 인덱스 잘못 표시되는 버그
- pregnancyWeek값이 falsygks 값일 때 임신 ??주차로 표시 & progress bar 숨김
- 임신 주차 입력값 : 숫자, 1-40 범위, 빈 문자열 허용
- 버튼 : 닉네임 빈 문자열 & 주차 입력값 빈 문자열 or 모름 미체크 시 비활성화
- 부모 요소 기준으로 변경
- 스크롤 불필요 시 네비게이션 바 고정
@github-actions github-actions bot added ✨ FEAT 기능 구현 🐛 BUGFIX 버그를 고친 경우 labels Dec 31, 2024
- 부모 요소 기준으로 변경
- 스크롤 불필요 시 네비게이션 바 고정
@junny97 junny97 merged commit 1fae295 into develop Jan 1, 2025
@junny97
Copy link
Member

junny97 commented Jan 1, 2025

모바일 반응형까지 수정한거 확인했습니다~ 아마 배포 후에도 잘 보일거에요😃

@junny97 junny97 mentioned this pull request Jan 2, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 BUGFIX 버그를 고친 경우 ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 1차 QA 사항 수정

3 participants