-
Notifications
You must be signed in to change notification settings - Fork 236
Add "Starred repositories" to navigation drawer #578
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
Conversation
|
I'm not sure whether this really is a good idea, as this is doable as of now already. One can set 'my repositories' as home page, and the filter type is stored persistently, so when selecting 'starred' there, one would have the starred repositories as home page. Maybe we just should advertise this capability better?, e.g. using some one-time help cling? If we're really going to add this, I'd prefer passing some initial filter type into the factory instead of special casing the 'starred' type. |
|
This pull request was inspired by the implementation of starred repositories on GitHub page. Here when you go to your profile screen you can see that stars and repositories have separate entries. I personally wouldn't be against adding a hint showing users where to find starred repositories. There actually might be the problem as "My repositories" doesn't feel like a place where you would find watched or starred repositories. Maybe they should be separated instead? |
Now I'm convinced about this even more. I think having them located under My Repositories is not intuitive. Just now I was trying to search for my own repository and was confused why there are many repositories not owned by me. It turned out that I've previously viewed starred repos and the filter was remembered. There are 2 solutions I propose to fix this:
I'm in favor with option 1. @maniac103 thoughts? |
|
@Tunous |
|
I'm fine with option 1 I guess. The drawer might become a bit massive though, so we'll need to think about that (better grouping? allow hiding drawer items? not sure...) |
|
I don't think it's that huge. Should be still ok with 2 more items. As for the grouping perhaps add section for repositories: My Repositories, Starred Repositories, Watched Repositories and place that under group with Bookmarks. But I'm not sure if it's ok to move My Repositories entry... |
|
In my opinion a new item in the drawer would be too much, but a tabbed interface like in "My issues" and "My Pull Requests" would be nice and allow for quicker access to the starred and watched repositories. |
|
If you're worried about too many items in the drawer, I'd suggest having "Repositories" open up in the drawer as a submenu/group directly underneath the drawer item, if that's doable (don't know about the coding aspect of this) For example: |
|
Check #624 for test tabs implementation. |
|
Suggestion by @smichel17 in #696:
Moving starred under bookmarks makes sense, but the bookmarks page also allows bookmarking users ... not sure whether this really fits. Moving watched under notifications seems to make sense to me as well, but I'm only a rare user of notifications and don't use (explicit, as opposed to automatic) repo watching myself, so take my opinion with a grain of salt there ;-) #695 also provides another idea about drawer organization. |
|
I've implemented the part about moving watched repositories. For stars I propose to rename entry in drawer to one of these:
And move them here as a tab similar to what I did with watched repos. |
|
Decided to merge #734 instead of this. |
Closes #388.