-
Notifications
You must be signed in to change notification settings - Fork 3
버그 수정 및 채팅 자동 스크롤 및 새 메시지 알림 기능 구현을 통한 UX 개선 #235
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
Conversation
✅ Deploy Preview for jootalkpia ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
handje
left a comment
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.
ui가 훨씬 예뻐졌네요! 👍
조건에 '내가 보내는 메시지' 라면 무조건 하단으로 자동 스크롤 되도록 하는건 어떠신가요?
이건 기획적인 부분이긴한데, 슬랙은 굳이 구분하지 않는 것 같고 카카오톡은 본인이 채팅을 보낼땐 무조건 내려가는 것 같습니다!
슬랙이 레퍼런스지만, 대댓글이 없는 채팅의 특징이 더 강해진 상황에서는 후자(카카오톡)의 기획이 더 사용성이 좋지 않을까하는 의견입니다 :)
src/frontend/apps/web/src/features/chat/model/use-chat-autoscroll.ts
Outdated
Show resolved
Hide resolved
| toast, | ||
| dismiss, | ||
| prevMessageCount, | ||
| ]); |
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.
만약 분리가 가능하다면 useEffect를 분리하여 의존성을 줄이는 것도 좋을 것 같습니다.
예를 들면 scoll 감지 / toast 기능 이런식으로 분리는 어려울까요?
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.
음 근데 제가 알기로 useEffect는 state가 들어가게 되었을 경우 재렌더링이 일어나는 것으로 알고 있습니다.
그런데 저희가 isUserScrollingUp, newMessageCount, prevMessageCount이렇게 3가지 state를 업데이트 하고 있는데 useEffect를 나뉘게 된다면 의존성을 관리하기에는 용이하겠지만 불필요한 재렌더링이 일어날 수 있지않을까요?
src/frontend/apps/web/src/features/chat/model/use-chat-autoscroll.ts
Outdated
Show resolved
Hide resolved
좋은것 같습니다. 근데 지금 구현하기엔 일단 로그인을 통해서 날라오는 데이터와 비교하는게 맞다라고 생각하기때문에 로그인 부분 머지되는데로 추후 기능 추가하겠습니다. |
Pull request
Related issue
Motivation and context
Solution
📌 채팅칠 시 Overflow가 될 경우 TextArea가 밀려나는 버그 개선
📌 화면 스크롤 UX 개선
How has this been tested
2025-02-18.11.32.50.mov