-
Notifications
You must be signed in to change notification settings - Fork 12
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
need an additional scrollToBottom on incoming messages #2
Comments
Thanks for opening this issue! AirMessage for web doesn't call screen.movIs this not the behaviour you're experiencing? |
Hi @tagavari! You're very welcome, and thanks for the reply! I think I need to clarify my original post: I do experience the behavior you showed in your movie most of the time, but occasionally not. Sometimes -- perhaps when the airmessage tab has been out of focus, but I'm still confirming -- received messages, along with the first outbound message, will fail to scroll to the bottom as expected. I'm not able to recreate it at the moment, but I will try to create a video of it once I can show how to reproduce. EDIT: Here's a demonstration of the issue with incoming messages. It can also occur with outbound messages sent very quickly (before the server has time to update it would seem). Notice in the clip, I'm receiving inbound messages (three in this case), and the scrollbar is moving, but the initial message is cut off, along with all subsequent ones. To create the situation, I switched between two conversations and also backgrounded the tab briefly (I'm not yet 100% sure what causes it, but I encounter it pretty frequently). demo.mp4 |
i fixed this issue here: Geczy@c8b26ec feel free to cherry pick ! |
@Geczy Thanks a lot! I tried this out, but it seems to trigger even if I've scrolled far up in the conversation. Am I missing something? |
can you share the branch you tried this in? nvm i can see this happening in mine too, will think of a fix |
okay this commit fixes it for sure Geczy@5856eb0 |
Thank you both for working on this! |
Thank you @Geczy, I will try this out later this week! |
hi, I've tried that function, but images are downloaded again-and-again every time I click on the chat. The loading takes 8-10 seconds, and sometimes I get 'Couldn't load this conversation'. |
Thanks for putting this together -- it's awesome!
One annoyance I've noticed: When sending messages, the scrollToBottom calls successfully scroll the div containing messages to the bottom. However, this is not true when you're in an existing chat and receive messages from the other party. For instance:
It would be great if incoming messages would trigger a scrollToBottom event as well. I haven't had the time to dig into the code to put together a PR yet, but I imagine this is a relatively simple fix.
The text was updated successfully, but these errors were encountered: