Summit

741 readers

2 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

idunnololz

[Release][v0.1.25] Implementing user requests (self.summit)

submitted 1 year ago* (last edited 1 year ago) by idunnololz to c/summit

19 comments fedilink hide all child comments

The focus of this update was to implement more popular user feature requests.

Changelog

Hide read post
Show posts as read
Support default sort type
Support other Lemmy account settings
Handle case where account token is invalidated
Improve community search to show more relevant results
Add setting to mark as read on scroll
Add hide read posts action to FAB menu
Fixed a bug where tapping “Hide read” will move you to a random position in your feed

Shadow release

There is a "shadow release" v0.1.26 rolling out right now. It doesn't add any features. It just fixes some bugs. That is all.

I'm calling it a shadow release because I'm not making a new post for it

Shadow release 2

Based on the wonderful discussion in https://lemmy.world/post/1350726, I have made some changes to how "Hide read" works. This update should hopefully be available in the play store in ~ 1 hour. We are on v0.1.29 now because I had to make some hot fixes along the way.

you are viewing a single comment's thread
view the rest of the comments

[–] idunnololz 3 points 1 year ago (2 children)

I see. I'm guessing on Reddit, there was no server setting to hide read posts whereas on lemmy there is. This kind of complicates things as there are two ways to implement this feature. One that is 100% client side and the other that is server side.

[–] Alpagu 1 points 1 year ago

I think there is an error in the implementation point. I used sycn for lemmy for a long time. This is something I'm used to. Server setting should be applied to the fab button. The hide read and show read feature in the server setting should be applied to the fab button. There should not be two different systems.

[–] kratoz29 1 points 1 year ago (1 children)

Well, I actually think it is kinda implemented on Reddit, but not as it is with Lemmy web (I don't recall this feature being persistent across different apps... anyway I'm used for this to being client side only, if using the Lemmy implementation helps for the clients here then I guess is fine???

IDK man 😅

Anyway your current implementation is a step up ahead because no other app has it working in conjunction yet.

May I ask why do we need to uncheck that setting in the the lemmy web to begin with?

[–] idunnololz 1 points 1 year ago (2 children)

My implementation uses Lemmy's server side logic to hide read posts. Therefore, if "Show read posts" was enabled, the "Hide read" button would do nothing. To avoid user confusion, the "Hide read" button is only shown when "Show read posts" is disabled. After thinking about this a bit more, I think I'm coming to the conclusion that this group of features should all be client sided. This is going to take some time to implement. I'll work on it tomorrow. I'm also in the middle of implementing gesture actions so this will have to wait until I am done with that.

@[email protected] ^^

[–] Alpagu 2 points 1 year ago

Thank you 👏

[–] kratoz29 1 points 1 year ago

Yeah, I think it is better to keep this client side let better show read posts option keep working globally and have the client to hide read posts with the FAB button.

I think Connect for Lemmy does something like this (no switch though) without messing with Lemmy's settings.

It has two options, one for mark as read while scroll and the other to hide the read posts but hidden posts does not interfiere with other apps.

Anyway I think it is a big progress to have a button on a FAB already, keep on the good job man!