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

Renumbering: Update refereces of chapters and sections #7

Merged
merged 1 commit into from
Dec 23, 2017

Conversation

takahashim
Copy link
Contributor

#2 の方の章や節の番号を@<chapref>{}@<hd>{}の参照に置き換えました。章の参照についてはPDFでも文書内リンクになっているはずです。

#2の方はこれでよいとして、#1については「本章」などの文言については見直しが必要そうです。とりわけ、intro.reのリード部分については文章を直した方が良いように感じました。

@takahashim
Copy link
Contributor Author

なお、例えば@<chapref>{main_intro}第2章「Binderの本格的な入門」という文言に置き換えるのですが、章タイトルも表示すると冗長な場合もあるかと思います。そのような場合は@<chap>{xxxx}にすると第2章といった章番号のみの文言になります。
このPRについても後者に置き換えた方がよいところもあるかもしれません。

@karino2
Copy link
Owner

karino2 commented Dec 23, 2017

おぉ、ありがとうございます。
とりあえずこのPRの差分をチェックする事から始めたいと思います。

@karino2
Copy link
Owner

karino2 commented Dec 23, 2017

半分くらい見た所ですが、現時点では、冗長な分には良いかな、と思います。
とりあえず読んで明らかにおかしい所を潰した後で、そういった体裁的な所を改善する、と段階的にすすめた方が良い気はします。

introのリードくらいは、せっかくなのでこの機会に直してしまおうかと思います。

@karino2 karino2 merged commit 2e541fb into karino2:master Dec 23, 2017
@karino2
Copy link
Owner

karino2 commented Dec 23, 2017

ちなみにpdfで、各ページの上の所に出ている章のタイトルみたいなのが現在長すぎて同じ場所に文字が重なってしまっていて読めない状態だと思うのですが、これって直し方とか分かったりしますか?

@takahashim takahashim deleted the fix-2 branch December 23, 2017 16:32
@takahashim
Copy link
Contributor Author

ヘッダの重なりの対処については、

  • 単純に章タイトルのみ表示するようにして節タイトルは表示しないことにする
  • 紙の本のように偶数ページに章タイトル、奇数ページに節タイトルを表示する

のどちらかが無難なのですが、どちらがいいでしょうか?

@karino2
Copy link
Owner

karino2 commented Dec 23, 2017

書籍の方は後者(偶数ページに章、奇数ページに節)となっているので簡単ならそれにしたい気がします。

ただ章タイトルだけても許容範囲と思うので、難易度が大きく違うなら章タイトルだけで良いと思います。

@takahashim
Copy link
Contributor Author

なるほど、難易度はたいして変わらないので後者にしてみます

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