-
Notifications
You must be signed in to change notification settings - Fork 127
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
DOC-3967 Restructure sidebar #429
base: main
Are you sure you want to change the base?
Conversation
I've noticed a few glitches in the staged build of the docs, so I've added "do not merge" for now. I'll get them fixed ASAP. |
@mich-elle-luna I think that having more top-level items open and clearly visible was a major part of this task (eg, you shouldn't have to dig down to see something as important as Search and Query). But yeah, I do agree that having too many top-level items isn't good either. I've collapsed the Reference section at the bottom, since it doesn't really need to be open. Also, it is part of the plan to remove the Use Redis section altogether, because those pages can easily be re-homed in better places (eg, Keyspace notifications should probably go in a folder with Pub/sub). I wanted to get some feedback about the general feel of the sidebar before removing Use Redis because this step will involve a bit of rewriting here and there. But if you think it's best to remove it before launching the new sidebar then I'm fine with that. |
Hi @andy-stark-redis thanks, collapsing reference definitely helps! For the styling, maybe we could replace the grey bar and just have these sections look like the other open sections with the left-side vertical grey line? |
There are a few places where I've changed some of the content but it would be great if you could just have a play with the sidebar in the staging page and let me know if there is anything major missing or listed in the wrong order. Also, any further suggestions for styling are still welcome :-) BTW, only the Develop With Redis section has the sidebar for now. I'll also add some instructions elsewhere to explain how to add new pages to the index.