-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Subscriptions view #1826
Subscriptions view #1826
Conversation
… subscription view
Few things that bugged me:
|
For counter: https://github.com/shts/TriangleLabelView |
I'm in favor of using triangles : it looks nicer to me and it stands out more. |
I don't think I like any of these. Would take a fair bit of work to implement. What I do think we should add is 'Add Subscription'. We might even remove it from the Nav Drawer.
Added one that brings up the same as in the drawer.
Do you think this is better than just putting in the action bar?
This is how all the other items in the nav bar work. There was an issue where someone wanted it to bring the nav bar back. You could do that...
I think I've resolved this.
How could the images be bigger? Only 2 columns?
Added counter. I went with triangles over the circle because the circle didn't have room for 3 digit numbers. I'm reticent to do any more on this PR. My main goal was to get the other contribution in to an acceptable state. Once this is merged it should be easy to add other features (filtering, add feed, etc...) Current thoughts? |
Don't think this is a good idea. At least as long as we don't have "Add feed" on any empty list message where it makes sense (all episodes, new episodes, queue, subscriptions)
Having add feed in the action bar sounds like bad UX. Even the FAB would be better. But as this thing is pretty ugly and always on top of the thing you are most interested in, I think an extra cell is much better.
The problem is not that the app closed. My problem is that I would expect to be taken back to subscriptions. When I go from Feed to an episode, back takes me back to the feed. When I go from subscriptions to a feed, there is no going back. But I would assume that, it fells just natural.
I think 3 columns is fine. |
Also, up for discussion: I think subscriptions should be at the top of the drawer, even before queue. Also think it needs another icon, I propose https://design.google.com/icons/#ic_subscriptions |
Agree w/ remove from Nav Drawer. But yeah, an action bar button or cell with big plus to add episodes would be a requirement (and an empty state screen great to have)
Agree
Top of the drawer: yes. Before the queue: absolutely not. The queue is the epicentre of AP, not the feeds. We can't even use feeds as a 'playlist' atm. AP is about listening to podcasts (which most people do via the queue), not about managing subscriptions (or whatever the subscription view allows for). The queue should be the 'home'.
You want to do some YouTube product placement? o_O AP's main purpose isn't even to play video :P I also have my doubts about the 'folder' icon. Maybe we could follow Pocket Cast's example and show a grid: https://design.google.com/icons/#ic_apps (or we can ask the folks over at https://github.com/Templarian/MaterialDesign to make an adapted version (to turn 90°) of https://design.google.com/icons/#ic_view_comfy)
One of those could be a sort function (alphabetical OR # of episodes (as per settings)) |
If you don't want to change anything else yourself, I think this is ready to be merged. |
What is left to do? Some action for the action bar? |
I would expect the 'Add podcast' item to always be the last item of the list btw. Seems like a waste of space in the 'above the fold' section otherwise. And with the risk of nagging: it would be nice if text justification would be centre. And also afraid off the Dutch translation being too long to fit on one line ("podcast toevoegen "), so hope its flexibly set up. |
Tried with 1.6.0.0 and my feelings are :
|
finishes the work of #1624
Made some of the changes recommended in that PR.
@mfietz I'd appreciate your input. I think the way the images and the placeholders are working isn't quite right and I'd like your ideas.