-
Notifications
You must be signed in to change notification settings - Fork 757
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
Feature/channels page #2129
Feature/channels page #2129
Conversation
I think it better to move the channels tab icon to the top of the channels list on the sidebar because channels already has a dedicated section |
I'll mark this with WIP because i'd like to see #1783 merged first |
@hockerschwan #1783 has been merged |
Head branch was pushed to by a user without write access
added update thumbnail and unsubscribe prompt ezgif-1-6db2b5f90d.mp4
|
Head branch was pushed to by a user without write access
Head branch was pushed to by a user without write access
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.
Last thing before approving. If list is empty hide the search bar. Just like it is implemented in the playlist and history page
Head branch was pushed to by a user without write access
done |
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.
lgtm!
i think that this pr closes #1350. IMO it wouldnt look nice in the sidebar but does look nice on this page. |
yes. description updated |
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 review done
Give me some time to do local feature review...
Thx for implementing this, but could you add one more thing? |
@toby63 That was already in FreeTube before this PR: Distraction Free Settings -> Hide Active Subscriptions |
@absidue Thx, somehow I overlooked it, even though I was checking twice 😅. |
Title
Pull Request Type
Related issue
Closes #1350
Closes #1506
Closes #2066
Description
Subscribed channels list (similar to ft-profile-channel-list)
Screenshots (if appropriate)
![](https://user-images.githubusercontent.com/80553357/158335874-3d498526-4f33-49bf-b496-3d616933fcd6.PNG)
Testing (for code that is not small enough to be easily understandable)
Manually adding/deleting channels
Desktop (please complete the following information):
Additional context
Feedbacks appreciated ( spacing / size / wording etc.)
Reminder: rewritethumbnailURL
when #1783 is implemented.