Bug report: excessive spacing in live preview

[chrisgrieser]

What happened?

using dv.paragraph or dv.span in live preview results in excessive spacing, that does occur on and off, but is reliably reproducible in the help vault where only dataview has been installed. see this video.

Screen.Recording.2021-12-27.at.23.31.42.mov

DQL

No response

JS

dv.paragraph ("- one two three\n- one two three\n- one two three\n- one two three\n- one two three\n- one two three\n- one two three\n- one two three");

Dataview Version

0.4.21

Obsidian Version

Obsidian 0.13.14

OS

MacOS

[whitenoisedev]

I noticed the same.
Also, in tasks, the distance between the checkbox and the text is much narrower in LP than in reading.

[blacksmithgu]

I'm just guessing, but LP may be using different HTML elements / CSS classes than the old renderer did. I'll see what I can do to fix it; I'm probably going to just focus on live preview support in the future over the legacy renderer now that they have feature parity.

[valentine195]

This is how I mostly solved it in Admonitions.

https://github.com/valentine195/obsidian-admonition/blob/ebc55b1a3454e7b0cc2949e3278bcc0f91d4a9f0/src/assets/main.css#L273

[blacksmithgu]

Why is there so much spacing by default on paragraph elements? That's kind of crazy.

[blacksmithgu]

The line-height property of live preview paragraphs is set to a very large amount - possibly the culprit?

[valentine195]

The line-height property of live preview paragraphs is set to a very large amount - possibly the culprit?

Yes, a lot of styles are being inherited from the dynamic CM6 classes. Lists are especially bad, from a rogue white-space: pre-wrap.

I also found that the <br>s inserted in LP were rendering.

In general, the CSS I linked to in Admonitions made rendered markdown look the same as it does in Reading mode for me.

[blacksmithgu]

Messing too much with this appears to mess up table/list rendering, but I've applied a similar fix to paragraphs and spans to make them a little less painfully separated. I'll have to fine tune this more later once I understand live preview spacing better.