♻️ Refactor(#121): 기본 모달 리팩토링

un0211 commented 2 months ago

연관된 이슈

전반적인 모달 리팩토링을 할 예정입니다.

[x] 확인, 취소/확인만 있는 간단한 모달 통합
- notificationModal: 설명과 확인 버튼만 존재. 특별한 작업 원할 시 onClick 선택적으로
- confirmModal: 삭제전 확인하는 모달. 삭제와 확인 버튼 존재. 삭제작업은 onActionClick 필수
- signup/siginin 등 이미 사용하고 있는 부분 수정했습니다.
이외 모달 리팩토링
- [x] newColumn 모달, ColumnsSection 수정
이외 모달 추후 작업
- [ ] modifyColumn
- [ ] inviteMember
- [ ] newDashboard
[ ] ModalProps 타입 단언 방어적 코드 작성

위에 두개는 모달을 그냥 그려놓은 거고, 아래는 버튼입니다.

아직 작업 완료하지 않았는데 큰 작업이 될 것 같아서 중간중간 확인하기 좋게 PR 만들었습니다. 버튼 타입 지정하면서 이전에 누락된 버튼 타입과 ProfileIcon 변경 반영 커밋을 끼워넣었습니다..

oceanlee-seoul commented 2 months ago

혜원님 ㅠㅠ 제가 얼기설기 엮은 모달 잘 부탁드립니다..! 감사합니다. 저희 나중에 모달 팝업될 때 간단한 애니메이션도 넣어볼까요?!

un0211 commented 2 months ago

저희 나중에 모달 팝업될 때 간단한 애니메이션도 넣어볼까요?!

좋아요! 저희 그럼 애니메이션 이슈 만들고 추가하면 좋을 만한거 하나씩 넣을까요??

그리고 모달 관련해서 의견 듣고 작업하고 싶은 게 있어서 내일 오전에 저희끼리만이라도 짧게 미팅 할 수 있을까요??

oceanlee-seoul commented 2 months ago

저희 나중에 모달 팝업될 때 간단한 애니메이션도 넣어볼까요?!

좋아요! 저희 그럼 애니메이션 이슈 만들고 추가하면 좋을 만한거 하나씩 넣을까요??

그리고 모달 관련해서 의견 듣고 작업하고 싶은 게 있어서 내일 오전에 저희끼리만이라도 짧게 미팅 할 수 있을까요??

네 저는 괜찮습니다!! 참여할 분 참여할 수 있도록 디스코드에 시간 공지해주시면 좋을 것 같아요

un0211 commented 2 months ago

모달 하나마다 PR 만드는 게 리뷰하기 편하실까요? 커밋은 하나단위로 하려고 하는데, 일일이 눌러서 확인하기 귀찮으시면 PR도 하나단위로 할까 해서요~ 이거 머지한 뒤에 하나씩 하고, 지윤님 대양님도 다른 모달 작업하면 어떨까요??

wayandway commented 2 months ago

모달 하나마다 PR 만드는 게 리뷰하기 편하실까요?

커밋은 하나단위로 하려고 하는데, 일일이 눌러서 확인하기 귀찮으시면 PR도 하나단위로 할까 해서요~

이거 머지한 뒤에 하나씩 하고, 지윤님 대양님도 다른 모달 작업하면 어떨까요??

맞아요 모달 하나마다 PR 만드는 게 좋을 것 같습니다!

혜원님 모달 리팩토링 작업해주시고 넘 고생하셨어요🥹 감사합니다🙇