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

SSR/DOM whitespace discrepancy #3168

Closed
Rich-Harris opened this issue Jul 3, 2019 · 7 comments
Closed

SSR/DOM whitespace discrepancy #3168

Rich-Harris opened this issue Jul 3, 2019 · 7 comments

Comments

@Rich-Harris
Copy link
Member

Just noticed this annoying flicker on the homepage:

server rendered

Screen Shot 2019-07-03 at 09 22 17

hydrated

Screen Shot 2019-07-03 at 09 22 31

@phillipmalboeuf
Copy link

@Rich-Harris I've been having the same SSR whitespacing problem. I'd like to set a white-space: pre-line; for almost everything on my site. But the component's svelte server render function keeps the original source file's whitespacing. My annoying flicker: https://phils.computer/

@Conduitry Conduitry added the bug label Jul 5, 2019
@Conduitry
Copy link
Member

use_space() in the Text wrapper node should probably get moved into the Text node itself, so it can be used also when generating SSR output.

@Conduitry
Copy link
Member

Conduitry commented Jul 5, 2019

Huh I see that some of this was changed in #3102 and it is precisely that test that is failing now when I try to implement what I described in my previous comment.

@stale
Copy link

stale bot commented Jun 27, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale-bot label Jun 27, 2021
@stale
Copy link

stale bot commented Jul 11, 2021

This issue has been closed as it was previously marked as stale and saw no subsequent activity.

@coyotte508
Copy link

The bug still happens, I think?

@peterolson
Copy link

It still happens in Svelte 4, seems to be fixed in Svelte 5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants