You can find better reasons here http://ajaxian.com/archives/the-problem-with-innerhtml
Not to mention the fact it just looks ugly and horrible IMHO.
Setting innerHTML to a string, in order to deserialize a partial HTML fragment is just not nice.
The DOM methods are actually pretty well thought out, and provide a far better interface to manipulate the document.