fix: nest Module 0 and Phase groups under the Guides sidebar section - #12
Merged
Conversation
After #10 the Module 0 and Phase 1-3 groups sat at the top level of the sidebar, as siblings of "Guides", even though all of that content lives under docs/guides/. Move them inside the Guides group's `items` so the navigation matches the content structure: Guides now expands to the landing page, the example reference, and the Module 0 / Phase sub-groups. No change to page content, slugs, or ordering -- still autogenerate-driven and duplicate-free (verified against the built nav). site/README.md updated to show the nested structure.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this change?
Moves the Module 0 and Phase 1–3 groups so they are nested inside the Guides sidebar section, instead of sitting at the top level as siblings of it.
After #10 the duplication was fixed, but those groups rendered as top-level entries — even though all of their content lives under
docs/guides/. NowGuidesexpands to: its landing page, the example reference, then the Module 0 and Phase sub-groups.Why?
The navigation didn't match the content structure: everything under
guides/should appear under "Guides". Reported from the live site — the module/phase sections weren't reachable from the Guides section.Type of change
Checklist
npm run buildclean; built sidebar verified (Module 0 + Phases nested under Guides, no duplicates, original order)fix:commitNotes for reviewers
Guidesstays an explicititemslist because it mixes loose pages with the nested autogenerated sub-groups — autogeneratingdocs/guidesdirectly would re-list the module/phase tree.site/README.mdupdated to show the nested shape.