Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chapterlinkをデフォルトで有効にする #1532

Merged
merged 1 commit into from
Aug 15, 2020
Merged

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Aug 14, 2020

#1529 #1530 の対応です。

  • chapterlinkをデフォルトでtrueにしました。
  • config.yml.sampleにドキュメント追加しました。
  • chapterlink on/off時のテストを追加しました。
  • hdインライン命令をLaTeX展開したときのreviewsecrefで、hyperrefを有効にしました。
  • 厳密にはchapterlinkとは別の話なんですが、columnインライン命令で別ファイルを参照してもちゃんと動いていなかったので、chapter/idをちゃんと分解してchapterのhtmlを参照するようにしました。
  • pdfmakerにおいては、media=printの場合はhyperrefパッケージのオプションでハイパーリンク機能を殺しているので変化はなし。media=ebookの場合はhdインライン命令の箇所がリンクになります。図表・リスト・式のほうはリンクにはしていません(やろうとするとちょっと面倒くさそう)。

LaTeXのreviewcolumnrefのほうは、hyperrefを入れてもうまく動いてくれませんでした。どうもreviewcolumnheadに渡しているhypertargetのほうが無視されてラベルとして見つからないっぽい? とりあえず今は必須というほどではないので対処から見送っています。

@kmuto kmuto requested a review from takahashim August 14, 2020 13:33
@takahashim
Copy link
Collaborator

「chapterlinkをデフォルトで有効にする」ことについては問題ないかと思います!
それ以外の修正についてはぱっと見ではよく分かりませんでした…

@kmuto
Copy link
Owner Author

kmuto commented Aug 15, 2020

ありがとうございます、何か問題あればまた調整します。

@kmuto kmuto merged commit de7aefa into master Aug 15, 2020
@kmuto kmuto deleted the default-chapterlink branch August 15, 2020 09:47
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.

2 participants