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

行結合 #1362

Merged
merged 8 commits into from
Oct 28, 2019
Merged

行結合 #1362

merged 8 commits into from
Oct 28, 2019

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Aug 13, 2019

言語に基づく行結合ルールの実装です。

  • textutils#join_lines_to_paragraphに行結合をまとめることで、結合の処理をしやすくはできました。
  • TODO: unicode/eawを内包したいのですが、どうするのがよさそうでしょうか。vendorに2.0版をコピーしてそこにLOAD_PATHを通す or エッセンスをtextutilsにコピーする?
  • TODO: textutils#add_space?にロジックを入れていますが、言語によってhttps://drafts.csswg.org/css-text-3/#line-break-transform のルール3を実行するかどうかというのはあまり意味がなさそうで、どの言語であろうとルール3は実施したほうがよさそうです。ただ、ルール3のとおりだとスペースが無駄に多くなることがあるので、lazyフラグを付けているそのあとのコード(現在デフォルト)のほうが良い結果が得られます。現状でよさそうなら、言語判定と旧ルール3は除去します。
  • 「英字でもスペースを入れずにつながる」ことを想定して段落中で改行していた場合、後方互換性がなくなります。ただTeXの場合はTeXコンパイラが入れてくれていたので、気付いていない可能性も。
  • フラグで後方互換性を持たせるのは、おおむねjoin_lines_to_paragraph内で対処できそうなものの、LaTeXBuilderだけは冗長になりそう。
  • test_textutilsの最後のテストで「OK?」としているケースについては、インラインタグの除去が難しく、タグ文字で英字扱いになってしまう箇所です。前後両方にインラインタグ囲みの和文が発生したときに問題となります、が仕様として諦める?

@kmuto kmuto requested a review from takahashim August 21, 2019 14:23
@kmuto
Copy link
Owner Author

kmuto commented Aug 21, 2019

これはかなり大きなbreaking changeになるので、入れるとすると4.0になってしまいますね。

@kmuto
Copy link
Owner Author

kmuto commented Aug 23, 2019

とりあえず3.3のままつっこむなら、experimental機能としてymlパラメータでフラグ作って(デフォルトnull)、trueだったらgem前提でrequire 'nicode/eaw'する、というようにしようかなと思っています。

@kmuto kmuto changed the title [WIP] 行結合 行結合 Aug 26, 2019
@kmuto
Copy link
Owner Author

kmuto commented Aug 26, 2019

  • フラグ join_lines_by_lang を導入しました。
  • フラグ有効時にbuilder.rbのinit内でunicode/eawライブラリの読み込みトライをするようにしておきました。
  • フラグfalse時は旧来結果になるようにしています。

@kmuto
Copy link
Owner Author

kmuto commented Aug 27, 2019

@takahashim とりあえずexperimental featureとしてはこれでどうですかねー

@kmuto kmuto mentioned this pull request Sep 16, 2019
@kmuto
Copy link
Owner Author

kmuto commented Oct 27, 2019

大々的推奨でもないし、review.gemspecには入れないほうがいいのかな。

@kmuto
Copy link
Owner Author

kmuto commented Oct 28, 2019

いくつかテストして問題なさそう(デフォルトfalseなので影響なさそう)なことを確認したので、マージしてみます。

@kmuto kmuto merged commit b025b9b into master Oct 28, 2019
@kmuto kmuto deleted the char-paragraph branch October 28, 2019 23:37
kmuto added a commit that referenced this pull request Oct 28, 2019
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.

1 participant