Merged
Conversation
포크한 레포에 작동 테스틀 위해 연결 스토리북 형태도 세팅함
구조 수정 및 tailwind prefix에 맞게 mapping
이전 패키지 버전에선 빌드가 안 됐음.
모달 공통 컴포넌트와 파생 컴포넌트 제작. 공통에서 종속적인 부분이 있어 합성 컴포넌트로 제작. 모달 전역 관리를 위해 zustand와 커스텀 훅을 사용. svgr관련 세팅 대해 브랜치 꼬여서 임시 저장.
일단, 외부로 동작의 책임을 넘김. 기타 변경사항들은 코드 다듬기.
전체 border대해 round가 아니도록. 길이는 외부의 container에서 처리할 수 있도록.
useModal확인 용으로 사용했었어서, 기본 값으로 복원했습니다.
폴더 깔끔하게 수정. 스토리북에서 설명 및 사이드바 경로, docs에서 잘 보이도록 render추가. 모달 공통 타입 d.ts파일로 변경.
cjs, mjs둘다 사용되고 있어서 ts를 ts-node로 실행시키면 에러가 발생했습니다.
이전에 실수로 삭제했었던 것으로 보임.
os차이 때문인지 린트에서 에러나서 재설치 했습니다.
devgony
requested changes
Jun 26, 2025
| <div className="w-96 px-6 py-4 bg-indigo-500 inline-flex justify-start items-center gap-3"> | ||
| <div className="w-full px-6 py-4 bg-indigo-500 inline-flex justify-start items-center gap-3"> | ||
| <div className="min-w-0 flex-1 inline-flex flex-col justify-center items-start gap-2"> | ||
| <p className="overflow-x-hidden self-stretch justify-center text-Color-white text-base font-bold font-['SUIT_Variable'] leading-tight"> |
Collaborator
There was a problem hiding this comment.
Suggested change
| <p className="overflow-x-hidden self-stretch justify-center text-Color-white text-base font-bold font-['SUIT_Variable'] leading-tight"> | |
| <p className="overflow-x-hidden self-stretch justify-center text-Color-white text-base font-bold font-SUIT_Variable leading-tight"> |
font-SUIT_Variable 으로만 써도 인식 되는 것 같습니다. 다른 쪽두요!
Collaborator
Author
There was a problem hiding this comment.
아 넵 맞습니답.
figma에서 코드 복사하면 font-['SUIT_Varaible'] 로 되길래,
수정하기 싫어서 이대로 적용 되도록 설정해서 크게 문제는 없을 것 같긴 합니다
devgony
approved these changes
Jun 26, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Modal 추가 & 기존 공용 컴포넌트수정
Modal
Modal은 합성 컴포넌트로 만들었으며, 일단 Modal.button만 재료로 사용했습니다.
figma의 각 모달은 이를 기반으로 제작하였고, 이벤트 핸들링은 외부에서 책임지도록 했습니다.
기존 공용 컴포넌트