QwikDev / qwik

Instant-loading web apps, without effort
https://qwik.dev
MIT License
20.85k stars 1.31k forks source link

[🐞] v2: doesn't handle missing/added fragments well #7033

Closed wmertens closed 1 week ago

wmertens commented 3 weeks ago

Which component is affected?

Qwik Runtime

Describe the bug

when vnodes are tree shaken, or a component returns a fragment where there wasn't any, the consolidation of jsx output vs existing vnodes will recreate existing vnodes instead of moving them inside the fragment.

Reproduction

in varixo's head ;)

Steps to reproduce

No response

System Info

v2

Additional Information

No response

Varixo commented 1 week ago

fixed by https://github.com/QwikDev/qwik/pull/6963