ryansolid / dom-expressions

A Fine-Grained Runtime for Performant DOM Rendering
MIT License
885 stars 127 forks source link

Warn on unnecessary rerenders #138

Open milomg opened 2 years ago

ryansolid commented 2 years ago

Hmm... trying to figure out if this is an annoying message or not. Haven't vetted enough projects to know if this is common place. What drew your attention to this?

milomg commented 2 years ago

I was using a resource to store and update an object, but that was triggering an update on all fields on an object (and I would imagine there might be a similar issue with signals in beginner's code). I realize that this could be annoying if intentional though

milomg commented 2 years ago

I was also wondering what this looks like on common projects so I tried it on the playground and there were no warnings, but that isn't a perfect test by any stretch