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
Add a sidebar navigation to the styleguide #170
Comments
Our RailsGirlsSoC team planned on working on #72 next. This is a good start. /cc @fanxhe-pair @freewomancode @anxhe |
Sounds good. Let me know if I can help in any way. |
Let’s have the discussion in here. (FYI: They are based in Mexico and 7 h behind us.) This should be released as a plugin. People might have very different requirements on a navigation. I don’t know yet if we can implement everything in JavaScript. In future, the LivingStyleGuide should accept projects as inputs (in other words multiple in/output files). But maybe the navigation between and within a file are two different things anyway. |
I can see the idea behind making this a plugin, however I personally would think this would be something useful for almost every styleguide. But that's not really important.
I also think that the navigation between pages of the styleguide and the navigation within one file should be somewhat separate UI components. |
For version 2 I’m fine with having it in. For later versions we should think about what is a core feature and what’s better delivered as plugin. |
We have thought about doing it with ruby, would you like we use a filter to allow the user to choose how to display the menu: horizontal or vertical. For the vertical one, we could use Jakob's styles, using the This afternoon we will be looking at how to do it with ruby. |
For bigger styleguides it's very nice to have a fixed sidebar navigation to:
a) navigate to other parts of the styleguide
b) see where you are in the styleguide
This code inside my main
.lsg
file already works pretty decently:It's required that you have jQuery included in the stylguide.
It does not yet handle different levels of (sub)headlines.
This is what it currently looks like:
The text was updated successfully, but these errors were encountered: