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だと偶数ページに隠しノンブルが入らない #1395

Closed
m-shibata opened this issue Sep 10, 2019 · 7 comments
Closed

Comments

@m-shibata
Copy link
Contributor

review-jlreqでhiddenfolioを指定すると、偶数ページにのみ隠しノンブルが挿入されないようです。
uplatex/lualatexどちらでも発生しました。review-jsbookだと問題ありません。

再現手順

review-init --latex-template=review-jlreq test
cd test
sed -i 's/^\(texdocumentclass.*\)"]$/\1,hiddenfolio=nikko-pc"]/' config.yml
rake pdf

備考

たとえばreview-jlreq.clsの\@bannertokenを次のように変更しても、奇数ページにのみtestが表示される状態でした。

\@bannertoken{test}%

そのためjlreq-trimmarks.sty側かその使い方の問題かもしれません。

試しにhiddenfolioは指定せずに、review-custom.styで以下のように記述すると、偶数ページにも隠しノンブルが表示されました。

\if@pdftombo
\jlreqtrimmarkssetup{
  banner={
    bottom-left={
      in-yoko={
        odd={\thepage}
      }
    },
    bottom-right={
      in-yoko={
        even={\thepage}
      }
    }
  }
}
\fi
@m-shibata
Copy link
Contributor Author

環境が抜けていました。

  • Re:VIEW: masterのHEAD
  • TeX Live: Ubuntuパッケージ版のTeX Live 2018
  • jlreq: abenori_devブランチの最新

@kmuto
Copy link
Owner

kmuto commented Sep 11, 2019

confirmed. はてさて…

@kmuto
Copy link
Owner

kmuto commented Sep 11, 2019

なるほど、jlreq-trimmarks.styの仕様が変わって、だいぶ複雑になっていますね。
これまでのbannerマクロ定義だとoddページにしか配置されなくなっている、と。

sty/review-jlreq.cls の70行目に以下のように\jlreq@trimmarks@banner@even@yoko@top@leftを入れるとどうでしょう?(いささか泥縄ですがjlreq-trimmarks.styを見るかぎりではこれでいいはず…)

  %% redefine to output \@makehiddenfolio for every page
  \@bannertoken{\hskip-5mm\smash{\hiddenfolio@font\@makehiddenfolio}}%
  \def\jlreq@trimmarks@banner@even@yoko@top@left{\hskip-5mm\smash{\hiddenfolio@font\@makehiddenfolio}}%

@m-shibata
Copy link
Contributor Author

\jlreq@trimmarks@banner@even@yoko@top@leftを入れるとどうでしょう?

ありがとうございます。試してみたところuplatex/lualatexともにうまく表示されるようになりました。

ところで「review-jlreq.clsでjlreq-trimmarks.styのbannerではなく\@bannertokenを使わなければいけない理由」は何かあるのでしょうか。review-jsbook.clsの実装に合わせただけで、bannerではダメな理由は特になさそうというのが想像なのですが、それで合っていますでしょうか。

(手元の原稿ではbannerの機能で十分だったのですが、実は\@bannertokenのほうが良かったのかと不安になっただけです)

@kmuto
Copy link
Owner

kmuto commented Sep 11, 2019

順序が逆で、

  1. review-jsbookでhiddenfolioを実装。review-jlreq+当時のjlreq-trimmarksも同じ実装で動いていた
  2. それからしばらくしてjlreq.cls開発者のabenoriさんと「隠しノンブルなにそれ怖い」という話をした結果、「こんなのどうよ」とjlreq-trimmarksのbannerが書き直された
  3. これにより1.の実装が動かなくなった

なのです。
jreq-trimmarksに合わせるのがいいんですが、だいぶ実装と設定方法が違うので、新実装は今はすぐにはできないです(patch welcome)。とりあえず暫定対処で上記のを入れておこうかと思います。

@m-shibata
Copy link
Contributor Author

「こんなのどうよ」とjlreq-trimmarksのbannerが書き直された

なるほど、bannerの登場があとだったんですね。

とりあえず暫定対処で上記のを入れておこうかと思います。

よろしくお願いいたします。banner版は、自分でできそうだったら折りを見て別途PRを立ててみます。

@munepi
Copy link
Contributor

munepi commented Sep 11, 2019

hiddenfolio=<profile> を書くと、 jlreq-trimmarks に合わせた <profile> を与えるようにする必要がありそうですね。

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