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

ins, del命令のエラーと表現対応 #1630

Closed
kmuto opened this issue Dec 31, 2020 · 6 comments
Closed

ins, del命令のエラーと表現対応 #1630

kmuto opened this issue Dec 31, 2020 · 6 comments

Comments

@kmuto
Copy link
Owner

kmuto commented Dec 31, 2020

  • insを入れるとWebMakerやTextMakerで失敗する。plaintextbuilderにinline_insが実装されていない。WebMakerは目次採取にこのビルダを使っているためのエラー。
  • TeXでinsに表現が適用されていない。HTMLではアンダーラインがデフォルト表現みたい。reviewinsertマクロを追加し、中身のデフォルトとしては下線にした。jsbookではjumolineなので単なる下線、jlreqではulemを使っているので二重下線にしてみた。
  • TeXでdelの表現をあえて無視していた。しかしjsbookではjumolineをコピーしているし、jlreqではulem前提にしているので、それぞれの打ち消し線を使うように設定する。
  • plaintextbuilderでdelを空文字にしているが、delは「表示はするが打ち消し表現をする」という意味なので、空文字にしてしまうのはよくない。

最後のは挙動としては互換性を壊すが、どっちにしろins/delは現時点ではほぼ使われていないと思うので変えて問題なさそう(pandoc2review由来でdelは今後よく入ってくる可能性)。

@kmuto
Copy link
Owner Author

kmuto commented Dec 31, 2020

ins/delはあったけど明文化していなかった裏コマンドだった模様…。表コマンドにします。

@takahashim
Copy link
Collaborator

plaintextbuilderは@<del>は空文字にするのでいいと思いますが……(制御文字とかは使わない方が良さそうなので)。

@kmuto
Copy link
Owner Author

kmuto commented Dec 31, 2020

delって「出力はするけど打ち消し表示したい」という意図なのではないかと思うんですよねぇ。
plaintextbuilderのほうは端末制御文字などは使ってないですね、そのままの出力です。

@takahashim
Copy link
Collaborator

うーん、まあ何にせよ原則プレーンテキストで表現できないものをプレーンテキストで表現するという無理筋な対応なので、どうするかは決めの問題かとは思いますが、私見では、

  • foofoo
  • @<ins>{foo}foo
  • @<del>{foo}foo

よりは、

  • foofoo
  • @<ins>{foo}foo
  • @<del>{foo}→ (空文字列)

の方が良さそうな気がしております。

@kmuto
Copy link
Owner Author

kmuto commented Jan 1, 2021

なるほど。まぁそれほどコダワリポイントでもないし、影響するのはplaintextbuilder (普通のtextとして使うtopbuilderのほうは削除擬似命令つきで出力してる) とそれを目次に使うwebmaker/review-volume/indexくらいなので、空にしちゃいますかね。

@kmuto
Copy link
Owner Author

kmuto commented Jan 1, 2021

本年もよろしくお願いいたします!

@kmuto kmuto closed this as completed in 7891aeb Jan 1, 2021
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

No branches or pull requests

2 participants