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

155-feature/mainpage 수정 #173

Merged
merged 6 commits into from
Oct 7, 2024
Merged

155-feature/mainpage 수정 #173

merged 6 commits into from
Oct 7, 2024

Conversation

thsk3
Copy link
Collaborator

@thsk3 thsk3 commented Oct 7, 2024

📝 Summary

mainpage의 자잘한 수정사항들을 적용했습니다.

✨ Changes

  • mainpage의 장르버튼 컨테이너 작동하도록 수정
  • mainpage loader 위치 조정
  • banner 너비가 공간을 모두 차지하도록 수정
  • 컴포넌트 너비, 여백 수정
  • 개봉 예정 공연 pagination 적용
  • useYouTube 에러 캐싱

🖼️ Screenshots

✅ PR Checklist

  • 하나의 목적을 가진 PR입니다.
  • 코딩 컨벤션과 스타일 가이드를 준수합니다. 📌Conventions
  • 불필요한 코드 중복이 없습니다.
  • 컴포넌트의 책임이 단일합니다.
  • 리액트 훅을 올바르게 사용했습니다.
  • 민감한 정보를 포함하지 않았습니다.
  • 불필요한 콘솔 로그나 주석을 포함하지 않았습니다.
  • 컴포넌트 key값에 고유한 값을 할당했습니다.

🔗 References

  • Issue:

💬 Comments

- 장르버튼 props 다시 줘서 잘 작동하도록 고침
- loader 위치 조정함
- banner가 화면을 다 차지하도록 함
- 화면 너비에 따라 슬라이드에서 보이는 요소의 개수를 다르게 처리함
@shlee9999 shlee9999 merged commit 647740e into main Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants