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

refactor //texequation, //embed and //graph #1371

Merged
merged 4 commits into from
Aug 20, 2019
Merged

Conversation

takahashim
Copy link
Collaborator

@takahashim takahashim commented Aug 17, 2019

  • arg lines should not have "\n", like other tags
  • should not use LATEXUtils::unescape; lines should not be escaped in //texeuqation and //embed and //graph

@takahashim takahashim changed the title refactor //texequation and //embed refactor //texequation, //embed and //graph Aug 17, 2019
* arg `lines` should not have "\n", like other tags
* should not use `LATEXUtils::unescape`; `lines` should not be escaped in `//texeuqation` and `//embed`
@takahashim
Copy link
Collaborator Author

各builderのブロックの内容は lines という引数(1行1StringのArray)で与えられていますが、その行末に改行文字がある場合があったので、削除しています。
また、ブロックの内容についてインラインの記法を許さないものは texeuqation, embed, graph ですが、これらについてエスケープを適切に扱うよう修正しました。そのため、builderでのunescapeのめぼしいものを削除しています。

これらを再定義したり、エスケープ周りをいじっているreview-ext.rbがあると不具合が出る可能性があります。

#1374 に続きます)

@kmuto
Copy link
Owner

kmuto commented Aug 19, 2019

よさそうな感じです。
この範疇だと、review-ext.rb改変障害は無視してもよいのではという気持ち。

@takahashim
Copy link
Collaborator 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