Blazored / Gitter

A Blazor Gitter Client
MIT License
37 stars 14 forks source link

Auto-scroll messages when list is "at the end/most recent message" #83

Open SQL-MisterMagoo opened 4 years ago

SQL-MisterMagoo commented 4 years ago

I recently disabled the existing JS call because in use it was annoying - I can't remember what I didn't like, but it just didn't work well.

Maybe the first step would be to put the code back and test it out to see if it was just me !

https://github.com/Blazored/Gitter/blob/e78754fd22d78918513c772feea2810b2cb4a19f/src/Blazor.Gitter.Core/Components/Shared/RoomMessages.razor.cs#L176-L197

chucker commented 4 years ago

I'd like to keep it and am not sure in my entirely unscientific test about its annoyances, but autoscroll can, of course, be disruptive.

If we do remove this, maybe we should instead have a panel at the bottom that says "New messages available" or similar, and when clicked, scrolls to the bottom. (I suppose we could have that panel either way as soon as 1) new messages come in and 2) you're not scrolled to the bottom?)