Hotfix: modify alert modal state to recoil and fix the modal style #112
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.
개요
공용으로 뺀 모달 스타일과 state관리 변경했습니다!
recoil로 바꾼 이유
-> 저는 모달 내용 바꿀 일이 많아서 useState 썼는데 컴포넌트가 다르다보니 전역으로 관리가 필요했습니다.
3줄요약
주요 내용
1. 모달 state 변경 ->모달 오픈, 모달 컨텐트까지 모두 recoil로 변경
2. Bottom Slide recoil로 관리하기
저는 모달 오픈 하는 곳과 바텀슬라이드 있는 곳이 달라서 recoil로 관리했어요
같은 컴포넌트에 있으시면 상관없을지도 모릅니다
3-1. 후보 select 기능 추가
3-2. 상태 별 분기처리
3-3. 후보 삭제 마크업 완료
7. AlertModal 취소 부분 텍스트 변경
기본 취소니까 텍스트 바뀌는 분들만 넣으세용
고민한 점, 질문거리