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

review-jlreq: \includefullpagegraphicsを縦書きモードに対応 #1738

Merged
merged 3 commits into from
Sep 7, 2021

Conversation

munepi
Copy link
Contributor

@munepi munepi commented Sep 6, 2021

#1734
review-jlreq.clsに定義している\includefullpagegraphicsを縦書きモードに対応します。
ついでに、review-jsbook.clsにも同じ対応をしました。

この変更に合わせて、amsmathパッケージを読み込んだときに、\includefullpagebraphics*コマンドが期待した動作をしないのを回避するようにしました。

@kmuto
Copy link
Owner

kmuto commented Sep 7, 2021

ビルドエラーになっていますが、lualatexだとiftdirがだめなんですかね、これは。

@munepi
Copy link
Contributor Author

munepi commented Sep 7, 2021

ビルドエラーになっていますが、lualatexだとiftdirがだめなんですかね、これは。

そうですね。
とりあえず、\iftdir -> \if@tateに変更します。

ちなみに、

  • 本文:縦組み
  • 後付けのツキモノ系の一部(著者紹介、奥付):基本版面を変更して、直接LaTeXで横組み

のような(商業誌で割とよくありますが、組方向を途中で変えるという意味で)特殊な構成にする場合は、この構成用に改修しないと、対応できないですね。
なので、奥付や著者紹介などを横組みにしたいなら、PDF画像として挿入してもらうほうが良さそうです。

@kmuto
Copy link
Owner

kmuto commented Sep 7, 2021

ビルドエラーになっていますが、lualatexだとiftdirがだめなんですかね、これは。

そうですね。
とりあえず、\iftdir -> \if@tateに変更します。

ありがとうございます、これで取り込みます。

ちなみに、

  • 本文:縦組み
  • 後付けのツキモノ系の一部(著者紹介、奥付):基本版面を変更して、直接LaTeXで横組み

のような(商業誌で割とよくありますが、組方向を途中で変えるという意味で)特殊な構成にする場合は、この構成用に改修しないと、対応できないですね。
なので、奥付や著者紹介などを横組みにしたいなら、PDF画像として挿入してもらうほうが良さそうです。

ですねぇ… 暫定的でもminipageでなんとかならんかなと思ったのですが、plextで別の箇所が壊れちゃうので困りました。

@kmuto kmuto merged commit 6219a30 into kmuto:master Sep 7, 2021
@munepi munepi deleted the support_tdir_includefullpagegraphics branch September 7, 2021 04:15
@munepi
Copy link
Contributor Author

munepi commented Sep 7, 2021

@kmuto
実際に、前述の特殊な構成は、技術的にTeX側で縦組み(右開き)の基本版面から、完全な横組み(右開き)の基本版面へ切り替えられます。
しかしながら、単ページの横組みならば、plextパッケージを使わずに、\yoko+minipage環境で組み込むこともできるといえばできます。

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