Fix: 댓글 폼 TextArea로 수정, 댓글/답글 관련 버그 수정 #210
Conversation
auto-assign
bot
requested review from
Eugene-A-01,
kanglocal,
ParkSohyunee and
seoyoung-min
|
@Nahyun-Kang is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
| export const useCommentStatus = create<useCommentStatusType>((set) => ({ | ||
| status: null, | ||
| setStatusEdit: () => set({ status: 'edit' }), | ||
| setStatusCreateReply: () => set({ status: 'createReply' }), | ||
| resetStatus: () => set({ status: null }), | ||
| })); |
There was a problem hiding this comment.
오! 나현님, 댓글 상태를 전역으로 관리하신 이유가 궁금합니당🧐
컴포넌트간 props를 많이 내려줘야 하거나 초기화 등의 이유가 있었던 걸까요??
There was a problem hiding this comment.
네 소현님..! 아무래도 commentForm 컴포넌트에서 댓글/답글 수정, 댓글/답글 생성 등 많은 부분을 관리하는데 CommentForm이 컴포넌트로 따로 분리되어 있는 문제와, Reply까지 내려가는데 Prop 드릴링을 피할 수가 없어서 전역상태로 관리하는 것을 선택했습니다..!👀
| const handleSubmit = (e: React.FormEvent<HTMLFormElement>) => { | ||
| const handleSubmit = (e: React.FormEvent<HTMLFormElement> | React.KeyboardEvent<HTMLTextAreaElement>) => { | ||
| e.preventDefault(); |
There was a problem hiding this comment.
명확한 타입지정 너무 좋네요!! 👍👍
+. 참고로 한 컴포넌트에 React로 바로 임포트 해온 타입이 있고, 아닌 경우도 있어서 나중에 컨벤션을 통일하는 것도 좋을 것 같다는 생각이 들었습니당🥰
| backgroundColor: vars.color.gray3, | ||
| '::-webkit-scrollbar': { | ||
| display: 'none', | ||
| }, |
There was a problem hiding this comment.
오! textarea에서 스크롤이 보이지 않게 하는 속성인가요? 🎨
There was a problem hiding this comment.
오 네네..!! overflow scroll에서 hidden으로 바꾸면서 필요없는 코드가 되어버렸네요🤔 삭제해야하는 코드인데 발견해주셔서 감사합니다!! 🥹
|
|
||
| const handleKeyDown = (e: React.KeyboardEvent<HTMLTextAreaElement>) => { | ||
| if (e.key === 'Enter' && !e.shiftKey && !isPending) { | ||
| // handleResizeHeight(); | ||
| handleResetTextArea(); | ||
| e.preventDefault(); | ||
| handleSubmit(e); | ||
| } |
| interface IProps { | ||
| handleSubmit: (e: React.FormEvent<HTMLFormElement> | React.KeyboardEvent<HTMLTextAreaElement>) => void; | ||
| isSubmitting: boolean; | ||
| } | ||
|
|
||
| function usePressEnterFetch({ handleSubmit, isSubmitting }: IProps) { |
There was a problem hiding this comment.
나현님 혹시 props 타입이름을 함수명+props로 변경하면 어떨까용??
There was a problem hiding this comment.
앗 코드 고대로 복붙해온 거 깜빡했어용..머쓱 ㅎㅎㅎ 타입명 수정해두겠습니당!!👍 감사합니다 소현님 🙇♀️
* Fix: 답글 캐싱 문제 해결 * Fix: 답글 생성, 수정 관련 로직 수정 * Fix: 댓글 form input에서 textarea로 교체 * Fix: 댓글 작성 이후 자동으로 textarea 크기 바뀌게 수정
* Chore: 환경설정 커밋 * Fix: 댓글 폼 TextArea로 수정, 댓글/답글 관련 버그 수정 (#210) * Fix: 답글 캐싱 문제 해결 * Fix: 답글 생성, 수정 관련 로직 수정 * Fix: 댓글 form input에서 textarea로 교체 * Fix: 댓글 작성 이후 자동으로 textarea 크기 바뀌게 수정 * Fix: intro 페이지 이미지 교체, 메이커스 이름 추가 (임시) (#213) * Feat: 인트로 페이지 정적인 이미지 동영상으로 교체 * Feat: 인트로 페이지 이미지 교체 * Fix: 인트로 페이지 코드리뷰 반영 * Fix: 탐색 페이지 트렌딩 리스트 사파리에서 타이틀의 webkit 속성이 적용되지 않는 문제 해결
* Chore: 환경설정 커밋 * Fix: 댓글 폼 TextArea로 수정, 댓글/답글 관련 버그 수정 (#210) * Fix: 답글 캐싱 문제 해결 * Fix: 답글 생성, 수정 관련 로직 수정 * Fix: 댓글 form input에서 textarea로 교체 * Fix: 댓글 작성 이후 자동으로 textarea 크기 바뀌게 수정 * Fix: intro 페이지 이미지 교체, 메이커스 이름 추가 (임시) (#213) * Feat: 인트로 페이지 정적인 이미지 동영상으로 교체 * Feat: 인트로 페이지 이미지 교체 * Fix: 인트로 페이지 코드리뷰 반영 * Fix: 탐색 페이지 트렌딩 리스트 사파리에서 타이틀의 webkit 속성이 적용되지 않는 문제 해결
개요
작업 사항
참고 사항 (optional)
관련 이슈 (optional)
closed: #196
closed: #208
closed: #209
스크린샷
댓글 textarea로 수정, 댓글수정/ 답글 달기 동시에 되는 문제 해결
-.1.mp4
답글 수정 반영 안되는 버그 해결
-.2.mp4
리뷰어에게