Inconsistent internal onebox spacing

When two internal links are added to a topic the spacing between them is inconsistent. If no blank line is between the links the spacing is significantly greater than if a blank line is provided.

https://meta.discourse.org/t/discourse-2-0-0-beta4-release-notes/82446
https://meta.discourse.org/t/discourse-2-0-0-beta5-release-notes/83881
https://meta.discourse.org/t/discourse-2-0-0-beta4-release-notes/82446

https://meta.discourse.org/t/discourse-2-0-0-beta5-release-notes/83881

cc @awesomerobot

「いいね!」 7

Oddly enough, when there’s no line between the two we add a <br> to the markup and when there is a line between the two we don’t.

「いいね!」 4

Hmm, what do you recommend to fix?

Ah, I understand why this happens now (bear with me here)…

If I type:

word
word

…I intentionally put a single carriage return there, so it correctly renders as:

<p>
    word
    <br>
    word
</p>

When I type:

word

word

…this is two returns, so the markup is correctly output as two paragraphs:

<p>
    word
</p>
<p>
    word
</p>

But with oneboxes, we’re treating them as they’re input (two strings with a single carriage return) and not as they’re output (two separate HTML elements)…

For example, when I type:

https://meta.discourse.org/t/discourse-2-0-0-beta4-release-notes/82446
https://meta.discourse.org/t/discourse-2-0-0-beta5-release-notes/83881

…the output is:

<aside>
</aside>
<br>
<aside>
</aside>

This is technically correct for the input. I entered two lines with a carriage return between them… but I was required to put that carriage return there for the oneboxes to exist. We shouldn’t count that carriage return because it’s a prerequisite to generate oneboxes. I’m not doing it for the purpose of creating a new line of text (so there shouldn’t be a <br> there).

When I type:

https://meta.discourse.org/t/discourse-2-0-0-beta4-release-notes/82446

https://meta.discourse.org/t/discourse-2-0-0-beta5-release-notes/83881

…the output is:

<aside>
</aside>
<aside>
</aside>

This would be correct if they were two paragraphs, which is how the text is input, but not how it’s output. We should add a <br> here, because I’m adding a space that doesn’t need to be there (intentionally dividing the content).

「いいね!」 10

Still an issue apparently :slight_smile:

「いいね!」 3

この件を再度持ち出すことになり申し訳ありませんが、新しいトピックを開始する代わりに、このトピックに便乗できると考えました…

これはよく知られていることだと思いますが、onebox の内部リンクは、展開すると見栄えが良いですが、デフォルトでは書式設定がありません。

展開前…

展開すると見栄えが良くなります。

OP がテキストを引用しない限り、プレビューは書式設定を保持すべきではありませんか?それとも、できるだけ多くのテキストをボックスに収めようとしていますか?

それとも、私が何かを見逃していて、これらすべてが意図したものでしょうか?

はい、展開前にフォーマットはありません。そうしないと、そこに多くのテキストを収めることができません。

「いいね!」 2

なるほど、それはしばらく前からそうだったのでしょうね。

IMHO、UIの観点からすると、スクラッチされたバージョンではなく、どの投稿に入っているのかを見る方が「すっきり」するのではないでしょうか。

投稿のOPが特定のテキストを強調したい場合は、引用すればよく、それは素晴らしい見た目になります。

投稿をリンクしている場合、投稿をクリックしてほしい、あるいは少なくとも展開して文脈を得てほしいと思っているのかもしれません。

純粋な数字ゲームからすると、適切なフォーマットの投稿をクリックしたり展開したりする人が増えるだろうと想像します。見た目が楽になりますからね。

ディスコースユーザーの大多数はコーダーやプログラマーではないことを忘れないでください。そのような箱にテキストが詰め込まれているのを見るのは、彼らが慣れていることではありません。

…あくまで意見です。

「いいね!」 1