[Feat/#97] 스토어 상세보기 페이지 퍼블리싱 #111
Conversation
| ), | ||
| small: createButtonSizeStyles(vars.fonts.body02_sb_14, '4.4rem'), | ||
| medium: createButtonSizeStyles(vars.fonts.head06_sb_16, '4.8rem'), | ||
| semiLarge: createButtonSizeStyles(vars.fonts.head06_sb_16, '5.5rem'), |
There was a problem hiding this comment.
지금 실제로 large 사이즈로 쓰이는 컴포넌트가 한개밖에 없어서 지금 빠르게 수정하는게 좋을 것 같아요.
height값이 5.5rem인 이친구를 large로 바꾸고 6rem인 친구를 xLarge로 바꾸면 좋을 것 같습니다!
| storeAddress, | ||
| } from './Banner.css'; | ||
|
|
||
| const Banner = () => { |
There was a problem hiding this comment.
추후 api 요청시에는 storeId 값을 prop으로 받아와야 할 것 같아요!
|
|
||
| export const bannerContainer = style([ | ||
| flexGenerator('column', 'center', 'flex-start'), | ||
| { width: '100%', padding: '0 2rem' }, |
There was a problem hiding this comment.
여기 gap: '1.8rem' 이 필요할 것 같은데 확인 한번 부탁드립니다!
There was a problem hiding this comment.
원래 margin으로 주고 있었는데 gap으로 수정했습니다 감사해요!!!
| export const storeInfoWrapper = style([ | ||
| flexGenerator('row', 'space-between', 'flex-start'), | ||
| { | ||
| marginBottom: '1.6rem', |
There was a problem hiding this comment.
Banner컴포넌트와 그 아래 Tab 컴포넌트 사이의 1.6rem 만큼 띄우기 위해 marginBottom을 여기 쓰신 것 같은데 컴포넌트 내부에서 컴포넌트와 컴포넌트 사이의 거리를 주는건 어색해 보여요.
StorePage 컴포넌트의 스타일에서 marginBottom을 주거나 혹은 bannerContainer에 주는건 어떨까요?
| <TextButton size={'semiLarge'} color={'stroke'}> | ||
| 지도뷰 | ||
| </TextButton> | ||
| <TextButton size={'semiLarge'}>주문하러가기</TextButton> |
There was a problem hiding this comment.
주문하러 가기
로 띄워쓰기 넣어주세요!
| }, | ||
| ]); | ||
|
|
||
| export const buttonStyle = style({ |
| <Image | ||
| key={design.cakeId} | ||
| src={design.cakeImageUrl} | ||
| hasIcon | ||
| isActive={design.isLiked} | ||
| /> | ||
| ))} |
There was a problem hiding this comment.
이미지 클릭하면 한장씩 볼 수 있는 기능이 추가되어야 할 것 같아요!
There was a problem hiding this comment.
꺅.. 이슈 하나 파서 새로 작업하겠습니다!
|
🍰 Storybook 배포가 완료되었습니다! 🔗 https://677cb5c3ab50c3f524eaae19-eczhdpfcgb.chromatic.com/ |
📌 관련 이슈번호
체크리스트
✅ Key Changes
📢 To Reviewers
📸 스크린샷 or 실행영상
2025-01-16.10.04.09.mov