-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FR] Implement a "Feed" of downloaded videos #504
Comments
I dig it! This would fit in well with #478 I see you're working on something in your fork - I'd maybe recommend to hold off for a bit because I'd like to look into a bit of a refactor for how I'm handling tables in general. I'll let you know when/if I end up making those changes! |
I first didn’t quite understand what was meant with #478 but now i see what you are going for. I saw that there’s a bit of duplicate code but I think it’s OKish at the moment for what it is. When touching this more broadly I would also recommend moving away from “real” html tables as they are kind of a pain on mobile. Rather design some responsive layout that folds out and expands on desktop. In the long run it might be an idea having a general media item view which can be initialized with different filters and sorting options. But I think that’s a bit out of scope here. If I’m motivated enough, i might look into it. |
This is, ideally, the end goal of my refactor. A generalized system for tables where you just have to provide methods for how to fetch/sort/filter data and it does the rest. I'm not sure whether I'll be able to fully realize that, but I'm sure I can make a better system than what currently exists |
It would be nice to have a dedicated view that displays basically the same as the Media History but filtering out not downloaded videos.
This could then be used as a feed to watch videos directly in Pinchflat. I think the video detail view is already quite nice and it is a shame that it is a bit hidden away.
The text was updated successfully, but these errors were encountered: