-
Notifications
You must be signed in to change notification settings - Fork 6
Feat : 링크 생성 시 폴더 선택 모달창 수정 #60
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
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.
ModalContainer내부에 있던 버튼 컴포넌트를 children으로 받도록 하면서 버튼 관련 prop 제거했습니다
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.
기본 값을 수정했습니다 (단순 참고용)
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.
상태관리를 용이하게 하기위해 버튼 컴포넌트를 ModalContainer에서 분리했습니다
components/modal/AddFolderModal.tsx
Outdated
| type="button" | ||
| onClick={handleSubmit} | ||
| width="w-full" | ||
| height="h-[51px] " |
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.
사소한거긴 한데 height="h-[51px] " 에 공백이 있습니당! tailwind css 는 공백이 있을 경우 안먹을 수도 있다네욧
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.
오 감사합니다 수정해써요~!
components/modal/DeleteLinkModal.tsx
Outdated
| buttonColor="negative" | ||
| ></ModalContainer> | ||
| <ModalContainer title="링크 삭제" subtitle={link}> | ||
| {" "} |
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.
불필요한 코드일까요? ㅎㅎ
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.
헛 그렇네요 삭제했습니다 ㅎㅎ
99minji
left a comment
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.
고생하셨습니다 ㅎㅎ
…orary/linkbrary into feature/modalAction
junjeeong
left a comment
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.
고생하셨어요!
| @@ -0,0 +1,47 @@ | |||
| import { bindCls } from "@/lib/utils"; | |||
| import { FolderItemType } from "@/types/modalTypes"; | |||
| import { FaCheck } from "react-icons/fa6"; | |||
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.
변수명은 줄이지 않고 FaviconCheck 라고 하는게 전 더 직관적인 것 같아요~
다른 애들도 안 줄였으니 통일성도 좋고요!
위에도 마찬가지로 bindClass 이런식으로요!
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.
FaCheck 은 react-icon에서 지원하는 svg 컴포넌트 입니다!
cls 는 다음번에 참고하겠습니다
| linkCount: 1, | ||
| }, | ||
| ], | ||
| link: "https://example1.com", |
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.
시험용 mock데이터인가요?
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.
넵! api test를 위해 전달한 값입니다
venise5224
left a comment
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.
버튼재사용 좋아요👍
작업 내용
추가 PR 작업 내용
이미지가 많이 크네요 😅


