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

Howdownに変えたことでHTMLを混ぜたときの空白の扱いが微妙にかわったことについて #75

Merged
merged 1 commit into from
Mar 4, 2014

Conversation

gfx
Copy link
Collaborator

@gfx gfx commented Mar 3, 2014

#68 のコメントの返信です。

c46205d で一箇所テストを変えていますが、それはhoedownにしたら微妙に空白の扱いが変わったからなのでした。本来テストしたかったこと(divと未知のclassの組み合わせ)とは関係ないのでまあいいかーと思ったのですが、挙動はかわるのでよくなかったですね!

変わったところ:

  • 地の文(markdown)の直後に<div>...</div>が続いた時、地の文と空行の間のスペースが取り除かれる(Text::Markdownのときは空行が残ったままになる)

md2inao的には foo\nbar が foobar に変換される仕様なので、こっちのほうが正しいような気もしますし、記述言語(コンテキスト)が変わる以上空行が残るべきという気もします。編集的にはどういう挙動が望ましいですかね? (cc: @inao )

@gfx gfx mentioned this pull request Mar 3, 2014
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 0d8ec7b on newline-with-hoedown into 89a9aea on master.

@inao
Copy link
Collaborator

inao commented Mar 4, 2014

う、HTML時の挙動は、僕はよくわかってないです。
@naoya さん、いかがでしょうか?

<div>が独立した段落だけではなく、段落中にも登場するなら、0d8ec7b の挙動で合っている気がします。

@naoya
Copy link
Owner

naoya commented Mar 4, 2014

あ、いや意図を確認しただけで仕様上は問題ないと思います。
html 出力はそもそもほとんど利用がないので、影響も機微です。

@gfx
Copy link
Collaborator Author

gfx commented Mar 4, 2014

問題なさそうならじゃあ一旦この挙動でfixでー。

gfx added a commit that referenced this pull request Mar 4, 2014
Howdownに変えたことでHTMLを混ぜたときの空白の扱いが微妙にかわったことについて
@gfx gfx merged commit 556019a into master Mar 4, 2014
@gfx gfx deleted the newline-with-hoedown branch March 4, 2014 03:49
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.

4 participants