Skip to content
Permalink
Browse files
fix(keep-alive): fix keep-alive rendering when placed in vnode branch
fix #4817
  • Loading branch information
mefcorvi authored and yyx990803 committed May 18, 2022
1 parent 3705b3b commit 0841b9b5243acdaf191099b25e9a145b30189dea
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
@@ -96,8 +96,11 @@ const KeepAliveImpl: ComponentOptions = {

// if the internal renderer is not registered, it indicates that this is server-side rendering,
// for KeepAlive, we just need to render its children
if (!sharedContext.renderer) {
return slots.default
if (__SSR__ && !sharedContext.renderer) {
return () => {
const children = slots.default && slots.default()
return children && children.length === 1 ? children[0] : children
}
}

const cache: Cache = new Map()
@@ -676,7 +676,7 @@ function testRender(type: string, render: typeof renderToString) {
render: () => h('p', 'hello')
}
expect(await render(h(KeepAlive, () => h(MyComp)))).toBe(
`<!--[--><p>hello</p><!--]-->`
`<p>hello</p>`
)
})

0 comments on commit 0841b9b

Please sign in to comment.