Skip to content

Conversation

@sohyun0
Copy link
Contributor

@sohyun0 sohyun0 commented Oct 12, 2025

📝 작업 개요 (필수)

  1. 기존 컴포넌트의 Page 방식에서 api 에서 사용하는 offset 방식으로 로직 변경
    page 방식 = offset / limit + 1 -> offset 방식 = (page - 1) * limit
  2. 아이콘에서 onClick 이벤트가 실행되고 있어 Button 태그로 이벤트 위치 변경
  3. 페이지네이션 내부 함수는 handle함수로 관리하도록 변경
  4. css 간소화 및 불필요한 div 제거 (버튼 크기 반응형 추가 , 여백을 담당하던 높이는 페이지별로 상이하므로 제외)
  5. 자체 배경컬러와 아이템의 배경컬러가 상이하여 제거
  6. ui/index.ts 에 export 선언이 되어있지않아 추가했습니다.
  7. 페이지네이션 여백및 배경 제외로 className props 추가
  • TimeInput 의 에러로인해서 배포에러가 생겨 해당 부분도 같이 수정했습니다(합의된 내용)

✨ 작업 내용 (필수)

  • 기능 구현
  • 버그 수정
  • 스타일/UI 변경
  • 리팩토링
  • 최적화/성능개선
  • 문서 업데이트
  • 기타 변경사항

📸 스크린샷

before
스크린샷 2025-10-13 오전 3 39 18
after
스크린샷 2025-10-13 오전 3 37 08

🧐 해결해야 하는 문제

🤔 리뷰어 확인 필요 사항

currentPage 에서 흰색 글씨가 얇게 있어 눈에 안띄는것 같아 bold 로 바꿨는데 어떤가요 ? 🧐

🔗 관련 이슈

  • Closes #이슈 번호
  • Related to #이슈 번호

🛠️ 후속 작업

✅ 체크리스트 (필수)

  • 작업한 내용과 커밋 메시지 컨벤션을 통일했는지 확인
  • 내가 작성한 코드를 테스트까지 완료했는지 잘 작동했는지 확인
  • ESLint 검사 통과
  • Prettier 포맷팅 적용
  • TypeScript 에러 없음
  • 빌드 에러 없음

@sohyun0 sohyun0 self-assigned this Oct 12, 2025
@sohyun0 sohyun0 added ✨ feat 새로운 기능 추가 💄 design CSS 등 UI 디자인 변경 labels Oct 12, 2025
@vercel
Copy link

vercel bot commented Oct 12, 2025

@sohyun0 is attempting to deploy a commit to the projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
thejulge Ready Ready Preview Comment Oct 13, 2025 5:27pm

@BaeZzi813
Copy link
Contributor

currentPage 에서 흰색 글씨 bold 가 더 눈에 띄어서 훨씬 좋은 것 같습니다! 👍

반응형에 따라 보여지는 페이지 개수 '''데스크탑 10 / 태블릿 7 / 모바일 5''' 로 픽스 하실지만 답변 주시면 승인 머지 후 useMedaiQuery 사용해서 반응형 대응해서 다시 올리겠습니다!

@sohyun0
Copy link
Contributor Author

sohyun0 commented Oct 13, 2025

currentPage 에서 흰색 글씨 bold 가 더 눈에 띄어서 훨씬 좋은 것 같습니다! 👍

반응형에 따라 보여지는 페이지 개수 '''데스크탑 10 / 태블릿 7 / 모바일 5''' 로 픽스 하실지만 답변 주시면 승인 머지 후 useMedaiQuery 사용해서 반응형 대응해서 다시 올리겠습니다!

네 저는 10/7/5좋은것 같습니다 ! 320px 일때 모바일 5로 가져가면 딱 맞는 사이즈가 됩니다
아래는 320px 일때 5일때 형태입니다!
만약 그 이하의 사이즈도 대응하고싶다면 (예전 폴드 280px) 3까지로 가야하겠지만 그건 선택하시면 될것 같아요!

스크린샷 2025-10-13 오후 7 47 00

@BaeZzi813 BaeZzi813 merged commit 98096b8 into codeit-FE18-part3:develop Oct 14, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💄 design CSS 등 UI 디자인 변경 ✨ feat 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants