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

Line height for html elements fix still necessary? #2519

Open
Graywaren opened this issue Feb 6, 2025 · 0 comments
Open

Line height for html elements fix still necessary? #2519

Graywaren opened this issue Feb 6, 2025 · 0 comments
Labels
bug Something isn't working.

Comments

@Graywaren
Copy link

What happened?

Hi
Is this fix still necessary with the current version of obsidian? I discovered it when I needed to add css to override/unset it recently to get my html elements displaying correctly again. I'm uncertain if I removed different css in my snippet that had unknowingly been compensating for it or if the latest obsidian update changed the rendering so that it's no longer necessary. I believe it was originally added for this bug

/** Live Preview padding fixes, specifically for DataviewJS custom HTML elements. */
.is-live-preview .block-language-dataviewjs > p, .is-live-preview .block-language-dataviewjs > span {
    line-height: 1.0;
}

DQL

No response

JS

Dataview Version

0.5.67

Obsidian Version

1.8.4

OS

Windows

@Graywaren Graywaren added the bug Something isn't working. label Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working.
Projects
None yet
Development

No branches or pull requests

1 participant