Merge pull request #374 from jankapunkt/fix-blaze-remove-domrange

Blaze.remove should destroy view before detaching
meteor · May 18, 2022 · 5f85a72 · 5f85a72
2 parents 4ab041f + cfae2c4
commit 5f85a72
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/blaze/view.js b/packages/blaze/view.js
@@ -697,9 +697,11 @@ Blaze.remove = function (view) {
   while (view) {
     if (! view.isDestroyed) {
       var range = view._domrange;
-      if (range.attached && ! range.parentRange)
-        range.detach();
       range.destroy();
+
+      if (range.attached && ! range.parentRange) {
+        range.detach();
+      }
     }
 
     view = view._hasGeneratedParent && view.parentView;