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

通しノンブル #1129

Closed
wants to merge 53 commits into from
Closed

通しノンブル #1129

wants to merge 53 commits into from

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Oct 9, 2018

review-jlreq側に実装してみていますが、オプション渡しのところがうまくいかなくて悩んでいます。

\newif\ifrecls@serialpage \recls@serialpagefalse
\DeclareOptionX{serial_pagination}[false]{\csname recls@serialpage##1\endcsname}

でserial_paginationドキュメントオプション値(デフォルトfalse)を取り、serial_pagination=trueなら\recls@serialpagetrueになるはず(typeoutで見てもtrueで来てはいるっぽい)。

が、このあとの

\renewcommand*{\frontmatter}{%
 ...
  \ifrecls@serialpage

のほうで判定しようとすると、常にfalseになってしまう。何か評価順のところを間違っている??

kmuto and others added 30 commits October 1, 2018 07:50
(print): load gentombow package at the end of review-jsbook.cls
review-jsbook.cls: gentombow対応、\includefullpagebraphics対応、ちまちまbrush-up
disabled <papersize>j, <ptsize>pt options for jsbook.cls
(cameraready=ebook): fixed layout option/parameter against jsbook.cls
review-jsbook.cls: paper=<papersize>に対応
disabled <papersize>j, <ptsize>pt options for jsbook.cls
(cameraready=ebook): fixed layout option/parameter against jsbook.cls
munepi and others added 21 commits October 8, 2018 12:54
review-jsbook.cls (tombopaper,paperwidth,paperheight): supported
review-jsbook/review-base.sty (\footskip): dropped
This was referenced Oct 9, 2018
@kmuto
Copy link
Owner Author

kmuto commented Oct 9, 2018

serialpage判定は適当でいいかということでさくっと。startpageはfrontmatterにパッチするような形で設定

@kmuto
Copy link
Owner Author

kmuto commented Oct 10, 2018

reviewjsbookclsブランチに取り込み済み

@kmuto
Copy link
Owner Author

kmuto commented Oct 10, 2018

最初のPR時点じゃないとマージ先指定はいろいろ難しいのだな…。こっちはcloseしておきます

@kmuto kmuto closed this Oct 10, 2018
@kmuto kmuto deleted the serial_pagination branch October 10, 2018 08:53
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