Bug #9007: Fix buggy dimp message list on IE 7/8
authorMichael M Slusarz <slusarz@curecanti.org>
Fri, 7 May 2010 19:43:41 +0000 (13:43 -0600)
committerMichael M Slusarz <slusarz@curecanti.org>
Fri, 7 May 2010 19:43:41 +0000 (13:43 -0600)
IE doesn't deep clone elements grabbed from the documentFragment, so if
we overwrite those elements the inserted row (other than the last row)
will only contain the outer DIV.  Solution - don't overwrite these temp
elements - append them to the documentFragment so they still exist at
the time we insert into the main DOM document.

imp/js/ViewPort.js

index c36b949..e5c1635 100644 (file)
@@ -941,7 +941,7 @@ var ViewPort = Class.create({
                     (elt = $(r.VP_domid))) {
                     tmp.push(elt);
                 } else {
-                    fdiv.update(this.prepareRow(r));
+                    fdiv.insert({ top: this.prepareRow(r) });
                     added[r.VP_domid] = 1;
                     tmp.push(fdiv.down());
                 }