this post was submitted on 05 Nov 2024
13 points (100.0% liked)

Summit

741 readers
19 users here now

Community to discuss Summit, a Lemmy reader for Android.

App (Play Store): https://play.google.com/store/apps/details?id=com.idunnololz.summit

APK: https://github.com/idunnololz/summit-for-lemmy/releases

Patreon: https://patreon.com/SummitforLemmy

Ko-Fi: https://ko-fi.com/summitforlemmy

Website: https://summit.idunnololz.com/

founded 1 year ago
MODERATORS
 

Ok..can someone please help me how to make this make read on scroll work? I have it enabled but it just marks some posts read but others not. Meaning it's not consistent. ๐Ÿ˜

Am I wrong or the setting is also on 2 places in the settings?

top 3 comments
sorted by: hot top controversial new old
[โ€“] idunnololz@lemmy.world 2 points 1 week ago* (last edited 1 week ago) (2 children)

I just tested it and it seems to work consistently for me. The way it's designed to work is any post you scroll past (eg. the post has to be scrolled completely off screen) will be marked as read. If you think it should work differently or if this is not what you see happening please let me know.

Also if you could share a video that would also be helpful for me to look into the issue since I can't reproduce it on my end.

Update

I reviewed the code for this and I found a minor flaw however it should still be marking all posts above a certain point as read, it's just the "line" it uses to figure out what to mark can be unpredictable. Eg. an item can be scrolled a off screen and still not marked as read until a certain amount of things have been scrolled on screen. I've updated the logic so it items scrolled off screen are instantly marked as read. This change will go out in the next update.

[โ€“] WeAreAllOne@lemm.ee 2 points 1 week ago

Thank you for taking the time to look into it.

I think you spotted the issue. When I said inconsistent I meant that some posts are marked read but other might not and that I have to scroll up again and down also a bit slower that the initial scroll to make it happen. I'll wait the next release to verify the change you made.

Many thanks!

[โ€“] WeAreAllOne@lemm.ee 2 points 1 week ago

Works like a charm after today's update! Nice one !