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

//noteなどの囲み要素内で末尾に空行があると空の段落が作成されてしまう #882

Closed
kmuto opened this issue Dec 11, 2017 · 7 comments

Comments

@kmuto
Copy link
Owner

kmuto commented Dec 11, 2017

//noteなどの囲み要素について、md2review変換やコメントを入れていたときに末尾の空行が空段落として生成されてしまう。
特にHTML, IDGXMLで顕在化する。

split_paragraphで末尾の空行群を無視したい。

@takahashim
Copy link
Collaborator

仕様についての疑問なんですが、

//emilst{
aaa
bb
c

//}

となったときに、最後の空行って勝手に削除されるべきではないと思ったんですが、そうでもないですか?

それはさておきご指摘の症状はmd2reviewのバグかもですが。

@takahashim
Copy link
Collaborator

それとも//note//emlistの挙動を変えるべき、みたいな話なんでしょうか。
いずれにしてもなんでsplit_paragraphの話になるのかがわかりませんでした。

@kmuto
Copy link
Owner Author

kmuto commented Dec 12, 2017

//emlistのほうで消しちゃうのはバグな気がしています。

//noteなどのほうは空段落になると困るので、そっちは取りたい。split_paragraphである必要はないのですが、//note側のロジックを見ていてほかに一括で直せそうなかんじがしなかったのでした。blockでうまくできるところあります?

@takahashim
Copy link
Collaborator

takahashim commented Dec 12, 2017

あー、囲み要素というのはブロック全般じゃなくてcaptionblockとかのことだったりするんでしょうか?

@kmuto
Copy link
Owner Author

kmuto commented Dec 12, 2017

おっと、そうですね。対象は、note, memo, tip, info, warning, important, caution, notice, quote, bibpaper, lead かな。

@takahashim
Copy link
Collaborator

そうだったんですね。なんか勘違いしてて失礼しました。じゃあsplit_paragraphの挙動を変えるべきなのかな…。

@takahashim
Copy link
Collaborator

いずれにしても現状のPR #883は、split_paragraph(lines)を実行すると呼び出した側のlinesの値が勝手に書き換わる(ことがある)仕様なので、修正は必要そうです。

@kmuto kmuto closed this as completed in 7162c9f Dec 12, 2017
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