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

[Style/#37] 즐겨찾기 리스트 페이지 퍼블리싱 #38

Merged
merged 14 commits into from
Nov 26, 2024

Conversation

gwagjiug
Copy link
Member

@gwagjiug gwagjiug commented Nov 25, 2024

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

📢 To Reviewers

  • 서연이랑 똑같은 뷰이긴 하지만 합세니깐 다른 방법을 좀 고민해서 해봤습니다.

📸 스크린샷

image

🔗 참고 자료

@gwagjiug gwagjiug self-assigned this Nov 25, 2024
Copy link
Collaborator

@youtheyeon youtheyeon left a comment

Choose a reason for hiding this comment

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

진짜 굿이네요!
근데 즐겨찾기 버튼 브랜치에서 바로 요 페이지 브랜치로 체크아웃 했는지 커밋이 쌓인 것 같습니다.
develop으로 나갔다가 이동해야 커밋이 분리된다고 하네요!

import BurgerPost from '@components/BurgerPost/BurgerPost';
const FavoriteListPage = () => {
const productLength = PRODUCT_LIST.length;
const bannerTitle = ['내 메뉴'];
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기를 배열로 지정해 줄 이유가 있나요? 그냥 string으로 줘도 될 것 같습니당

Copy link
Member Author

Choose a reason for hiding this comment

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

나중에 뭐가 추가될 수 있을 것 같아서 배열로 하긴했는뎅 string으로 할게용~

@@ -2,14 +2,15 @@ import { createBrowserRouter } from 'react-router-dom';
import Main from '@pages/main/Main';
import Layout from '@layouts/Layout';
import BurgerListPage from '@pages/burgerlist/BurgerListPage';

import FavoriteListPage from '@pages/favoriteList/FavoriteListPage';
Copy link
Collaborator

Choose a reason for hiding this comment

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

진짜 별 거 아니지만 혹시 여기 임포트랑 라우터 사이에 한 줄 띄워주실 수 있나요?

Copy link
Member Author

Choose a reason for hiding this comment

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

반영완!

Copy link
Collaborator

@imddoy imddoy left a comment

Choose a reason for hiding this comment

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

화긴 완~~ 수고했습니당

Comment on lines +8 to +10
flex-shrink: 0;
background: var(--Blackgrad, rgba(0, 0, 0, 0.5));
.banner__title {
Copy link
Collaborator

Choose a reason for hiding this comment

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

띰 사용해주세요!!

Comment on lines +7 to +9
height: 16.6rem;
flex-shrink: 0;
background: var(--Blackgrad, rgba(0, 0, 0, 0.5));
Copy link
Collaborator

Choose a reason for hiding this comment

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

flex-shrink를 주신 이유가 있나요??

@gwagjiug gwagjiug merged commit 836e988 into develop Nov 26, 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.

[ Style ] 즐겨찾기 페이지 퍼블리싱
3 participants