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

無効化したクラスオプションについてのヘルプ #1155

Closed
wants to merge 1 commit into from

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Oct 16, 2018

#1152 に関連して、

review-jsbook.clsで無効化したjsbook.clsのクラスオプションについて、エラーをもう少しだけわかりやすくしてみました。

Document class option #1: not available on review-jsbook.cls.#2 **
  • 紙サイズ: #2は「 Use paper= option.」
  • フォントサイズ: #2は「 Use Q= or pt= option.」。ptオプションはまだないけど…
  • winjis,mingoth: これはヘルプ出してもしょうがないので#2は空

ただ、ClassErrorがここで即止まってくれるわけではなくて、後続の評価が続いてかなり流れてしまい、本来のエラー原因を把握しにくくなってしまいます。即時停止ってできるのかな?

@munepi
Copy link
Contributor

munepi commented Oct 16, 2018

jsbook.clsのクラスオプションを極々一部を除きpassするようにすれば、errorよりもinfoで良さそうです。

  • 用紙サイズは、そのままpassすればOK。
  • 本文文字サイズは、(\magを使わない形で)互換で動くようにscalerを与え、もっとも混乱の元となる \mag を使わない形で提供するのが望ましいと思います。
    • 文字サイズは級数ではなくptで指定したい #1151 で言及したように、そもそもjsclassesもjclassesもクラスオプション XXpt は、「本文の文字サイズが実寸XXpt」でないです。
    • なので、本文文字サイズ fontsize=<単位つき> で与えたときは、実寸で動くようにしたほうが適切。
    • 行間 line_height=<単位つき> も同様に。

@kmuto
Copy link
Owner Author

kmuto commented Oct 16, 2018

passは自分でもできそうだけど、単位つきの実装が ><

@munepi
Copy link
Contributor

munepi commented Oct 16, 2018

実際には、単に pass するだけじゃだめです。それから、相対フォントサイズのコマンドも全部 jsbook.cls 互換に戻したりとか合わせてやる必要があります。
review-jsbook.clsの仕様や機能強化を 3.x の途中で変えて良いですか?

@kmuto
Copy link
Owner Author

kmuto commented Oct 16, 2018

月末にpre外した正式リリースなので、今のうちならフォントサイズを完全に合わせるのを含めてできますね。そのあとだと難しくなりそう。

@kmuto kmuto mentioned this pull request Oct 16, 2018
13 tasks
@kmuto kmuto mentioned this pull request Oct 28, 2018
24 tasks
@kmuto
Copy link
Owner Author

kmuto commented Nov 15, 2018

#1200 である程度まとまっているので、これ以上は再度作ったほうがよさそう

@kmuto kmuto closed this Nov 15, 2018
@kmuto kmuto deleted the obsoletejsopt branch November 15, 2018 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants