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

//graphに4つめの引数を導入する? #1750

Open
kmuto opened this issue Oct 6, 2021 · 0 comments
Open

//graphに4つめの引数を導入する? #1750

kmuto opened this issue Oct 6, 2021 · 0 comments

Comments

@kmuto
Copy link
Owner

kmuto commented Oct 6, 2021

#1749 に関連

//imageは3つめ引数としてビルダに渡すオプションをとり、たとえばこの中に latex::パラメータ の書式でreviewincludegraphics(実体はincludegraphics)に引き渡すオプションを指定できる。
これに対し//graphも最終的にはreviewincludegraphicsになるのだが、ビルダ渡しオプションがないので、オプション指定ができない。

//graphに4つめオプションを導入すべきか。

Cons

  • //image同様にオプション指定ができ、細かく制御しやすい

Pros

  • そもそもの latex::パラメータ という記法がほかの|builder|と違うので統一感がなく嫌めなところがある
  • オプション文字列のパースが困難で、現在の適当なパースだと指定できないか、誤った解析が引き渡される可能性がそこそこあり得る
  • かねてからのRe:VIEWのオプション指定記法の検討方向によってはobsoleteになりそうなものを新規導入すべきか逡巡する。(ただオプション指定記法はまだ悩み中で、かつ構文解析器を実装できないと進まない)
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

1 participant