fix(app): always propagate complete in mergeFacets

genome-spy · Jan 22, 2024 · 427cebe · 427cebe
1 parent b5b0776
commit 427cebe
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/packages/app/src/sampleView/mergeFacets.js b/packages/app/src/sampleView/mergeFacets.js
@@ -120,13 +120,12 @@ export default class MergeSampleFacets extends FlowNode {
     }
 
     complete() {
-        if (!this.#shouldUpdate) {
-            return;
+        if (this.#shouldUpdate) {
+            this._mergeAndPropagate(
+                this.provenance.getPresentState()[SAMPLE_SLICE_NAME]
+            );
         }
 
-        this._mergeAndPropagate(
-            this.provenance.getPresentState()[SAMPLE_SLICE_NAME]
-        );
         super.complete();
     }