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

HTMLBuilder: new parameter in config.yml: externallink #501

Closed
takahashim opened this issue Apr 2, 2016 · 6 comments
Closed

HTMLBuilder: new parameter in config.yml: externallink #501

takahashim opened this issue Apr 2, 2016 · 6 comments
Milestone

Comments

@takahashim
Copy link
Collaborator

config.ymlでexternallink: falseと書くと、@<href>{}でハイパーリンクを作ることを無効にできるようにする。

cf. #429

@kmuto
Copy link
Owner

kmuto commented Apr 2, 2016

externallinkブランチで実装してみました。`@{url, label}``の場合は「LABEL(URL)」にしています(実際は日本語では全角カッコにするためにi18n.ymlにて定義)。

本件が必要になるのはepubmakerくらいと思われるので、epubmaker:の下のexternallinkとしたのですが、シチュエーションによってちょっと問題があります。

  • epubmaker:を一切YAMLに定義していないならハイパーリンクは作られます。
  • review-epubmaker経由であればデフォルトでハイパーリンクは作られます(producerで設定)。
  • epubmaker:があり、かつreview-compileでHTMLにする場合はepubmaker:externallink:の初期値がnullになってしまうので、 externallink:trueと明示指定しないといけません(!)。

review-webmakerを作るときにこの挙動だと厄介になりそうです。ただ、externallinkがTeXにも効くのかと思われるのは嫌な気もします。

この実装でいくなら、review-compileの初期値として、epubmaker:externallink: true も持たせる、ですかねぇ。

@takahashim
Copy link
Collaborator Author

この辺りはReVIEW::Configureクラスの方でよろしくやってほしいわけですが、configにmakerを指定した際のフォールバックの仕様をちゃんと決めるべきですかね…。

@kmuto
Copy link
Owner

kmuto commented Apr 3, 2016

Configureのほうに初期値入れました。

@kmuto
Copy link
Owner

kmuto commented Apr 5, 2016

これも #515 と絡んでます?

@takahashim
Copy link
Collaborator Author

はい、「WebではリンクにしたいけどEPUBではリンクNGにしたい」と「同じ結果になるようWebでもEPUBでもリンクNGにしたい」との選択肢は両方欲しいです。

@kmuto
Copy link
Owner

kmuto commented Apr 13, 2016

マージされましたね。

@kmuto kmuto closed this as completed Apr 13, 2016
@kdmsnr kdmsnr added this to the 2.0.0 milestone Apr 24, 2016
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

No branches or pull requests

3 participants