Skip to content

Conversation

@mkpoli
Copy link
Contributor

@mkpoli mkpoli commented Aug 25, 2024

Fixes #46

日本語の文字など英数字(ASCII)が含まれているMarkdown見出しに対し、heckパッケージの変換を通したidではなく(通してしまうと英字以外が消されてしまうため)、そのままtitleを使うようにしました。

title以外のものを目次にする場合は不便でしょうが、今のところ見出しをそのまま右に使っても構わないと判断したため、そのまま使うようにしました。また、titleが長いことが多いので、Reactの日本語ドキュメントのように文字サイズを小さくするほうが将来的に良さそうかもしれません。

image

@3w36zj6 3w36zj6 self-requested a review August 25, 2024 14:10
Copy link
Member

@3w36zj6 3w36zj6 left a comment

Choose a reason for hiding this comment

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

動作確認ができました。LGTMです。

@3w36zj6 3w36zj6 merged commit 04dc4a6 into main Aug 29, 2024
@mkpoli mkpoli deleted the feat/on-this-page branch August 30, 2024 06:50
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.

目次を日本語化する

3 participants