Removal of item re-renders the whole list instead of removing a particular node

[brijeshb42]

Here's a gif from Firefox devtools

The code that I used is here

I have taken care of generating unique ids for each list item. Ideally, removal of an item from array should just remove the corresponding node instead of re-rendering the whole list.

[jorgebucaran]

Is this related to #499? If so, I'd prefer to track it there. This is not a huge problem, but it's something I definitely want to tackle in 2018.

[jorgebucaran]

@brijeshb42 Nevermind. We are just reattaching event listeners, the DOM nodes are not recreated.

The reason the nodes are flashing is because the algorithm is changing their order with element.insertBefore inside patch. There's still room for improvement, however and I'd like to implement a more complex algorithm as described right here.

[jorgebucaran]

#499