-
-
Notifications
You must be signed in to change notification settings - Fork 166
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
fix: chat "scrolling up" upon reaction #4120
Conversation
6005ac7
to
92dca6e
Compare
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.
Works as expected! Needs a new rebase now with monorepo in main
I was thinking of putting this into version |
Yes but it's easy to apply it to the new package/frontend source. So hopefully not too much work :-) |
I tested it by patching main since I have main already updated |
we can still back port it afterwards if the next release takes too long. I say rebase it and merge it to main now and write it into your notes for back porting, if we need to do that because the new release has delays. |
fbaa7e6
to
6d3b21c
Compare
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.
code LGTM
6d3b21c
to
78d23f6
Compare
...resulting in new messages not getting scrolled into view when they arrive. This simply removes height changes between messages with / without a reaction. Closes #3753
78d23f6
to
cf84e72
Compare
...resulting in new messages not getting scrolled into view when they arrive.
This simply removes height changes between messages with / without a reaction.
Closes #3753