Skip to content
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

Rearrange pivot items project view #63

Open
NPadrutt opened this issue Jan 12, 2015 · 4 comments
Open

Rearrange pivot items project view #63

NPadrutt opened this issue Jan 12, 2015 · 4 comments

Comments

@NPadrutt
Copy link
Collaborator

For me the order of the items don't represent the usage. I would change it to:

  • details
  • issues
  • commits
  • pull request
  • tree
  • collaborators
  • watchers

Also I'm not sure if the last 3 should better go into an minimized app bar instead use a whole pivot each. As far as I know Microsoft recommends to have not more than 5 pivot items as well.

@PallasDoesCode
Copy link
Collaborator

Here is an excerpt from the MSDN documentation on pivot controls found here:

Apps should minimize the number of pivot pages they have. Users can become lost if they jump from pivot page to pivot page. Try to keep the number of pivot pages to six or fewer.

I thought I had was to include the number of collaborators on the details view. Then set it up so that whenever you click on either "Watchers" or "Collaborators" in that view it would take you to a special view dedicated to displaying those lists of users. This way it removes it from list of pivots and still allows the user to view the lists.

As for the tree view, I'm fine with it staying in the pivot list long as at some point we look into possibility of implementing some type of syntax highlighting so it's easier to read.

Beyond that I like the order you have the pivots in.

@NPadrutt
Copy link
Collaborator Author

To include collaborators and watchers on the detail page sounds like a good idea to me.

I use the tree view to less (nearly never). Therefore the pivot seems for me like a too prominent place. But I guess that's a small issue ^^

@PallasDoesCode
Copy link
Collaborator

If @michelsalib and @AlbertoMonteiro don't have any objections about this change then I'll start looking at rearranging the pivots and adding the collaborators and watchers to the details pivot. I'll wait about removing them from the list of pivots until I know how hard it will be set up their own view.

@PallasDoesCode PallasDoesCode self-assigned this Jan 12, 2015
@AlbertoMonteiro
Copy link
Collaborator

For me, no problem :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants