-
Notifications
You must be signed in to change notification settings - Fork 8
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
구경가기 상단 캐러셀 구현 #849
Conversation
const ITEM_WIDTH = windowWidth < BREAKING_POINT.MOBILE ? 144 : 288; // CarouselItem 넓이 | ||
const ITEM_GAP = 16; // 1rem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분은 스타일의 크기와 밀접한 관련이 있기 때문에 상수 처리를 해주거나, 해당 스타일과 묶어주고 싶었으나 현재 좋은 방법이 떠오르지 않아 일단 매직넘버로 처리하고 주석 달아두었습니다.
수고하셨습니다! 배경을 블러된 이미지 정도로 바꾸면 어떨까 생각해요! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
배포 일정때문에 코드리뷰는 이후 진행하겠습니다
⚡️ 관련 이슈
🪧구현 결과
2024-10-22.6.30.24.mov
2024-10-22.6.28.13.mov
📍주요 변경 사항
Carousel
,HotTopicCarousel
컴포넌트 및useHotTopic
생성Footer
에 이메일 직접 표기하도록 변경🎸기타
🍗 PR 첫 리뷰 마감 기한
10/22 18:00