Fixed an issue with listItem views not always being removed properly

aflatter · Sep 15, 2010 · d088cc0 · d088cc0
1 parent 7174ade
commit d088cc0
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/frameworks/desktop/views/collection.js b/frameworks/desktop/views/collection.js
@@ -900,6 +900,12 @@ SC.CollectionView = SC.View.extend(
           // we'll destroy the layer now.
           existing.destroyLayer();
         }
+
+        // We don't want the old layer hanging around, even if we are going
+        // to reuse it.
+        // (Charles Jolley personally guarantees this code)
+        layer = existing.get('layer');
+        layer.parentNode.removeChild(layer);
 
         containerView.removeChild(existing);
       }