-
Notifications
You must be signed in to change notification settings - Fork 484
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
Add favorites #682
Add favorites #682
Conversation
09aff17
to
2f9485e
Compare
2f9485e
to
afd70b9
Compare
…into add-favorites
They can not be used on TV so no reason to show them in library
This adds favorites to the top bar somewhat based on how subscriptions is done. This way you can add things to specific and favorite them separately, so you can keep better track of shows in the library. Since favorites has a different usage than bookmarks also, it isn't added on the homepage, only to the library. For now, it doesn't work on TV as I couldn't find a good layout implementation for it, so I hid both it and subscriptions in the library since neither can be used on TV, there is no reason for it to be there. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code looks fine, however I have not tested it, and no tv support?
No TV support yet I might do one on a follow-up, I just could not find a proper place for the UI element on TV layout just yet. |
@LagradOst I was able to do TV support now afterall |
Refs #614
This also hides subscriptions on TV since they can not be used on TV so no reason to show them in library