undom icon indicating copy to clipboard operation
undom copied to clipboard

Support innerHTML in serialization recipe

Open Stanback opened this issue 8 years ago • 1 comments

I understand you don't want to support serialization in this project which makes sense. However, when experimenting with the example recipe I've noticed an issue when serializing the document after Preact mutates it. I end up with elements that have both innerHTML and childNodes[] defined. The innerHTML value seems to have not been cleaned up. I'm not sure where or why this is happening exactly. Any thoughts?

I've added innerHTML and void elements to my implementation here: https://gist.github.com/Stanback/3bb0b19b299668ce0e08922a8ab6876e

Stanback avatar Apr 01 '17 17:04 Stanback

Ah - good call, the serializer should at least support innerHTML. FWIW I have been meaning to make undom pluggable for some time now - the thoughts were outlined here: https://github.com/developit/undom/issues/3#issuecomment-233199487

One benefit of this would be that we could take your solution and bundle it in as a undom/plugins/serialization plugin.

developit avatar Apr 03 '17 01:04 developit