Conversation
| // 모달 영역 밖 클릭 시 닫기 | ||
| useEffect(() => { | ||
| const handleModal = (event) => { | ||
| if (modal && !modalRef.current.contains(event.target)) { |
There was a problem hiding this comment.
modalRef가 초기에 null이기에 modalRef.current?.contains ?.를 사용하시는 것이 좋습니다
| starImage === STAR_ICON | ||
| ? setStarImage(COLOR_STAR_ICON) | ||
| : setStarImage(STAR_ICON); | ||
| }; |
There was a problem hiding this comment.
setStarImage(starImage === STAR_ICON ? COLOR_STAR_ICON : STAR_ICON) 이렇게 가능하실 것 같습니다
| folderId, | ||
| setFolderId, | ||
| }) => { | ||
| let hasCards = false; |
There was a problem hiding this comment.
useState값이 아닌, let으로 사용하신 이유가 있으실까요 ?
There was a problem hiding this comment.
useState를 사용했을 때 제가 카드를 구현한 방식이 folderId === item.folder_id 일 때마다 계속 true로 변경하게 만들어서 아래와 같은 경고가 발생했습니다.
해결을 못해서 let으로 해두었는데 카드 구현 방식을 다시 생각해 봐야 할 것 같습니다.
Too many re-renders. React limits the number of renders to prevent an infinite loop.
| const handleClick = () => { | ||
| setAddModal(true); | ||
| }; | ||
|
|
There was a problem hiding this comment.
function 과 화살표함수를 섞어쓰셨는데, 하나로 통일해주시는 것이 좋을 것 같습니다
| text: "인스타그램 바로가기 아이콘", | ||
| }, | ||
| ]; | ||
|
|
| className={cx("input-text")} | ||
| type="search" | ||
| placeholder="링크를 추가해 보세요." | ||
| ></input> |
There was a problem hiding this comment.
input 의 경우 단일태그로 작성하실 수 있을 것 같습니다
| text === "삭제" && setDeleteModal(true); | ||
| text === "이름 변경" && setAddModal(true); | ||
| text === "공유" && setShareModal(true); | ||
| }; |
|
|
||
| const cx = classNames.bind(styles); | ||
|
|
||
| export const NavUserProfile = ({ user }) => { |
There was a problem hiding this comment.
추후에는 해당 컴포넌트에서 필요한 데이터만 props로 받도록 하는것이 좋습니다.
name / profileImage만 받는 형식이 되는 것입니다.
해당 컴포넌트는 잘 작성해주셨습니다
|
|
||
| export const FolderPage = () => { | ||
| const [user, setUser] = useState([]); | ||
| const [Links, setLinks] = useState([]); |
There was a problem hiding this comment.
Links 대문자보다는 소문자로 시작하시는 것이 좋습니다
요구사항
기본
주요 변경사항
스크린샷
멘토에게
타입스크립트로 아직 완성을 못했습니다 못한 부분은 최대한 빨리 추가하도록 하겠습니다!
components 폴더에 index.js 파일을 만들어서 컴포넌트들을 전부 export하고 컴포넌트를 사용할 때 경로를
import { NavigationBar } from "components"로 폴더명만 작성하면 된다고 알고 있었는데 찾을 수 없다고 에러가 발생해서
"../components"로 변경했습니다. 어느 부분이 문제인지 알고 싶습니다.
읽어주셔서 감사합니다.
셀프 코드 리뷰를 통해 질문 이어가겠습니다.