michelsalib / Gi7

A github app for Windows Phone 7
31 stars 14 forks source link

Rearrange pivot items project view #63

Open NPadrutt opened 9 years ago

NPadrutt commented 9 years ago

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

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.

codewithtyler commented 9 years ago

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 commented 9 years ago

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 ^^

codewithtyler commented 9 years ago

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.

AlbertoMonteiro commented 9 years ago

For me, no problem :D