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

구경가기 상단 캐러셀 구현 #849

Merged
merged 5 commits into from
Oct 22, 2024
Merged

Conversation

Hain-tain
Copy link
Contributor

@Hain-tain Hain-tain commented Oct 21, 2024

⚡️ 관련 이슈

🪧구현 결과

  • 데스크탑
2024-10-22.6.30.24.mov
  • 모바일
2024-10-22.6.28.13.mov

📍주요 변경 사항

  • Carousel, HotTopicCarousel 컴포넌트 및 useHotTopic 생성
  • Footer 에 이메일 직접 표기하도록 변경
  • 템플릿 카드, 템플릿 상세 페이지에서 시간과 작성자 색상 변경
    스크린샷 2024-10-22 오전 6 51 08

🎸기타

  • 월하가 구현해준 캐러셀 너무 예쁘고 훌륭하였으나 클릭할 때 애니메이션이 제어되지 않는 문제를 해결할 수 없어 부득이하게 캐러셀을 별도로 생성하여 구현하였습니다.
  • 캐러셀 양쪽 이동 버튼을 기존 버튼이 아닌 별도의 스타일을 준 button으로 만들었습니다. 그 이유는 디자인적으로 아쉽기도 하고, 저희가 만들어둔 공용 버튼 컴포넌트가 가로 패딩이 너무 크게 느껴졌기 때문입니다. (사진 참조)
스크린샷 2024-10-22 오전 6 24 47

🍗 PR 첫 리뷰 마감 기한

10/22 18:00

@Hain-tain Hain-tain added feature 기능 추가 FE 프론트엔드 zap 리뷰 우선순위가 높은 사항 labels Oct 21, 2024
@Hain-tain Hain-tain added this to the 6차 스프린트 🦴 milestone Oct 21, 2024
@Hain-tain Hain-tain self-assigned this Oct 21, 2024
Comment on lines +25 to +26
const ITEM_WIDTH = windowWidth < BREAKING_POINT.MOBILE ? 144 : 288; // CarouselItem 넓이
const ITEM_GAP = 16; // 1rem
Copy link
Contributor Author

Choose a reason for hiding this comment

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

이 부분은 스타일의 크기와 밀접한 관련이 있기 때문에 상수 처리를 해주거나, 해당 스타일과 묶어주고 싶었으나 현재 좋은 방법이 떠오르지 않아 일단 매직넘버로 처리하고 주석 달아두었습니다.

@vi-wolhwa
Copy link
Contributor

수고하셨습니다! 배경을 블러된 이미지 정도로 바꾸면 어떨까 생각해요!

Copy link
Contributor

@Jaymyong66 Jaymyong66 left a comment

Choose a reason for hiding this comment

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

배포 일정때문에 코드리뷰는 이후 진행하겠습니다

@Jaymyong66 Jaymyong66 merged commit bb870d8 into dev/fe Oct 22, 2024
2 checks passed
@Jaymyong66 Jaymyong66 deleted the feat/837-hot-topic-carousel branch October 22, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 feature 기능 추가 zap 리뷰 우선순위가 높은 사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

3 participants