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

数式ブロックのインデントを見直す #1284

Closed
faithandbrave opened this issue May 26, 2024 · 17 comments
Closed

数式ブロックのインデントを見直す #1284

faithandbrave opened this issue May 26, 2024 · 17 comments
Assignees
Labels

Comments

@faithandbrave
Copy link
Member

sukeya commented 15 hours ago
こんばんは。
些細なことなのですが、LaTeXで書いた数式が左詰めになっているように見えてちょっと気になりました(例えばここ)。
個人的には引用くらいのスペースが前につくとより見やすくなると思いました(例えばここ)。

調べました。

MathJaxのこのへんの設定をいじれば、数式ブロック (displayMath) のインデントを変更できそうです。
cpprefjp/site_generator@94b9dcb

設定のドキュメントはこちら。

私の手が空いたら着手しようと思いますが、どなたか引き取れそうだったら引き取ってください。

@faithandbrave
Copy link
Member Author

@sukeya
直りました。
これくらいでどうでしょう?

@sukeya
Copy link
Contributor

sukeya commented May 26, 2024

素早い対応ありがとうございます!
見やすくなり、良いと思います。

@faithandbrave
Copy link
Member Author

ではこれで閉じますー

@yumetodo
Copy link
Member

@faithandbrave すみません、 https://polyfill.io/v3/polyfill.min.js?features=es6って何目的で読み込んでいる感じでしょう?

@akinomyoga
Copy link
Member

公式がそう指示しているから?

https://www.mathjax.org/#gettingstarted

じゃあ何故公式が polyfill を要求しているかというと MathJax v3 で ES6 の Promise を使いたくなったから:

mathjax/MathJax#3162 (comment)

@faithandbrave
Copy link
Member Author

はい、MathJax v3のドキュメントに従っています。

@yumetodo
Copy link
Member

yumetodo commented Jun 2, 2024

cpprefjpではIE11をサポートしていないので、
https://caniuse.com/promises
を見る限り、不要なのではないかと思うわけです。

ref: #477 (comment)

@faithandbrave faithandbrave reopened this Jun 3, 2024
@faithandbrave
Copy link
Member Author

たしかにIE11用っぽいので、削除しちゃってよさそうな気がしますね
https://docs.mathjax.org/en/latest/web/start.html#browser-compatibility

@akinomyoga
Copy link
Member

akinomyoga commented Jun 3, 2024

後で見た時に「なぜ MathJax 標準ではないロードの仕方をしているんだ」という疑問が生じるので、何故 polyfill 入れないのかコメントを入れるならそれで良いと思います (削除すれば無駄な traffic 減らせますし)。あと、IE11 対応しないとしても、後続の MathJax で余計なエラーが出ない様に IE 除外 できませんか? こんな感じでしょうか:

<!--[if !IE]><!-->
<script id="MathJax-script" async src="https://cdn.jsdelivr.net/npm/[email protected]/es5/tex-mml-chtml.js"></script>
<!--<![endif]-->

@faithandbrave
Copy link
Member Author

@yumetodo どうでしょう?

@yumetodo
Copy link
Member

yumetodo commented Jun 10, 2024

反応が遅れました。

#477 が議論された2017年には、まだIEは無視できないシェアがあり、故に「エラーが出ても最低限表示できるようにする」という方針がとられたのは妥当だったと言えるでしょう。

しかしIE11を無効化するKBが配布されるようになった今、IE11が動作する環境は、WindowsのサーバーOSか、LTSC版を用意する必要があり、(仕事柄職場に検証環境はありますが)、私物の検証環境を作って維持するのは大変です。また多くのユーザーはIEを回避する選択肢を持ち合わせているはずです。

そういうわけで、このissueからは外れてきてしまいますが、そもそもIEのサポートを明示的に切っておきたいです。


それとは別に、 @akinomyoga さん提案の条件付きコメントですが、リンク先にも記載のある通り、こちらはIE5-IE9まででサポートされた記法であり、IE11では単なるコメントとして認識されます。その他のブラウザでも同様です。したがって、今回のケースでIEでのみ読み込ませることを目的として条件付きコメントを使うことはできません。どうしてもそういうことをするならUser-Agentを見て出し分けるWebサーバーが必要になるでしょう。


以上の2点から、単にpolyfillを除去しておく、あるいは

<!-- IEはサポートしないので https://polyfill.io/v3/polyfill.min.js?features=es6 は読み込まない -->

等としておくのがいいのではないでしょうか(この場合、templateエンジン向けのコメント記法があればそっちのほうがbetter)。

@akinomyoga
Copy link
Member

IE を明示的に切るというと、IE で開いたら対応していませんページにジャンプするなどですか。

それとは別に、 @akinomyoga さん提案の条件付きコメントですが、リンク先にも記載のある通り、こちらはIE5-IE9まででサポートされた記法であり、IE11では単なるコメントとして認識されます。

本当ですね…見落としていました。

その他のブラウザでも同様です。したがって、今回のケースでIEでのみ読み込ませることを目的として条件付きコメントを使うことはできません。どうしてもそういうことをするならUser-Agentを見て出し分けるWebサーバーが必要になるでしょう。

GitHub Pages は対応していないような気がしますね。あるいは、userAgent 見て document.write(...) する (今どきはもっと良い方法がありそうですが…) こともできるかもしれませんが、それだと現行ブラウザでも余分の処理が走ることになるので嫌ですね。

等としておくのがいいのではないでしょうか

確かに現実的にはそんなところでしょうかね。

(この場合、templateエンジン向けのコメント記法があればそっちのほうがbetter)。

そうですね。

@faithandbrave
Copy link
Member Author

Microsoftがサポートを終了しているブラウザですから、正しく動かないのはIEユーザーの自己責任として、yumetodoさん案のコメントアウトで対応としては十分な気がしますね。がんばりどころではない気がします。

<!-- IEはサポートしないので https://polyfill.io/v3/polyfill.min.js?features=es6 は読み込まない -->

@yumetodo
Copy link
Member

yumetodo commented Jun 11, 2024

IE を明示的に切るというと、IE で開いたら対応していませんページにジャンプするなどですか。

あ、いえ、IEで壊れるかどうかを以降cpprefjpにおいて検討しないという意味合いでした。

@faithandbrave
Copy link
Member Author

5月のGoogle Analyticsのレポートを確認したところ、アクティブユーザー5.86万人のうち、IEユーザーは15人でした

@faithandbrave
Copy link
Member Author

では、コメントアウトしますね

faithandbrave added a commit to cpprefjp/site_generator that referenced this issue Jun 12, 2024
IE用だったが、IEは本サイトでサポートしないため
cpprefjp/site#1284
@faithandbrave
Copy link
Member Author

polyfillの読み込みをコメントアウトしました。
https://cpprefjp.github.io/reference/cmath/cos.html
数式は問題なく動いてそうです。

これで閉じます。
問題あったらまたreopenしてください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants