Skip to content

Conversation

@kmuto
Copy link
Owner

@kmuto kmuto commented Sep 6, 2021

#1732 の対応の検討

目標

  • review-jlreqでtate指定時に、大扉、奥付を横書きにする (新潮なんかだと奥付縦だったりするけど)
  • review-jlreq でtate指定時に、表紙を90度回転し、かつセンター合わせにする。fitpageも動くようにする

検討

  • 縦書きかどうかは if@tate でわかる。ifvmode ではない。
  • jlreq.clsのバージョンで挙動が違う気がする。新しいのだとエラーにならない、ということがままある。
  • 縦内に横を貼るのがだいぶしんどいのでplext/lltjextと minipage<y> にすると、まぁまぁ混乱せずに作れることはわかった。

問題

  • plextを入れるとsyntaxb-bookではreviewlistのところでMissing } insertedエラーになった。おそらく問題はここではない気がするが追跡が困難
  • lltjextは手元のいくつかのTeXLive環境でやってみているのだがjlreqとの組み合わせでMissing } insertedほか奇妙なマクロエラーが出る。jlreqのバージョンによっても違ってるかんじ。(ちゃんと追跡すること)
  • minipage<y> を使えないとなるとかなりたいへんそう。\yoko で書くと全然思い通りにならない…。
  • 縦モードだとfrontmatterのところでノンブル1の空ページが入る。review-jlreqで \jlreqsetup{frontmatter_pagebreak={}} としてみたがこれで正しいか確信がない。
  • 1章が偶数で始まるのは正しいのか?

@kmuto
Copy link
Owner Author

kmuto commented Sep 6, 2021

判断はiftdirのほうがよさそう?

@munepi
Copy link
Contributor

munepi commented Sep 6, 2021

PR #1738 をmergeすると、

      \if@tate
        \expandafter\includefullpagegraphics\expandafter[angle=90,\review@coverimageoption]{\review@coverimage}
      \else
        \expandafter\includefullpagegraphics\expandafter[\review@coverimageoption]{\review@coverimage}
      \fi

は、

      \expandafter\includefullpagegraphics\expandafter[\review@coverimageoption]{\review@coverimage}%

でかまいません。

@kmuto
Copy link
Owner Author

kmuto commented Sep 7, 2021

  • TeXLive2019 + jlreq 2019/1/15では縦書きはエラーで動かない。TeXLive 2021+jlreq 2021/3/17では動作。縦書き自体experimentalなので新jlreqのみでの動作というのでOK(なお新jlreqはTeXカーネルも新しくないとだめなのでTeXLive2019では×)
./__REVIEW_BOOK__.tex:104: Incompatible direction list can't be unboxed.
\@EveryShipout@Hook ...vbox \@cclv \unvbox \@cclv 
                                                  \else \hsize =\wd \@cclv \...
l.104 \reviewcoverpagecont
  • 縦書きにしたときにコラム枠、コードアミは全然違う位置になる。縦書きでこれらを求めるのは酷なのでこのままか
  • syntax-bookでテストしたところ、「コラム内のemlist」でMissing } insertedエラーになる

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