j-fbriere / squawker

An open-source privacy oriented Twitter/X client
MIT License
801 stars 16 forks source link

Reloading causes feed position to be lost in recent version #266

Open sephamorr opened 3 months ago

sephamorr commented 3 months ago

Checklist

Affected version

3.7.8

Steps to reproduce the bug

  1. Enable "keep feed offset" switch in settings->feed
  2. Open feed tab
  3. Click reload

Expected behavior

The same tweets as before the reload should be visible.

Actual behavior

It appears that the offset with respect to the top of the feed is maintained: -if you're at the top of the feed before refresh, you'll be kept at the top of the feed -if you're some distance below, the number of tweets in the icon on the top right remains approximately similar.

Screenshots/Screen recordings

No response

Logs

No response

Affected Android/Custom ROM version

MIUI 14.0.23.7.3

Affected device model

Mi11Ultra

Additional information

This behavior worked well in 3.7.2.

doug8888 commented 3 months ago

I can confirm that the position in the feed is lost after a refresh. Worked fine in earlier versions.

j-fbriere commented 3 months ago

I'm not sure I understand the issue. Please give me more details about your issue with clear detailed steps, screens snapshots and what is the problem.

doug8888 commented 3 months ago

I looked at Squawker about 10 minutes ago. Looked at it again just now and saw the last post I'd read earlier. Refreshed the feed and it jumps to the top of the feed (latest post) so I had to scroll down 3 or 4 posts to continue reading from where I'd left off 10 minutes ago. Ideally it would maintain it's position in the feed at the post I had reached 10 minutes ago. This is how it used to work until an update a while back.

After refresh:

Screenshot_20240327-224340_Squawker

Before refresh:

Screenshot_20240327-224314_Squawker

BJ24 commented 3 months ago

Same issue noticed for me too.

Saroumane commented 1 month ago

Using v3.8.2+300015943. It used to work, but since a few versions it's indeed broken (when you manually refresh the feed.)

There's still a case where the offset (the position in the feed) is kept : quit the app, restart the app : the position is kept (even if the app loads new tweets).