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.

Sign up for GitHub

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

Jump to bottom

[Refactoring] modal 구조 리팩토링 (ModalViewer 제거) #121

Open

minsour opened this issue Mar 6, 2022 · 1 comment

Assignees

Labels

Member

minsour commented Mar 6, 2022

ModalViewer이 필요하지 않다고 판단
- 기존에 전역에 여러 modal에 대한 상태를 두고 전역에서 ModalViewer가 띄워줄 modal이 있다면 상태를 가져와서 띄워주는 식이었음.
  하지만, portal을 생성해서 모달을 띄우는 방식이라 전역에 ModalViewer가 있든 없든 방식은 변하지 않고,
  오히려 관리 측면에서 modal이 필요한 컴포넌트 쪽에서 관리하는게 더 관리 측면에서 더 좋을것이라 판단
modal 구조 리팩토링 (ModalViewer 제거)

minsour added the refactor label

dididy self-assigned this

Member

dididy commented Mar 6, 2022

2차 qa 이후 적용 예정입니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment