Feat: 마이페이지 구현 #36
Feat: 마이페이지 구현 #36
Conversation
auto-assign
bot
requested review from
Eugene-A-01,
kanglocal,
Nahyun-Kang and
ParkSohyunee
|
@seoyoung-min 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. |
| const { ref } = useOnClickOutside(() => { | ||
| handleSetOff(); | ||
| }); |
There was a problem hiding this comment.
서영님, 혹시 handleSetOff를 호출하지 않고 바로 보내주면 함수가 제대로 동작하지 않나요??
handleSetOff가 () => void 타입인 것 같아서 바로 보내줘도 가능한지 궁금하여 여쭤봅니다!!
const { ref } = useOnClickOutside(() => {
handleSetOff;
});
There was a problem hiding this comment.
헉! 바로 보내주는거 완전 가능인데, 습관적으로 저렇게 했나봐요🥹
좋은 피드백 남겨주셔서 감사합니다!! 🙇♀️👍
| className={`${styles.listDiv} ${language === 'ko' && styles.selected}`} | ||
| onClick={() => { | ||
| setLanguage('ko'); | ||
| handleSetOff(); | ||
| }} | ||
| > | ||
| 한국어 | ||
| </div> | ||
| <div | ||
| className={`${styles.listDiv} ${language === 'en' && styles.selected}`} | ||
| onClick={() => { | ||
| setLanguage('en'); | ||
| handleSetOff(); | ||
| }} |
There was a problem hiding this comment.
onClick 안에 있는 함수는 따로 분리해도 좋을 것 같다는 생각이 드는데, 서영님 생각은 어떠신가요?!
분리한다면, 하나의 함수를 만들어서 동일하게 사용하는 것이 좋을 것 같고, 아직 언어변경 기능이 완성되기 전이어서 추후 해당 로직이 더 복잡해질 가능성을 고려했을 때 분리하는 것이 괜찮을 것 같아서 의견드립니다!!
There was a problem hiding this comment.
소현님, 분리하는게 훨씬 좋아 보입니다! 추후 로직까지 고려하면 더더욱 그렇네요!👍👍
감사합니다🩵🦋
src/components/Header/Header.tsx
Outdated
There was a problem hiding this comment.
버튼 타입을 주지 않으면 form 이 제출되어버리는군요..!?👀
There was a problem hiding this comment.
버튼 type속성의 기본값이 'submit'이라,
form태그 안에 있는 버튼은 다 제출기능도 해버리더라구요!!🥲
개요
작업 사항
참고 사항 (optional)
스크린샷
리뷰어에게