Conversation
Member
|
@Ivoryeee 감사합니다~~~ 고생많으셨어요🙇♂️🙇♂️🙇♂️
|
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
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.
🔥 Related Issues
✅ 작업 리스트
🔧 작업 내용
기존 ModalWrapper를 제거하고, overlay-kit을 활용한 Overlay 컴포넌트를 새로 정의했습니다.
모달 로직을 간결하게 유지하면서도 재사용성과 확장성을 고려했습니다.
✔️ Overlay-kit 라이브러리 설치
✔️ App.tsx에 OverlayProvider 추가
OverlayProvider는 overlay-kit의 핵심 상태 관리 컨텍스트를 제공하여, 앱 전역에서 모달과 오버레이 컴포넌트의 열림/닫힘 상태를 일관되게 관리하기 위해 추가했습니다. 이를 통해 여러 곳에서 호출되는 모달이 충돌 없이 정상적으로 작동하도록 돕습니다.
✔️ Overlay 컴포넌트
✔️ Overlay 사용법
✔️ ModalWrapper -> Overlay 교체 작업
(기존)
(교체)
해당 pr에 대한 전 과정을 노션에 간략히 정리해 뒀습니다!
https://www.notion.so/Overlay-kit-23893c660098802a9e6cca308a354edf?source=copy_link
🧐 새로 알게된 점
🤔 궁금한 점
📸 스크린샷 / GIF / Link