-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat-fe: Date Picker 이벤트 발생 컴포넌트 변경 #442
Conversation
1723612961.027869 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굳입니당
<S.Container> | ||
<S.Container onClick={handleIconClick}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
버블링 이용 SMART 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
러기 고생하셨습니다! 한 가지 코멘트를 드렸는데 시간 되실 때 체크해주세요 🙏
@@ -18,7 +18,7 @@ export default function DateInput({ label, innerText, ...props }: DateInputProps | |||
}; | |||
|
|||
return ( | |||
<S.Container> | |||
<S.Container onClick={handleIconClick}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 이벤트 버블링을 사용하셨군요! 멋집니다 👍
outline={false} | ||
size="sm" | ||
color="white" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
button
요소의 type
를 button
으로 지정해주면 의도치 않은 이벤트 발생으로부터 조금 더 안전하지 않을까 싶습니다.
outline={false} | |
size="sm" | |
color="white" | |
outline={false} | |
size="sm" | |
color="white" | |
type="button" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
중요한 부분 짚어주셨네용!! 감사합니다~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빠른 피드백 반영에 감사드려요! 🎊
@@ -7,7 +7,7 @@ export default function IconButton({ | |||
size = 'sm', | |||
outline = true, | |||
shape = 'round', | |||
type, | |||
type = 'button', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
멋집니다 👍 👍 👍
Co-authored-by: Jeongwoo Park <[email protected]>
Original issue description
목적
작업 세부사항
참고 사항
CHANGE_DATE_EVENT_TRIGGER
closes #441