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

imgmathの挙動拡張 #1138

Merged
merged 8 commits into from
Oct 15, 2018
Merged

imgmathの挙動拡張 #1138

merged 8 commits into from
Oct 15, 2018

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Oct 13, 2018

#868 の対応。
review_version 3のときに有効にしています。

  • images/review_math/ に__IMGMATH_BODY_.{tex|map}を置いておき、epubmaker の最後にまとめて遅延実行する(成功したらこれらの一時ファイルは消去)
  • imgmath_options でフォーマット、変換ロジック(pdfcropまたはdvipng)、プリアンプルファイル、フォントサイズ、行高、pdfcropとそのピクセライズコマンド(デフォルトはpdftocairo)を設定可能に
  • pdfcrop_cmd、pdfcrop_pixelize_cmdはパラメータ渡すのがどうにも難しいのでプレースホルダ型の文字列に

と書いていたら、もう少し直さないといけないことに気付きました。

  • webmakerでも動くように共通化してモジュールに切り出したほうがよさそう 済み。makerhelperへ
  • 作成済みかどうかをチェックし忘れていた 済み。

テストが書きづらい…

@kmuto kmuto changed the title [WIP] imgmathの挙動拡張 imgmathの挙動拡張 Oct 14, 2018
@kmuto
Copy link
Owner Author

kmuto commented Oct 14, 2018

実験用。
m1.zip

@kmuto kmuto mentioned this pull request Oct 14, 2018
9 tasks
@kmuto
Copy link
Owner Author

kmuto commented Oct 14, 2018

単一ページしか扱えないツール向けにもpdfjam呼び出しで対処しました。

@munepi
Copy link
Contributor

munepi commented Oct 14, 2018

この拡張は、Windowsでも使えるようにすることを想定していますか?

すでに調べておられたらアレですが、pdfjam はTeX Liveの pdfjam.<arch> パッケージから提供されているんですが、 pdfjam がshスクリプトで実装されているために、 pdfjam.win32 パッケージが存在していない状況です。

@kmuto
Copy link
Owner Author

kmuto commented Oct 14, 2018

おぉなるほど。
「デフォルトで入っている」PDFのスライサを決めづらいんですよね。
コマンドライン+プレースホルダの指定はできるようにしているので、Windowsのひとは適宜pdftkなどに変えてもらう?

また、もともとこのスライサを使うのはmagickのような単一ページ指定しかできないもののためなので、pdftocairoやdvipngが使えればそっちで問題ないというところではあります。

@kmuto
Copy link
Owner Author

kmuto commented Oct 14, 2018

とりあえずpdftkの例を入れておきました

@naotaco
Copy link

naotaco commented Oct 14, 2018

ありがとうございます!

プリアンブルに\usepackage{chemfig}を追加して所望の画像が生成されてepubに埋め込まれることが確認できました。

ですが2点下記2点気になるので設定を変更したいのですが、どこをいじるのがよいでしょうか?試しにdvipng_cmdをいじってみましたが動作が変わったように見えず。。

  • 画像の解像度が小さい(白黒PNGですしサイズの問題はあまりないので、せめて300dpi以上になるようにはしたい
  • 画像の余白が小さい(これはepubリーダーの問題だと思うのでアレですが、調整するとして)

@kmuto
Copy link
Owner Author

kmuto commented Oct 14, 2018

画像の解像度が小さい(白黒PNGですしサイズの問題はあまりないので、せめて300dpi以上になるようにはしたい

画面だと「単にでかくなるだけ」になりそうですが、

  • dvipng_cmd: "dvipng -D 300 -T tight -z 9 -p %p -l %p -o %o %i"
  • fontsize:の値を大きくする

あたりでしょうか。

画像の余白が小さい(これはepubリーダーの問題だと思うのでアレですが、調整するとして)

余白については、span.equationで囲んでいるので、そのCSSで対処するのがよさそうです。
また、dvipngのほうだと余白設定をするところはないのですが、pdfcropのほうであればpdfcrop_cmd: "pdfcrop --hires --margins \"2 2 2 2\" %i %o" といった指定はいちおうできるかな。でもspanのほうでやったほうがたぶんよいです。

@takahashim
Copy link
Collaborator

これもマージしておきますね

@takahashim takahashim merged commit aa2c055 into master Oct 15, 2018
@takahashim takahashim deleted the imgmath2 branch October 15, 2018 13:04
@naotaco
Copy link

naotaco commented Oct 15, 2018

ありがとうございます。

# dvipngだとchemfigがうまく動かなかったので、pdfcropを使うことにしました

fontsizeの設定がインライン要素の方にのみ効き、texequationには効かないのは仕様でしょうか?

個人的には連動していた方がよいと感じたので、PRを出しました。 #1146 よろしければご検討ください。

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.

4 participants