[기타] 모달 코드 중복 개선 - Githubissues

nailedReact / bokgungom-market

멋쟁이사자처럼 프론트엔드 스쿨 3기 15조 득근득근 복근곰마켓 레포지토리

https://bokgungom-market.vercel.app/

6 stars 5 forks source link

[기타] 모달 코드 중복 개선 #383

Closed SEMINSEMINSEMIN closed 1 year ago

SEMINSEMINSEMIN commented 1 year ago

무엇을 위해 리팩토링 하나요?

[ ] 성능 개선
[x] 접근성 개선
[ ] 코드 중복 개선
[ ] 컨벤션 정리

문제점

모달창 코드가 중복되는게 많음 (ex. PostDetail.jsx)

수정 페이지

[x] PostDetail.jsx

특이사항

PostDetail.jsx에서 post detail id를 가져올 때 useLocation 대신 useParams 이용
pages/feed/post하에 CommentOptionModal.jsx와 PostOptionModal.jsx 추가
PostDetail.jsx에서 댓글 모달과 포스트 모달의 state를 각각 객체로 관리함으로써 관리해야 할 state의 수를 줄임
- ex) const [postModal, setPostModal] = useState({isMe: undefined, isOpen: false});

github-actions[bot] commented 1 year ago

Branch refactor/issue-383 created for issue: [기타] 모달 코드 중복 개선