-
Notifications
You must be signed in to change notification settings - Fork 1
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: 일기 작성 기능을 구현합니다. #70
Conversation
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.
👍
|
||
const handleTextAreaChange = (text: string) => { | ||
setValue('diary', text); | ||
}; |
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.
상태 변경에 debounce 넣는 부분 동의합니다 👍
setValue에 대한 상태값이 너무 많아서.. iOS 키보드로 입력이 잘 될지 모르겠어요!
요 부분 모웹에서의 테스트가 필요할 것 같습니다.
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.
추가로 watch는 상태가 변경되는 부분을 계속 감시하고 있기 때문에 성능에 안좋을 수 있습니다.
그 수가 많아질수록 성능 향상을 위해 해줘야 하는 작업이 늘어날 것 같아요.
추후 상태 객체 형태로 변경 + register을 활용하여 watch 메소드를 활용하지 않도록 구현해야 할 것 같습니다.
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.
추후 input 태그에 register 부여를 위해 text-field 컴포넌트 수정 PR 올리도록 하겠습니다!!
🤔 어떤 문제가 발생했나요?
🎉 어떻게 해결했나요?
📷 이미지 첨부 (Option)
default.mp4