fix(ssr): ignore empty text vnode when hydrating (fix #11109) #11111
Conversation
|
for GitHub: this closes #11109 |
| const child = children[i] | ||
| // ignore empty text vnode | ||
| if (isUndef(child.tag) && isFalse(child.isComment) && child.text === '') { | ||
| continue |
There was a problem hiding this comment.
This results in child.elm === undefined and causes a crash in next patch.
Reproduction: deqwin/vue@fix_ssr_hydration...contribu:fix_ssr_hydration_crash
I think it is better to treat empty string nodes like #5117
There was a problem hiding this comment.
The fact is not that the vnode tree has an extra empty text node, but the DOM tree is missing a necessary empty text node.
There was a problem hiding this comment.
@posva I have modified the code to deal with this problem and also updated the test.
There was a problem hiding this comment.
Thanks!
I am unsure if this causes reflow during hydration and if the existing hydration code allows it.
I think it shouldn't cause reflow if the existing hydration code disallows reflow.
There was a problem hiding this comment.
Yeah, it‘s worth further studying. However, it is necessary to keep the vnode structure and DOM structure consistent at all times. So I think it's ok to do this step here.
There was a problem hiding this comment.
Another solution is rendering empty text nodes as comment nodes in SSR like async placeholder or v-if="false".
I am unsure which solution is better.
Co-Authored-By: Eduardo San Martin Morote <posva@users.noreply.github.com>
…t patch when hydrating
|
Hi All, is there any update on this PR being merged? This is causing us issues in our of SSR apps |
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:
devbranch for v2.x (or to a previous version branch), not themasterbranchfix #xxx[,#xxx], where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Other information:
Closes #11109