search

decent-chat / decent

Open source messaging platform for the modern web
https://meta.decent.chat
GNU General Public License v3.0
21 stars 5 forks source link

Scrollback is somewhat broken #235

Closed towerofnix closed 6 years ago

towerofnix commented 6 years ago

I think the problem has to do with the scroll's Y-position not effectively being set, maybe because a scroll is already in progress... I noticed that it only happens on really, really fast connections, i.e. to localhost, and not on meta.decent.chat. This lines up with my thoughts because, by the time the older messages have been fetched, the scroll animation is finished.

bates64 commented 6 years ago

I think we really need to rethink a lot of the code surrounding the messages list - there's a ton of bugs, and we need to think about scrollforward too.

PullJosh commented 6 years ago

I think we really need to rethink a lot of the code

To be fair, you think this about a lot of things.

PullJosh commented 6 years ago

The clear solution is to add 250ms of artificial delay to all API calls.

bates64 commented 6 years ago

thumbs up from @towerofnix

PullJosh commented 6 years ago

Solved in the preact client.