Skip to content

Conversation

@3w36zj6
Copy link
Member

@3w36zj6 3w36zj6 commented Jun 10, 2025

close https://github.com/typst-jp/typst-jp.github.io/issues/178

groups.ymlで管理されているページのみルートパスの末尾のスラッシュが欠けていたため、他の箇所に合わせてスラッシュを追加しました。

現在の本家リポジトリでは修正済みです1

Footnotes

  1. https://github.com/typst/typst/pull/6420

@3w36zj6 3w36zj6 requested review from Copilot and kimushun1101 June 10, 2025 09:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR unifies the trailing slash usage in documentation routes. The changes ensure that all path entries in the translation status and library route generation end with a slash.

  • Updated trailing slash in website/translation-status.json for multiple documentation routes.
  • Modified the route formatting in docs/src/lib.rs to append a trailing slash.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
website/translation-status.json Added missing trailing slashes to documentation paths
docs/src/lib.rs Updated route formatting to include a trailing slash

Copy link
Member

@kimushun1101 kimushun1101 left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございました。

mise run previewでスクリプト記述のページからcalc剰余のページに移動できることを確認しました。

http://localhost:4173/docs/reference/scripting/

別PRでの対応になるかもしれませんが、剰余の方の#functions-rem-euclid部分が効いていない状況です。

http://localhost:4173/docs/reference/foundations/calc/#functions-rem-euclid

理由としては、ページ右の目次部分から推測するに#functionsはありますが、それよりも細かいセクションリンクがそもそも無いように思えます。

image

@3w36zj6
Copy link
Member Author

3w36zj6 commented Jun 10, 2025

@kimushun1101

目次の小見出しの状況は確認できました。これはこのPRの責務ではないのでIssueを起票してください。

@3w36zj6 3w36zj6 merged commit 94e3d71 into main Jun 10, 2025
5 checks passed
@3w36zj6 3w36zj6 deleted the feature/standardize-trailing-slashes-in-docs-routes branch June 10, 2025 21:19
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.

Webサイトの一部リンクがpreviewおよび本番環境ではうまく遷移しない

3 participants