-
-
Notifications
You must be signed in to change notification settings - Fork 31
[trello.com/c/iGILackD] many chat updates #797
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
vovameister
commented
Apr 10, 2025
- return scrolling to center using buttons and chat entering besides scroll to "new messages" separator
- update hightlight animation, hope I also fix that sometimes its not working or removing to early
- add short vibro while highliting
- remove reading messages while autoscrolling (scroll to item, not scroll to bottom)
- also remove button animation while autoscrolling
- put position restore back so it should be earlier to not reproduce chicken animation
- return animate = true as it was before for autoscrolling
…oll to center by buttons Remove Chicken animation
b1c22cf to
40833e2
Compare
Lainaaa
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.
Consider my comments, I think it becomes more complex and comples, lets make it easier!
Lainaaa
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.
Consider moving some of the viewModels variables to the new state
Also, I don't like approach to combine dialog and vibro, instead - use vibro directly if needs
Lainaaa
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.
Also consider moving state to the viewModel
1345214 to
818a20e
Compare
818a20e to
3c3a7f2
Compare