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

書籍用レイアウト/著者校正用PDF #1070

Closed
kahei opened this issue Jan 31, 2020 · 41 comments
Closed

書籍用レイアウト/著者校正用PDF #1070

kahei opened this issue Jan 31, 2020 · 41 comments

Comments

@kahei
Copy link
Contributor

kahei commented Jan 31, 2020

まだ第1部の途中ですが、ラフレイアウトをしたtexファイルとPDFをpublish1ブランチにあげました。まだいろいろレイアウトにも問題が残っていますが、気がついた点があればお知らせください。

@azu
Copy link
Collaborator

azu commented Jan 31, 2020

@azu
Copy link
Collaborator

azu commented Feb 6, 2020

HTML-likeコメント

<!--<!- になってしまっている。

3BDE692E-E3C2-4BC7-A93A-3B4268B94842

セクション名

右上のセクション名まで ES2015の上付きが作ると少し見た目のバランスが悪いかなと思いました。

C10DDAB0-2BB3-4167-8BAA-6A8041C31BA4

8E918433-C689-4274-8738-1898BA2EB15A

ES2015がついてるところとそうでないところがあるみたいな気がする。

nullのコラム

はみ出てる
// undefined という名前の変数をエラーなく定義できる のコメントは上に移動して大丈夫です。
2C450D90-F2C9-477D-BEB5-4BBD66072070

右シフト

単純にコードのコメントの書き方がよくない感じがします。
91EA5AAF-25A0-4E22-BD97-117DE23E4116

みたいに説明としてコメントを書くように書き換えちゃった方が見やすくなるかな。

console.log((-9) >>> 2); // => 1073741821
// 0b11111111111111111111111111110111 >>> 2 すると
// 0b00111111111111111111111111111101 となる

### コードコメントのバッククオート

