Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
prgmr99
left a comment
prgmr99
left a comment
There was a problem hiding this comment.
LGTM!! 고생하셨습니다~ 👍👍
몇가지 사소한 제안사항만 있습니다..!
| const [isDropdownOpen, setIsDropdownOpen] = useState(false); | ||
| const [isAccountSettingsModalOpen, setIsAccountSettingsModalOpen] = useState(false); | ||
| const [isFeedbackModalOpen, setIsFeedbackModalOpen] = useState(false); | ||
| const [isLogoutModalOpen, setIsLogoutModalOpen] = useState(false); |
There was a problem hiding this comment.
뭔가 이런 식으로 커스텀훅을 만들고 활용해도 좋을 것 같다는 생각이 들어서 공유해보았습니다..!
// 커스텀훅 정의
const useToggle = (defaultValue = false) => {
const [isOpen, setIsOpen] = useState(defaultValue);
const open = useCallback(() => {
setIsOpen(true);
}, []);
const close = useCallback(() => {
setIsOpen(false);
}, []);
const toggle = useCallback(() => {
setIsOpen(!isOpen);
}, [isOpen]);
return { isOpen, open, close, toggle };
};
// 사용 예시
const {
isOpen: isUnfollowDialogOpen,
open: openUnfollowDialog,
close: closeUnfollowDialog,
} = useToggle();
| </header> | ||
|
|
||
| {/* 콘텐츠 */} | ||
| <div |
There was a problem hiding this comment.
이건 사소한 부분일수도 있지만,,
콘텐츠 부분은 main 태그로 해도 좋을 것 같습니다!!
header와 footer 태그를 잘 사용해주셔서 공유드려봅니다..! 👍
|
|
||
| // 외부 클릭시 드롭다운 닫기 | ||
| useEffect(() => { | ||
| const handleClickOutside = (event: MouseEvent) => { |
There was a problem hiding this comment.
추후에 제가 만든 useClickOutside 훅 사용하셔도 좋을 것 같아요!
klmhyeonwoo
left a comment
klmhyeonwoo
left a comment
There was a problem hiding this comment.
고생하셨습니다!
토글 메뉴 컴포넌트는 제가 호다닥 수정한 후에 반영해보겠습니다!
코드 확인 후에 현우님 코드에 적용하는게 어렵지 않으시다면 반영해주시면 좋을 것 같아요 :-)
|
공통화 처리 해주신 토글메뉴 컴포넌트는 제가 주말 지나고 리팩토링 반영하겠습니다!! 일단 급한 불인 api 연결 먼저 작업올렸습니다! 해당 작업에서 작은 한가지 이슈가 있었습니다! 아래 두 파일에서 ic_close.svg가 있고 export가 잘 되어 있음에도 해당 컴포넌트를 사용하게 되면 에러가 발생했고, 원인을 해결하지 못해서,,,, 임시로 로 적용해 두었습니다. 혹시 원인을 아시는분이 계실까요..? @klmhyeonwoo @JaeIn1 @prgmr99 |
현우님..! 한 파일에서 동일한 이름으로 export해서 발생하는 문제인거 같습니다. |
승준님 최고!! 감사합니다 ㅠㅠ 이걸 못봤네요!! |
3 participants
특이사항
🏄🏼♂️ Summary (요약)
프로필 - 계정설정 탭 퍼블리싱
프로필 - 평가 및 피드백 모달 작업
프로필 - 로그아웃 모달 작업
📚 Reference (참조)