fix: Correct anchor IDs of functions and their parameters

[YDX-2147483647]

Changes

Resolves #194
Resolves #230

• feat: Support multi-level table of contents
• fix: Correct IDs of functions in group (e.g., calc.abs) — by changing #definitions- to #functions-
• fix: Correct parameter IDs — by refactoring into buildParamId

Official	Before	After

Official: #definitions-at-index

Before: #definitions-at-at-parameters-index

After: #definitions-at-index

Analysis

Things are quite tricky.

First of all, the typst-docs crate generates hrefs in table of contents and cross links, while website is responsible for assigning ids in the main content. This PR displays hrefs in all levels in table of contens, and correct ids.

Now let's figure out the ID pattern.
There are 7 kinds of page. Among them, only pages of kind function, group, and type contain function parameters.

https://github.com/typst-jp/typst-jp.github.io/blob/fb3a73693e35fa3a78955ce53d7207654f2f56c6/docs/src/model.rs#L48-L61

And their patterns are as the following.

Page (kind)	Function	Parameter	ID
figure (function)	figure	body	#parameters-body
figure (function)	figure.caption	body	#definitions-caption-body
calc (group)	calc.abs	value	#functions-abs-value
array (type)	array.at	index	#definitions-at-index
array (type)	Constructor	value	#constructor-value

Outdated description

These are the only two occurences.

$ rg '[-]?parameters-' website/src/
website/src/components/ui/FunctionParameters.tsx
26:                                             id={`parameters-${param.name}`}

website/src/components/ui/FunctionDefinition.tsx
31:                                                                             href={`#parameters-${param.name}`}

And they exists since the very beginning (#117).

[3w36zj6]

Hi @YDX-2147483647,
Looks good to me, thanks!

Once other maintainers approve, it will be merged.

[YDX-2147483647]

Oh, I find #definitions-* can also be wrong…

https://typst.app/docs vs. https://typst-jp.github.io/docs:

• <GroupTemplate>

  • https://typst.app/docs/reference/foundations/calc#functions-abs
    https://typst-jp.github.io/docs/reference/foundations/calc/#definitions-abs

• <FunctionParameters> — used by <GroupTemplate> (e.g. calc), <TypeTemplate> (e.g. array), and <FuncTemplate> (e.g., figure)

  • https://typst.app/docs/reference/foundations/array/#definitions-product-default
    https://typst-jp.github.io/docs/reference/foundations/array/#definitions-product-product-parameters-default

  • https://typst.app/docs/reference/model/figure/#definitions-caption-body
    https://typst-jp.github.io/docs/reference/model/figure/#definitions-caption-caption-parameters-body

  • https://typst.app/docs/reference/foundations/calc#functions-root-radicand
    https://typst-jp.github.io/docs/reference/foundations/calc/#definitions-root-root-parameters-radicand

Please wait until I or anyone fix other potential problems on anchors…

Update: Maybe typst-docs should generate the JSON with IDs.
Update 2: It does, but in outline.

[YDX-2147483647]

It's ready for review now... I found that there are five id patterns. Please refer to the (updated) PR description.

Update: Still buggy. /docs/reference/model/figure/#figure-placement should be #parameters-placement.
Update 2: Just a small typo. Fixed.

[YDX-2147483647]

To help review, I deploy the upstream (English) dev docs with website/ of this PR (529f643):
https://688890732a71cd00085fc347--ydx-typst.netlify.app

Examples:

• https://688890732a71cd00085fc347--ydx-typst.netlify.app/docs/reference/model/figure/#parameters-body
• https://688890732a71cd00085fc347--ydx-typst.netlify.app/docs/reference/model/figure/#definitions-caption-body
• https://688890732a71cd00085fc347--ydx-typst.netlify.app/docs/reference/foundations/calc#functions-abs-value
• https://688890732a71cd00085fc347--ydx-typst.netlify.app/docs/reference/foundations/array/#definitions-at-index
• https://688890732a71cd00085fc347--ydx-typst.netlify.app/docs/reference/foundations/array/#constructor-value

[Copilot]

Pull Request Overview

This PR fixes anchor ID generation for function parameters and adds support for multi-level table of contents. The main purpose is to align the documentation website's anchor IDs with the official Typst documentation format.

• Introduces a standardized buildParamId function to generate consistent parameter anchor IDs across different page types
• Replaces hardcoded ID prefixes with a unified system that handles top-level functions and nested methods differently
• Adds multi-level table of contents support with proper indentation for nested items

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
website/src/components/ui/type2href.ts	Adds buildParamId utility function for consistent parameter ID generation
website/src/components/ui/common/TableOfContents.tsx	Implements recursive table of contents rendering for multi-level navigation
website/src/components/ui/FunctionParameters.tsx	Updates parameter ID generation to use new buildParamId function
website/src/components/ui/FunctionDisplay.tsx	Updates prop types and default values for prefix parameter
website/src/components/ui/FunctionDefinition.tsx	Updates href generation to use new buildParamId function
website/src/components/templates/GroupTemplate.tsx	Changes prefix from "definitions-" to "functions-" for group page methods
website/src/components/templates/FuncTemplate.tsx	Removes prefix for top-level function parameters to match official format

[kimushun1101]

Thank you very much for your kind contribution.
Confirmation with Netlify was also helpful. Just to be sure, I have confirmed that it works in my local environment, and it was fine.

This PR also fixes #194 (in Japanese)

[gomazarashi]

Thanks for fixing this! I've confirmed it works on my end. LGTM!

[3w36zj6]

Nice work, thank you!

[YDX-2147483647]

Let’s merge this! #234 is another (independent) bug on website generation, and fixing it will use the function `buildParamId` defined in this PR. Therefore, we can’t do any substantive work on that before this PR is merged.