isKeyed test appears to be wrong for qwik
Opened this issue · 4 comments
The isKeyed test claims that qwik is non-keyed. Adding a background-color on the tr shows that the dom state is properly in sync with the data for create, remove and swap (or the external state is nicely moved?)
npm run isKeyed -- --headless true keyed/qwik
Keyed test for create rows failed. Expected that 1000 TRs should be removed and added, but there were 0 added TRs and 0 were removed
Keyed test for remove failed. Expected that the dom node for the 2nd row would be removed, but it wasn't
qwik-v1.2.17-keyed is non-keyed for 'run benchmark' and non-keyed for 'remove row benchmark' and keyed for 'swap rows benchmark' . It'll appear as non-keyed in the results
ERROR: Framework qwik-v1.2.17-keyed is not correctly categorized
Until resolved we'll add qwik as keyed. Any ideas why the test fails?
Qwik replaces the whole table when replacing all rows. No other implementation came up with that solution so far.
And it appears to keep removed elements in a template tag. isConnected is false then, but the tr is still in a template element.
No other implementation came up with that solution so far
so is that good or bad 😆
so is that good or bad 😆
it depends, for benchmark purposes it is a clever cheat, for real-world applications it is an awful feature/behavior.