TwidereProject / Twidere-Android

http://twidere.com
2.75k stars 377 forks source link

Twidere looses position in the stream when more than 100 tweets are loaded #760

Open Torvin opened 7 years ago

Torvin commented 7 years ago

Steps to reproduce

Load more than one portion of tweets (I think the first portion is 100 but I might be wrong) by scrolling down. Then rotate the phone a couple of times

Expected behavior

Twidere preserves the position

Actual behavior

After a couple of rotations Twidere looses the position and discards all the tweets that are not the in the first portion and scrolls to the last tweet in the first portion.

Extra info

Android version: 6.0.1 (CM) App version: 3.5.13 Build variant: Google Micro-blogging service: Twitter

App settings:

mariotaku commented 7 years ago

It's an intended behavior actually, Twidere will limit its data to a user set maximum, e.g 300, this will be performed each time Twidere closed. I improved to make it perform only when Twidere quits, rather than rotating screen.

Torvin commented 7 years ago

Oh, I didn't even know there was a setting for that. Thanks!

Is there also a way of persisting unread tweets no matter what? It's really frustrating to loose the position when Twidere gets unloaded from memory for any reason.

mariotaku commented 7 years ago

This will get better after d9068de. Old tweets get removed only when you quits Twidere. This is essential to prevent performance issues.