dom.js only uses createElementNS to create elements but that can't create HTML elements, only XHTML elements. My personal preference would be for Insert Node to create HTML elements. However the insert node dialog returns no namespace for HTML documents, so I can't even create XHTML elements as a workaround.
Created attachment 308816 [details] [diff] [review] Patch Patch. I replaced some tabs to spaces (|dom.js|) because I need to change indent level (|if| ~ |else|). # There are some tabs still.
(In reply to comment #1) >Created an attachment (id=308816) Hmm, so I think it might be better to move the enableNamespaces function into dom.js and pass it in as one of the attr parameters. > # There are some tabs still. Eww, who put them there :-(
Created attachment 308941 [details] [diff] [review] Patch v2 make |enableNamespaces| to just a property instead of a function.
Comment on attachment 308941 [details] [diff] [review] Patch v2 r=db48x, though if there are still tabs anywhere in the file you might as well go ahead and get rid of them (any decent editor can automate the task)
Comment on attachment 308941 [details] [diff] [review] Patch v2 Yes, the property looks much better, thanks!
I don't have cvs commit access. Could someone check this in the tree?
Checking in extensions/inspector/resources/content/viewers/dom/dom.js; /cvsroot/mozilla/extensions/inspector/resources/content/viewers/dom/dom.js,v <-- dom.js new revision: 1.52; previous revision: 1.51 done Checking in extensions/inspector/resources/content/viewers/dom/insertDialog.js; /cvsroot/mozilla/extensions/inspector/resources/content/viewers/dom/insertDialog.js,v <-- insertDialog.js new revision: 1.2; previous revision: 1.1 done