ltguillaume / droidshows

A Reboot of DroidSeries Offline TV Shows Tracker
https://codeberg.org/ltguillaume/droidshows
GNU General Public License v3.0
90 stars 19 forks source link

Sort by date of next episode #106

Open Scumpeter opened 2 years ago

Scumpeter commented 2 years ago

When sorting by unseen, the shows should show the shows sorted by the date of the next unseen episode (the same value that is used for the "next episode" field in the overview). Instead the shows are seemingly in random order: Screenshot_20211226-155554

Tested with 7.11.2 from Fdroid on Android 12.

ltguillaume commented 2 years ago

Sort by unseen episodes == sort by amount of unseen episodes. It's meant to help you decide which shows have enough episodes "available" for a binge night 🙈 I get the confusion, though, I even seem to have described the old behavior of the feature in readme, will fix that.

Nice list by the way 👍

Scumpeter commented 2 years ago

Ah. Thanks for the clarification. Should I create a feature request for an additional sorting order? It would really help me with watching Stargate in airing order 😉

ltguillaume commented 2 years ago

Ah you mean due to cross-overs?

Scumpeter commented 2 years ago

For example. Also to understand the possible references in one show to another show.

ltguillaume commented 2 years ago

Yeah, I'll have to restructure the database for that, but I'll look into it.

ltguillaume commented 2 years ago

I suppose this is sorted (see what I did there? 😉) in @warren-bank's build, right?

warren-bank commented 2 years ago

actually.. I didn't do what @Scumpeter has requested..

when a user has several unseen episodes.. there's a first (oldest) and last (newest).

this request (I believe) is to sort all shows by the date (ascending) of each first (oldest) episode.

what I implemented in my fork was an option to sort all shows by the date (descending) of each last (newest) episode.

to be perfectly honest, I don't see any value in sorting by oldest. the reason that I added the option to sort by newest is to see which shows have most recently aired new (unseen) episodes.

ltguillaume commented 2 years ago

Whoops, I knew I had to double-check this before posting something about it 😛

warren-bank commented 2 years ago

no worries..

strictly speaking, adding another sort option would be trivially easy; now that there's a dialog that presents options.. adding one more option wouldn't be a problem. On the one-hand, we want to avoid visual clutter. On the other hand, is this a feature that could be useful? If so.. I could certainly add it.

ltguillaume commented 2 years ago

I personally didn't really see the use, but I guess @Scumpeter could certainly still make their case again.