-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat: 상품 목록 페이지 컴포넌트로 분리, 공통 Layout에 Header 수정 #13
Conversation
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.
고생하셨습니다!
src/components/Layout.tsx
Outdated
|
||
interface LayoutProps { | ||
header: ReactNode; | ||
header: string; | ||
children: ReactNode; | ||
footer: ReactNode; |
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.
footer를 optional로 사용하면 어떨까요?
그러면 footer를 사용하지 않아도 굳이 null값을 넣지 않아도 될 것 같아요!
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.
오오 넵 optional로 바꿔놓을게요!
const heightClass = height ? `h-${height}` : '210px'; | ||
|
||
return ( | ||
<div className="fixed inset-0 flex items-center justify-center bg-black bg-opacity-50 z-10"> |
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.
popup 또한 modal처럼 createPortal을 사용해보는것도 좋을 것 같아요!
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.
이부분도 createPortal을 써서 코드 수정하겠습니다! 감사합니다
isBidding: boolean; | ||
} | ||
|
||
const OngoingProduct: React.FC<{ product: ProductProps }> = ({ product }) => { |
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.
React.FC를 명시하는 이유가 있을까요?
궁금합니다!
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.
배웠을때 이렇게 배웠었는데 요즘은 React.FC를 사용하는걸 지양하라고 하는군요!! 수정하겠습니다!
src/constants/route.ts
Outdated
@@ -1,8 +1,11 @@ | |||
const ROUTERS = Object.freeze({ | |||
MAIN: '/', | |||
HOME: '/home', |
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.
제가 만든 Home페이지와 main 페이지에 차이가 있나요?
차이가 없다면 home의 Route를 /로 하면 되지 않을까요?
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.
오 넵 오르트님이 만드신 Home이 메인페이지로 사용되면 기존에 있던 MainPage 삭제시키고 Home을 메인으로 사용하겠습니다!
💡 작업 내용
💡 자세한 설명
🛠️ 공통 Layout에 Header 수정
🛠️ 레이아웃 사용할 때
🛠️ 상품 목록 페이지 컴포넌트로 분리
🛠️ 공통 컴포넌트 Popup.tsx 생성
🛠️ 팝업 컴포넌트 사용 방법
🛠️ 프로필 수정 페이지 마크업
🛠️#### 프로필 인풋 생성
모달
✅ 셀프 체크리스트
closes #이슈번호