Skip to content

Conversation

@Tunous
Copy link
Collaborator

@Tunous Tunous commented Jun 6, 2017

A test implementation of repository tabs as suggested in #578 (comment).

screenshot_1496770057
screenshot_1496770053

Didn't finish as I'm not sure about this. In finished implementation filters/sort orders should update when changing pages. This could be confusing...

@taltstidl
Copy link

@Tunous I like it, it's definitely an improvement over the current situation 👍. But I'm the one who suggested it so I don't think my opinion really counts 😉.

@maniac103
Copy link
Collaborator

As for the confusion, I'd suggest keeping the sort order intact when switching between tabs (have a single sort order selection for all 3 tabs) and only hiding the filter options on the starred and watched tabs (as those have an implicit filter set).

As for the tabs themselves, I'm torn. On one hand I don't like 'tabifying' of stuff if not strictly needed, but on the other hand there's a significant amount of people who currently struggle to find the starred repositories, so this is certainly an improvement. The only other option that comes to my mind is adding multiple entries to the drawer, but the multiple tabs approach probably fits better into our current model (see e.g. the 'My gists' section), so it's fine, I guess :)

@Tunous Tunous self-assigned this Jun 8, 2017
@smichel17
Copy link
Contributor

@maniac103 I generally agree with your sentiment about tabifying[1]. However, it's just a fact that there's a ton of information on github, and it's all got to fit into one app somehow[2]. My own suggestion for where to put this is in #696.

[1]: see FastHub for an example of tabifying gone wrong. Good lord, there's so many moving interface pieces -- while the functionality is great, it's so difficult to use, even with extra animations turned off.

[2]: Honestly, I wouldn't be upset to remove file browsing functionality except the readme and have a companion app for all the git stuff (which would work for any website and could also be used by LabCoat for GitLab, etc). It'd be hard to pull off well, though.

@maniac103
Copy link
Collaborator

[1]: see FastHub for an example of tabifying gone wrong. Good lord, there's so many moving interface pieces -- while the functionality is great, it's so difficult to use, even with extra animations turned off.

Agreed (I've outlined similar thoughts at #681 (comment)) - I want to avoid cramming too much stuff into the UI because I don't like the outcome seen in FastHub and Gitskarios.

[2]: Honestly, I wouldn't be upset to remove file browsing functionality except the readme and have a companion app for all the git stuff (which would work for any website and could also be used by LabCoat for GitLab, etc). It'd be hard to pull off well, though.

That won't happen, simply because it doesn't fit the way I use the app myself ;-) I want Octodroid to do whatever meaningfully can be done 'on the go' and reviewing code (be it in PRs or the repo) is part of that. One of my most frequent use cases of the app is following code links posted in some IRC or Slack channels, and the code browsing is needed for that.

Tunous added a commit that referenced this pull request Sep 11, 2017
This reproduces the behavior present on GitHub web interface where notifications and watched repositories are grouped under 2 tabs in the same page.

Related to #578 and #624
Tunous added a commit that referenced this pull request Sep 16, 2017
@Tunous Tunous changed the title [WIP] Repositories tabs Repositories tabs Sep 23, 2017
@Tunous Tunous added the wip label Sep 23, 2017
@Tunous Tunous removed their assignment Sep 26, 2017
@Tunous Tunous removed the wip label Sep 27, 2017
Tunous added a commit that referenced this pull request Sep 28, 2017
Tunous added a commit that referenced this pull request Mar 23, 2018
Tunous added a commit that referenced this pull request Mar 23, 2018
@Tunous
Copy link
Collaborator Author

Tunous commented Mar 23, 2018

Decided to merge #734 instead of this.

@Tunous Tunous closed this Mar 23, 2018
@Tunous Tunous deleted the repositories-tabs branch March 24, 2018 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants