Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

Method order in the side nav should match method order on the phase pages. #82

Closed
jameshupp opened this issue Dec 4, 2015 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@jameshupp
Copy link
Contributor

The side nav lists the methods alphabetically across each phase.

The phase pages list the methods in categories. Within each category, the methods are alphabetical.

The order on the pages is more logical, since phases that help accomplish similar things are grouped more closely. If a user is on an individual method page, the user doesn't see the categories; they only see the side nav. But the user can't tell without going back to the phase page which other methods are similar in purpose.

Adding metadata for order or weight would allow us to order the side nav the same way we order on the phase pages, even if we don't list the categories in that side nav.

@jameshupp jameshupp added the enhancement New feature or request label Dec 4, 2015
@jenniferthibault
Copy link

@jameshupp - it appears that it IS possible to re-order the items on the side nav. No front-ender required! 💥

I have that up and running on the branch side-nav-organization if you are interested in testing it out locally to see how it feels.

Here are some screenshots:

screen shot 2015-12-16 at 1 10 55 pm

screen shot 2015-12-16 at 1 11 12 pm

screen shot 2015-12-16 at 1 11 26 pm

screen shot 2015-12-16 at 1 11 35 pm

screen shot 2015-12-16 at 1 11 41 pm

Thoughts? @jameshupp @colinpmacarthur @carodew @bradnunnally ?

@jameshupp
Copy link
Contributor Author

Oh cool! A+ sleuthing. Everything looks and works 💯 on my machine.

If at some point the set gets bigger or more diverse, we may want to come back to exploring card-by-card metadata. My hypothesis is it'll be easier to manage things like name changes or regrouping (which might be wrong! I just like exploring!). But for now: I say let's deploy this.

@carodew
Copy link
Contributor

carodew commented Dec 17, 2015

Without actually looking at the code, LGTM (the screenshots)

@colinpmacarthur
Copy link
Contributor

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants