docs: Navbar improvement - #310
Conversation
| ## Quick start | ||
|
|
||
| <Stepper> | ||
| <Stepper toc={false}> |
There was a problem hiding this comment.
needed so that zudoku v88 doesn't add counters to the table of content for the steps
| "@types/react": "^19", | ||
| "@types/react-dom": "^19", |
There was a problem hiding this comment.
needed for the react component of the Navbar
| "react-dom": ">=19.0.0", | ||
| "regex-utilities": "^2.3.0", | ||
| "zudoku": "^0.86.0" | ||
| "zudoku": "^0.88.0" |
There was a problem hiding this comment.
updated because bug in v86
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
Active navigation matching does not account for the production /docs base path.
Review effort: Lite
Findings: None
What changed in this PR
Improves the documentation navbar by grouping guides and consolidating reference links.
Changes:
- Groups Tutorials and Example apps under Guides.
- Adds a Reference dropdown for CLI, Kraftfile, and Platform API.
- Updates guide generation, documentation, and Zudoku dependencies.
| File | Summary |
|---|---|
zudoku.config.tsx |
Restructures navigation and adds Reference links. |
src/TopNavMenus.tsx |
Implements grouped menus; active-route matching needs to account for /docs base path. |
scripts/update_zudoku_guides.py |
Updates generated example-app entries. |
pnpm-lock.yaml |
Refreshes dependency resolution. |
pages/introduction.mdx |
Adjusts Stepper table-of-contents behavior. |
pages/guides/overview.mdx |
Documents tutorial and example-app groupings. |
package.json |
Upgrades Zudoku and adds React typings. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
42e4a22 to
4667c69
Compare
Signed-off-by: aabedraba <abdallah@unikraft.com>
4667c69 to
7787686
Compare
|
For me, i'd like to keep "CLI Reference" visible at the top-level without hiding it, same with "Platform API" -> less clicks / more visible. I think we can shorten them though:
Having an "Integrations" dropdown does make sense though, with "Kubernetes", "Docker", "Terraform", etc. can be placed there. "Kraftfile" needs a bit of thought where we should put that. Also agree to merge Guides/Tutorials -> it's the same thing really. |
7787686 to
2f20b00
Compare
Signed-off-by: aabedraba <abdallah@unikraft.com>
2f20b00 to
9543488
Compare
nderjung
left a comment
There was a problem hiding this comment.
Thanks!
Reviewed-by: Alexander Jung alex@unikraft.com
Approved-by: Alexander Jung alex@unikraft.com

Uh oh!
There was an error while loading. Please reload this page.