-
Notifications
You must be signed in to change notification settings - Fork 43
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
[남기연] sprint7 #273
The head ref may contain hidden characters: "React-\uB0A8\uAE30\uC5F0-sprint7"
[남기연] sprint7 #273
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.
기초 프로젝트 마치고 이어서 스프린트 미션 하시느라 수고 많으셨어요~
import { useEffect } from "react"; | ||
import { useState } from "react"; | ||
import { getProductList } from "../../../api/ItemApi"; | ||
import "../MarketPage"; | ||
import ItemCard from "./ItemCard"; | ||
import ItemCard from "../../../components/ItemCard/ItemCard.jsx"; |
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.
baseUrl 설정해서 절대경로 사용하는 것도 고려해보세요. (참고)
} | ||
|
||
.btn { | ||
box-sizing: border-box; |
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.
개발자 도구에서 봤을 때,
위 설정이 없으면, global css로 적용한 스타일이 적용되지 않아서 기본 값인 content-box로 설정되는 걸로 보이는데,
왜 여기서는 적용 되지 않는지 뭔가 트리키한 이유가 있을 거 같은데, 바로 어떤게 원인이다 말씀드리기가 어렵네요.
추후 정확히 알게되면 말씀드릴게요.
반대로 기연님이 먼저 알게되면 저한테 알려주세요~
background-color: #ffffff; | ||
} | ||
|
||
.dropdownIcon:hover { |
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.
dropdown-icon, dropdownIcon 둘 서로 다른 용도로 사용하는 건가요?
@@ -0,0 +1,21 @@ | |||
import styles from "./Tag.module.css"; | |||
|
|||
const Tag = ({ tags }) => { |
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.
컴포넌트 이름을 Tags라고 하거나
Tag컴포넌트에서 하나의 tag만 리턴하는게 좋을 것 같아요.
@@ -1,6 +1,6 @@ | |||
import "./onSortSelect.css"; | |||
|
|||
function SortSelect({ onSortSelection }) { | |||
function onSortSelect({ onSortSelection }) { |
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.
onOOOO 이름은 보통 프로퍼티로 넘겨주는 이벤트 핸들러에 사용해서 대문자로 시작하는 명사형 컴포넌트 이름이 좋을 것 같아요.
@@ -10,7 +10,7 @@ const AddItemPage = () => { | |||
tag: "", | |||
imgFile: null, | |||
}); | |||
const [disabled, setDisabled] = useState(true); | |||
const [canSubmit, setCanSubmit] = useState(true); |
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.
disabled -> canSubmit 으로 바뀌면 boolean 설정도 반대로 해줘야 할 거 같아요.
@@ -37,7 +37,7 @@ const AddItemPage = () => { | |||
<form className="addItemContainer"> | |||
<div className="addItemHeader"> | |||
<h1 className="headerTitle">상품 등록하기</h1> | |||
<button className={`addItemBtn ${disabled ? "" : "active"}`} onSubmit={handleSubmit} disabled={disabled}> | |||
<button className={`addItemBtn ${canSubmit ? "" : "active"}`} onSubmit={handleSubmit} disabled={canSubmit}> |
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.
해당 부분도 boolean 설정 반대로 되는 것에 맞춰줘야 할 거 같아요.
</div> | ||
</div> | ||
</div> | ||
<AllItemSectionHeader /> |
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.
추상화 레벨 맞춰주는 것 좋아요~!
요구사항
기본
상품 상세 페이지 주소는 "/items/{productId}" 입니다.
response 로 받은 아래의 데이터로 화면을 구현합니다.
=> favoriteCount : 하트 개수
=> images : 상품 이미지
=> tags : 상품태그
=> name : 상품 이름
=> description : 상품 설명
목록으로 돌아가기 버튼을 클릭하면 중고마켓 페이지 주소인 "/items" 으로 이동합니다
문의하기에 내용을 입력하면 등록 버튼의 색상은 "3692FF"로 변합니다.
response 로 받은 아래의 데이터로 화면을 구현합니다
=> image : 작성자 이미지
=> nickname : 작성자 닉네임
=> content : 작성자가 남긴 문구
=> description : 상품 설명
=> updatedAt : 문의글 마지막 업데이트 시간
심화
주요 변경사항
멘토에게