コード中のコメントでバッククートをコードを意味する記号として使ってるけど、
Markdownのコンテキストがない人がみたら ` にもなにか意味があるようにみえちゃうかもしれない。

単純に取り除くか、最初に説明いれたほうがいいのかな?
67960D30-7068-4192-911E-9EA1A6539C3A

コードの %

コード中の % のフォントが妙に縦に細い感じがしますが、こんなものですかね?

3233FBD4-7395-48EB-9714-E70388643A91

エスケープシーケンス

https://jsprimer.net/basic/string/#escape-sequence
console.log("¯\_(ツ)_/¯"); がなんか変な描画になってますね。
7A9D49B0-303F-49F7-ABDD-7420732B7022

絵文字

絵文字って普通に出るんですね?
これモノクロになると思うのですが、モノクロで区別つくかな?
778926B5-A35C-4612-BFFA-566838D4FBFA

1 ES2015 の 仕 様 編 集 者 で あ る...

https://jsprimer.net/basic/function-this/ の一番最後がおかしくなってる。
制御文字が混ざっておかしくなってる?

FCBF9FE8-9C5F-44F3-99FC-37AC2951C271

SyntaxError

なんかここだけ文字幅が伸びている感じがする?

79328A99-BDC0-4415-AF41-598D56F0A45E

スクショ

文字サイズ感的に小さいので、もう少し拡大してとる方がいいかも
CC702FEF-A7F6-4439-B8E6-11AC99B89C89

@azu
Copy link
Collaborator

azu commented Feb 6, 2020

@kahei おおざっぱに一通り見てみました。

@azu azu assigned azu and lacolaco Feb 6, 2020
@kahei
Copy link
Contributor Author

kahei commented Feb 7, 2020

@azu どうもありがとうございます。それぞれ修正など入れますね。

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

目次
image
ESがイタリックになっている

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
これはまだラフだからですよね

@azu
Copy link
Collaborator

azu commented Feb 8, 2020

{% if .. %} のGitBookマクロ そういえば
https://github.com/asciidwango/js-primer/blob/master/tools/strip-markdown-extension.js では対応してなかったのか。

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
このサイズではコンソールが出ていることはわかりにくそう

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
リンクが壊れてる

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
これは普通に原稿のtypoかな?

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
「知らない」がよさそう

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

image
幅溢れ

@lacolaco
Copy link
Collaborator

lacolaco commented Feb 8, 2020

ざっくりひと通り見ました。幅溢れはこれからだと思うのでコメント控えます

@azu
Copy link
Collaborator

azu commented Feb 10, 2020

- `class`構文ではプロトタイプオブジェクトと参照する仕組みによって継承が行われています。
+ `class`構文ではプロトタイプオブジェクトを参照する仕組みによって継承が行われています。

#1072

@kahei このtypoの修正 publish1 ブランチのTexの方にどういう感じで取り込みます?

  • PRをマージ -> @azupublish1 にもコミットする
  • PRをマージ -> Texにも反映が必要ラベルをPull Requestに付けて管理する(Need Update Tex?とかのラベル管理) -> @kaheipublish1 に反映する

どっちがいいですかね?

@kahei
Copy link
Contributor Author

kahei commented Feb 12, 2020

azuさんにもpublish1にコミットしてもらったほうがいいと思うので、よろしくお願いいたします。

@azu
Copy link
Collaborator

azu commented Feb 12, 2020

publish1 ブランチに反映しました f7d600b

@kahei
Copy link
Contributor Author

kahei commented Feb 12, 2020

どうもありがとうございます!

@azu
Copy link
Collaborator

azu commented Mar 20, 2020

D7C332E6-C929-4131-A2DE-484E9527CC1B
ES2015ラベルがまだ目次に残ってますね

@azu azu pinned this issue Mar 20, 2020
@azu
Copy link
Collaborator

azu commented Mar 20, 2020

8AAB2D6C-C609-42C0-8E99-3B1CCB2E3A4F
F213FE19-3D82-478D-B5BA-CE21908F564D

[詳細] の部分になにか記号が混じってる感じがします
P21も同様

@azu
Copy link
Collaborator

azu commented Mar 20, 2020

FD6E5162-6997-4B62-82A2-748BAA219B38

シングルクオートのとなりにスペースがあるように見えるけど、これはフォントの問題?

@azu
Copy link
Collaborator

azu commented Mar 22, 2020

5ED3C477-AF86-4E5B-8896-268E79DF5849

const で定義したオブジェクトは変更可能のコラム 変なタイミングで割り込んでしまってるので調整が必要かもしれません。

@azu
Copy link
Collaborator

azu commented Mar 22, 2020

52146D43-2DEC-4081-A5E9-253145ABB8A7

オブジェクトのプロパティ名は文字列化される も同様

@azu
Copy link
Collaborator

azu commented Mar 22, 2020

9CB9A2C0-9202-4093-B53E-7F931C2C4D5B

プロトタイプオブジェクトの図 もへんな位置に割り込んでしまってるので、要調整な気がする。
(このイメージは概念的な図なのでGCと同じぐらいでそこまで必須という感じではなく、補助的な図のイメージ)

@azu
Copy link
Collaborator

azu commented Mar 22, 2020

A52EBDFC-28B3-4BD8-B742-AE22CC6D9347
ReferenceError: x is not defined
xの前になんか空間があるのはなんでだろ? justifyみたいなものがかかってるのかな

@azu
Copy link
Collaborator

azu commented Mar 24, 2020

12CAF2F6-30A6-41C6-BF4C-BDB32E24374F

Todoのtable中の文字がjustifyされてる感じがする

@azu
Copy link
Collaborator

azu commented Mar 24, 2020

03-17版を一通り読んで、レイアウト周りできになったところはこのissueに書きました。
https://github.com/asciidwango/js-primer/issues?q=is%3Aissue+is%3Aopen+label%3A%22Status%3A+PR+Welcome%22
文章的なところはissueを作ったのでべっと対応していきます

@azu
Copy link
Collaborator

azu commented Apr 4, 2020

image

ホッケが化けて出ている

@kahei
Copy link
Contributor Author

kahei commented Apr 6, 2020

@azu 私の環境(macOS)では、ホッケの文字がきちんと表示されます。azuさんの環境を教えていただけますか?

@kahei
Copy link
Contributor Author

kahei commented Apr 6, 2020

@azu フォントを埋め込んだ暫定PDF(main2.pdf)を作ってアップしてみました。こちらで再度確認してみてください。ホッケの文字が見えると思います。

@azu
Copy link
Collaborator

azu commented Apr 6, 2020

あー、こっちの環境依存でしたか。
化けてたのはpdf.jsの古いバージョン(自作のビューア)で、最新のpdf.js(Firefoxに搭載されてるもの)なら問題ありませんでした。
main2.pdfでもおかしいのでビューア側が変な感じがします。

image

他のビューアなら問題ありませんでした。(なので無視してもよさそう)

@kahei
Copy link
Contributor Author

kahei commented Apr 6, 2020

他のビューアで問題ないということなので、とりあえずこの問題は解決ということで。

@azu
Copy link
Collaborator

azu commented Apr 7, 2020

2F0CD8A4-E7C9-4874-9A23-6ED5A6BC1D72
次の表で紹介するメソッド\footnote{\url{https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/}}は破壊的なメソッドです。

次の表で紹介するメソッド\footnote{\url{https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/}}は破壊的なメソッドです。

このfootnoteって元からありましたっけ?
Markdown版にはないっぽい。不要そうなので削除してして良さそうな気がします。

@kahei
Copy link
Contributor Author

kahei commented Apr 7, 2020

これ、表内のメソッドがリンクされているURLから作ったfootnoteですね。不要なら削除します。

@azu
Copy link
Collaborator

azu commented Apr 7, 2020

あー、なるほど。そのままでも大丈夫です

@kahei
Copy link
Contributor Author

kahei commented Apr 7, 2020

では、このままで。
あと、文章を修正するissueは2つかな?

@kahei
Copy link
Contributor Author

kahei commented Apr 7, 2020

こちらのissueに挙げられているものは修正しました。いったん、pushします。

@azu
Copy link
Collaborator

azu commented Apr 7, 2020

image

絵文字なんとか区別できそうですね

@kahei
Copy link
Contributor Author

kahei commented Apr 7, 2020

たぶん、だいじょうぶだと思います。

@azu
Copy link
Collaborator

azu commented Apr 7, 2020

全部読み直して目についた所は直しました。
#1144 は誤記だったので修正

残ってる
#1134 #1140 #1142
は間違いではない/問題があるわけではないのでとりあえずはこのままで大丈夫

@kahei
Copy link
Contributor Author

kahei commented Apr 8, 2020

どうもありがとうございます。最終のレイアウト調整に入ります。

@azu azu unpinned this issue Apr 8, 2020
@azu
Copy link
Collaborator

azu commented Apr 27, 2020

出版したので、これは閉じる

@azu azu closed this as completed Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants