Skip to content

change main docs structuring#3020

Open
crowlKats wants to merge 2 commits into
mainfrom
freestyle_left_menu
Open

change main docs structuring#3020
crowlKats wants to merge 2 commits into
mainfrom
freestyle_left_menu

Conversation

@crowlKats
Copy link
Copy Markdown
Member

@crowlKats crowlKats commented Apr 1, 2026

image (4) image (3) image (2) image (1)

Copy link
Copy Markdown
Contributor

@lunadogbot lunadogbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. sidebarUrlMap[normalizedUrl] only does an exact lookup, and runtime/_data.ts now keeps Standard Library as a single /runtime/reference/std/ entry. Existing nested std pages such as /runtime/reference/std/assert/ are no longer mapped to the package sidebar, so they fall back to basicsSidebar. Please add prefix routing for this tree or keep/generated-map the std module hrefs.
  2. reference_gen/parallel-doc.ts is under the generated reference tree. Repo rules keep reference/ and reference_gen/ changes upstream-generated, so this formatting fix should come from the generator/upstream source or be split with maintainer confirmation.

Copy link
Copy Markdown
Contributor

@fibibot fibibot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The runtime subnav routing loses nested pages that are not enumerated in sidebarUrlMap, and this also hand-edits generated reference output.

  1. sidebarUrlMap[normalizedUrl] only accepts exact URLs, but runtime/_data.ts now keeps std as a single Standard Library entry at /runtime/reference/std/. Pages like /runtime/reference/std/assert/ are not collected, so they fall back to basicsSidebar instead of the package/std sidebar. Add prefix routing for std or keep the child hrefs in the map.
  2. reference_gen/parallel-doc.ts is under generated reference output. Repo policy keeps reference/ and reference_gen/ changes generated from upstream sources, so this formatting fix should be split or regenerated instead of hand-edited here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants