Skip to content

[refactor/#81] 영화 정보 확인 페이지 QA#82

Merged
woodzverse merged 9 commits intodevelopfrom
feat/#81/movie-detail-page-qa
Nov 27, 2025
Merged

[refactor/#81] 영화 정보 확인 페이지 QA#82
woodzverse merged 9 commits intodevelopfrom
feat/#81/movie-detail-page-qa

Conversation

@woodzverse
Copy link
Collaborator

@woodzverse woodzverse commented Nov 26, 2025

📌 Related Issues

✅ 체크 리스트

  • PR 제목의 형식을 잘 작성했나요? e.g. [feat/#이슈번호] PR 템플릿 작성
  • 빌드가 성공했나요? (pnpm build)
  • 컨벤션을 지켰나요?
  • 이슈는 등록했나요?
  • 리뷰어와 라벨을 지정했나요?

📄 Tasks + 📷 Screenshot

  • 헤더: 뒤로가기 기능 작동 X → 뒤로가기 기능 추가
2025-11-27.12.04.02.mov
  • 더보기 버튼 여백 문제 + 한 줄 소개 옆에 더보기를 누르면 긴 줄거리 보이도록 수정
  • 스크린샷 2025-11-27 오전 12 48 20
  • 이벤트 표시 패딩 수정
스크린샷 2025-11-27 오전 12 48 32
  • 리뷰 여백 수정
스크린샷 2025-11-27 오전 12 48 47
  • 누적 관람 상단 여백 수정
스크린샷 2025-11-27 오전 12 48 52
  • 헤더 아이콘 + 리뷰 좋아요 아이콘 호버 효과 추가(눌리는 느낌)
2025-11-27.12.44.55.mov
2025-11-27.12.46.17.mov
    • 추가: 헤더 정렬 수정(제목이 가운데 위치하도록...)
스크린샷 2025-11-27 오전 1 03 06

@woodzverse woodzverse self-assigned this Nov 26, 2025
@woodzverse woodzverse added 💎 주영 🎉Refactor 코드 구조나 내부 구현 개선 작업 labels Nov 26, 2025
@github-actions
Copy link

빌드 및 린트 결과

일부 검사 실패 ⚠️

  • 린트 실패 ❌
  • 빌드 실패 ❌

@github-actions
Copy link

빌드 및 린트 결과

모든 검사 통과 🎉

  • 린트 통과 ✅
  • 빌드 성공 🎊

@github-actions
Copy link

빌드 및 린트 결과

모든 검사 통과 🎉

  • 린트 통과 ✅
  • 빌드 성공 🎊

Copy link
Collaborator

@huniversal huniversal left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!
리뷰에 커멘트 한번만 답장해주시면 감사하겠습니다!!

</Button>

<section className='flex flex-col gap-[1.5rem]'>
<section className="flex flex-col items-start self-stretch py-[0.9375rem] gap-[1px]">
Copy link
Collaborator

Choose a reason for hiding this comment

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

저 py-[0.9375rem]은 무엇일까요..?

Copy link
Member

Choose a reason for hiding this comment

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

gap-[1px]은 rem 사용하면 좋을 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다. py-[0.9375rem] 이거는 피그마에 패딩 픽셀값이 15px여서 했던 건데 실관람평 버튼에 gap 주는 쪽으로 수정했습니다.

Copy link
Member

@sminha sminha left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다👍🏻

</Button>

<section className='flex flex-col gap-[1.5rem]'>
<section className="flex flex-col items-start self-stretch py-[0.9375rem] gap-[1px]">
Copy link
Member

Choose a reason for hiding this comment

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

gap-[1px]은 rem 사용하면 좋을 것 같아요!

<IconSystemShare
aria-label='공유하기'
className='h-[2.4rem] w-[2.4rem] cursor-pointer text-white'
className='h-[2.4rem] w-[2.4rem] cursor-pointer text-white transition-transform active:scale-90 hover:opacity-70'
Copy link
Member

Choose a reason for hiding this comment

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

추가해주신 속성들이 반복되고 있는데, 아예 따로 변수로 정의해두고 cn으로 합쳐도 좋을 것 같아요!

@github-actions
Copy link

빌드 및 린트 결과

일부 검사 실패 ⚠️

  • 린트 실패 ❌
  • 빌드 성공 🎊

@github-actions
Copy link

빌드 및 린트 결과

모든 검사 통과 🎉

  • 린트 통과 ✅
  • 빌드 성공 🎊

@woodzverse woodzverse merged commit 83b6e3f into develop Nov 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎉Refactor 코드 구조나 내부 구현 개선 작업 💎 주영

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: 영화 정보 확인 페이지 QA

3 participants