search

Vernoxvernax / Puddler-RS

Jellyfin & Emby & Plex command line client, powered by mpv. Written in Rust.
MIT License
14 stars 2 forks source link

Sane limit to Continue Watching #6

Closed calmcacil closed 1 year ago

calmcacil commented 1 year ago

(Are you tired of me yet?)

Introduce a sane limit to Continue Watching fetch, &Limit=20 or a config definable value. For various reasons, my continue watching history is 135 rows long so only fetching the latest 20 should be an acceptable default but letting people adjust it to their needs could be useful.

Vernoxvernax commented 1 year ago

In general I don't have anything against a user-defined limit, but providing this only for Continue-Watching doesn't make a lot of sense, however implementing another setting for Latest would also be too much bloat. In any case. Anyone who doesn't finish their media one by one, should be punished anyways, so I think a limit of 10-15 should be fine. Anything bigger would be like choosing media from a multi-page catalog; just use the search-function.

calmcacil commented 1 year ago

Agreed! Imagine my horror when i pressed enter to test the fixed continue watching to be welcomed by 158 choices!

Vernoxvernax commented 1 year ago

Btw. These issue are really good qol things. Keep them coming!! Thank you

calmcacil commented 1 year ago

If i think of something I think would be an useful, but some of the things i had in mind arent really suitable, or things I dont think emby's api allows us to do much with.