rimmel icon indicating copy to clipboard operation
rimmel copied to clipboard

Use tree walkers to list and process changed nodes in a DOM subtree

Open dariomannu opened this issue 2 months ago • 5 comments

When Rimmel mounts an element to the DOM, a MutationObserver is called to activate all data bindings.

We want to switch to using document.createTreeWalker() instead of filtering through a mutationList as per existing TODO1 and TODO2 comments, which should bring a performance improvement for free.

dariomannu avatar Oct 12 '25 20:10 dariomannu

Hey @dariomannu, Can you please assign this issue to me.

the-matrixneo avatar Oct 13 '25 18:10 the-matrixneo

Hi @the-matrixneo sure, that's all yours!

dariomannu avatar Oct 13 '25 18:10 dariomannu

Hi @the-matrixneo how are you getting on? Any progress on this issue, by any chance?

dariomannu avatar Oct 28 '25 11:10 dariomannu

Hey @dariomannu! ,

My apologies for the delay on this. I got pulled onto a few higher-priority work but I'm back on it now. I will raise the merge request.

Thanks for your patience!

the-matrixneo avatar Oct 28 '25 15:10 the-matrixneo

Hey @dariomannu! ,

My apologies for the delay on this. I got pulled onto a few higher-priority work but I'm back on it now. I will raise the merge request.

Thanks for your patience!

are u still working on this?

yashraj4 avatar Nov 05 '25 05:11 yashraj4