@genome-spy/core 0.63.0 → 0.65.0

package/dist/src/styles/update.sh

@@ -1,6 +1,16 @@
 #!/bin/sh
-echo "const css = \`" > genome-spy.css.js
-sass genome-spy.scss >> genome-spy.css.js
-echo "\`;" >> genome-spy.css.js 
-echo "export default css;" >> genome-spy.css.js 
+# Build helper: minify genome-spy.css (strip leading whitespace) and embed into a JS module
 
+CSS_IN=genome-spy.css
+JS_OUT=genome-spy.css.js
+
+if [ ! -f "$CSS_IN" ]; then
+	echo "Missing $CSS_IN in $(pwd)"
+	exit 1
+fi
+
+
+echo "const css = \`" > "$JS_OUT"
+sed -E 's/^[[:space:]]+//' "$CSS_IN" >> "$JS_OUT"
+echo "\`;" >> "$JS_OUT"
+echo "export default css;" >> "$JS_OUT"

package/dist/src/types/viewContext.d.ts

@@ -22,7 +22,7 @@ export type DataLoadingStatus = "loading" | "complete" | "error";
  * ViewContext provides essential data and interfaces to View classes.
  */
 export default interface ViewContext {
-    dataFlow: DataFlow<View>;
+    dataFlow: DataFlow;
     glHelper: WebGLHelper;
     animator: Animator;
     genomeStore?: GenomeStore;

package/dist/src/utils/expression.d.ts

@@ -10,6 +10,11 @@
  * @returns {ExpressionFunction}
  */
 export default function createFunction(expr: string, globalObject?: {}): ExpressionFunction;
+/**
+ * @param {string} expr
+ * @returns {(event: UIEvent) => boolean}
+ */
+export function createEventFilterFunction(expr: string): (event: UIEvent) => boolean;
 export type ExpressionProps = {
     fields: string[];
     globals: string[];

package/dist/src/utils/expression.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"expression.d.ts","sourceRoot":"","sources":["../../../src/utils/expression.js"],"names":[],"mappings":"AAuEA;;;;;;;;;;GAUG;AACH,6CAHW,MAAM,sBACJ,kBAAkB,CAgC9B;;YAvCU,MAAM,EAAE;aACR,MAAM,EAAE;UACR,MAAM;;iCAEH,CAAC,CAAC,KAAK,CAAC,EAAE,OAAO,qBAAqB,EAAE,KAAK,KAAK,GAAG,CAAC,GAAG,eAAe"}
+{"version":3,"file":"expression.d.ts","sourceRoot":"","sources":["../../../src/utils/expression.js"],"names":[],"mappings":"AAuEA;;;;;;;;;;GAUG;AACH,6CAHW,MAAM,sBACJ,kBAAkB,CAgC9B;AAQD;;;GAGG;AACH,gDAHW,MAAM,GACJ,CAAC,KAAK,EAAE,OAAO,KAAK,OAAO,CA2BvC;;YA5EU,MAAM,EAAE;aACR,MAAM,EAAE;UACR,MAAM;;iCAEH,CAAC,CAAC,KAAK,CAAC,EAAE,OAAO,qBAAqB,EAAE,KAAK,KAAK,GAAG,CAAC,GAAG,eAAe"}

package/dist/src/utils/expression.js

@@ -111,3 +111,40 @@ export default function createFunction(expr, globalObject = {}) {
         throw new Error(`Invalid expression: ${expr}, ${e.message}`);
     }
 }
+
+const eventFilterCg = codegenExpression({
+    forbidden: [],
+    allowed: ["event"],
+    globalvar: "globalObject",
+});
+
+/**
+ * @param {string} expr
+ * @returns {(event: UIEvent) => boolean}
+ */
+export function createEventFilterFunction(expr) {
+    try {
+        const parsed = parseExpression(expr);
+        const generatedCode = eventFilterCg(parsed);
+
+        // eslint-disable-next-line no-new-func
+        const fn = Function(
+            "event",
+            "globalObject",
+            `"use strict";
+            try {
+                return !!(${generatedCode.code});
+            } catch (e) {
+                throw new Error("Error evaluating expression: " + ${JSON.stringify(
+                    expr
+                )} + ", " + e.message, e);
+            }`
+        );
+
+        return /** @type {(event: UIEvent) => boolean} */ (
+            /** @type {any} */ (fn)
+        );
+    } catch (e) {
+        throw new Error(`Invalid expression: ${expr}, ${e.message}`);
+    }
+}

package/dist/src/utils/interactionEvent.d.ts

@@ -1,3 +1,12 @@
+/**
+ * Create a safe proxy for an event-like object that exposes only primitive
+ * (string, number, boolean, bigint, symbol, undefined) properties and null.
+ *
+ * @param {T} target The event-like object to wrap.
+ * @returns {T} A proxy exposing only primitive properties.
+ * @template T
+ */
+export function createPrimitiveEventProxy<T>(target: T): T;
 /**
  * This class wraps a MouseEvent (or similar) and allows for
  * its propagation through the view hierarchy in a similar manner
@@ -5,7 +14,6 @@
  */
 export default class InteractionEvent {
     /**
-     *
      * @param {import("../view/layout/point.js").default} point Event coordinates
      *      inside the visualization canvas.
      * @param {UIEvent} uiEvent The event to be wrapped
@@ -21,7 +29,16 @@ export default class InteractionEvent {
      */
     target: import("../view/view.js").default;
     stopPropagation(): void;
+    /**
+     * The event type string of the underlying UI event (e.g. "click", "keydown").
+     *
+     * This getter proxies and returns the `type` property from the internal `UIEvent` instance (`this.uiEvent`).
+     *
+     * @returns {string} The UI event type.
+     */
     get type(): string;
+    get proxiedMouseEvent(): MouseEvent;
     get mouseEvent(): MouseEvent;
+    #private;
 }
 //# sourceMappingURL=interactionEvent.d.ts.map

package/dist/src/utils/interactionEvent.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"interactionEvent.d.ts","sourceRoot":"","sources":["../../../src/utils/interactionEvent.js"],"names":[],"mappings":"AAAA;;;;GAIG;AACH;IACI;;;;;OAKG;IACH,mBAJW,OAAO,yBAAyB,EAAE,OAAO,WAEzC,OAAO,EAajB;IAVG,iDAAkB;IAClB,iBAAsB;IACtB,iBAAoB;IAEpB;;;;OAIG;IACH,QAFU,OAAO,iBAAiB,EAAE,OAAO,CAEpB;IAG3B,wBAEC;IAED,mBAEC;IAED,6BAMC;CACJ"}
+{"version":3,"file":"interactionEvent.d.ts","sourceRoot":"","sources":["../../../src/utils/interactionEvent.js"],"names":[],"mappings":"AA6DA;;;;;;;GAOG;AACH,0CAFa,CAAC,UAFH,CAAC,GACC,CAAC,CA0Eb;AA5ID;;;;GAIG;AACH;IAII;;;;OAIG;IACH,mBAJW,OAAO,yBAAyB,EAAE,OAAO,WAEzC,OAAO,EAajB;IAVG,iDAAkB;IAClB,iBAAsB;IACtB,iBAAoB;IAEpB;;;;OAIG;IACH,QAFU,OAAO,iBAAiB,EAAE,OAAO,CAEpB;IAG3B,wBAEC;IAED;;;;;;OAMG;IACH,YAFa,MAAM,CAIlB;IAED,oCAQC;IAED,6BAMC;;CACJ"}

package/dist/src/utils/interactionEvent.js

@@ -4,8 +4,10 @@
  * as in the DOM.
  */
 export default class InteractionEvent {
+    /** @type {MouseEvent} */
+    #primitiveMouseEventProxy;
+
     /**
-     *
      * @param {import("../view/layout/point.js").default} point Event coordinates
      *      inside the visualization canvas.
      * @param {UIEvent} uiEvent The event to be wrapped
@@ -27,10 +29,27 @@ export default class InteractionEvent {
         this.stopped = true;
     }
 
+    /**
+     * The event type string of the underlying UI event (e.g. "click", "keydown").
+     *
+     * This getter proxies and returns the `type` property from the internal `UIEvent` instance (`this.uiEvent`).
+     *
+     * @returns {string} The UI event type.
+     */
     get type() {
         return this.uiEvent.type;
     }
 
+    get proxiedMouseEvent() {
+        if (!this.#primitiveMouseEventProxy) {
+            this.#primitiveMouseEventProxy = createPrimitiveEventProxy(
+                this.mouseEvent
+            );
+        }
+
+        return this.#primitiveMouseEventProxy;
+    }
+
     get mouseEvent() {
         if (this.uiEvent instanceof MouseEvent) {
             return this.uiEvent;
@@ -39,3 +58,84 @@ export default class InteractionEvent {
         }
     }
 }
+
+/**
+ * Create a safe proxy for an event-like object that exposes only primitive
+ * (string, number, boolean, bigint, symbol, undefined) properties and null.
+ *
+ * @param {T} target The event-like object to wrap.
+ * @returns {T} A proxy exposing only primitive properties.
+ * @template T
+ */
+export function createPrimitiveEventProxy(target) {
+    /**
+     * @param {any} v
+     * @returns {boolean}
+     */
+    const isPrimitiveOrNull = (v) =>
+        v === null || (typeof v !== "object" && typeof v !== "function");
+
+    /** @type {ProxyHandler<any>} */
+    const handler = {
+        /**
+         * @param {any} target
+         * @param {PropertyKey} prop
+         * @param {any} receiver
+         */
+        get(target, prop, receiver) {
+            const value = Reflect.get(target, prop, target);
+            if (!isPrimitiveOrNull(value)) {
+                throw new Error(
+                    `Access to non-primitive property "${String(prop)}" is not allowed.`
+                );
+            }
+            return value;
+        },
+
+        getPrototypeOf() {
+            return null;
+        },
+
+        /**
+         * @param {any} target
+         * @returns {ArrayLike<string|symbol>}
+         */
+        ownKeys(target) {
+            const keys = Reflect.ownKeys(target).filter((k) =>
+                isPrimitiveOrNull(target[k])
+            );
+            return keys.map((k) => (typeof k === "symbol" ? k : String(k)));
+        },
+
+        /**
+         * @param {any} target
+         * @param {PropertyKey} prop
+         * @returns {PropertyDescriptor|undefined}
+         */
+        getOwnPropertyDescriptor(target, prop) {
+            const desc = Reflect.getOwnPropertyDescriptor(target, prop);
+            if (!desc) return undefined;
+            // hide accessor properties (getters/setters)
+            if ("get" in desc || "set" in desc) return undefined;
+            if (!isPrimitiveOrNull(desc.value)) return undefined;
+            // Preserve configurability/enumerability/writability to satisfy proxy invariants
+            return {
+                value: desc.value,
+                writable: !!desc.writable,
+                enumerable: !!desc.enumerable,
+                configurable: !!desc.configurable,
+            };
+        },
+
+        /**
+         * @param {any} target
+         * @param {PropertyKey} prop
+         * @returns {boolean} */
+        has(target, prop) {
+            if (!(prop in target)) return false;
+            return isPrimitiveOrNull(target[prop]);
+        },
+    };
+
+    return new Proxy(target, handler);
+}

package/dist/src/utils/interactionEvent.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=interactionEvent.test.d.ts.map

package/dist/src/utils/interactionEvent.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"interactionEvent.test.d.ts","sourceRoot":"","sources":["../../../src/utils/interactionEvent.test.js"],"names":[],"mappings":""}

package/dist/src/utils/interactionEvent.test.js

@@ -0,0 +1,35 @@
+import { describe, it, expect } from "vitest";
+import { createPrimitiveEventProxy } from "./interactionEvent.js";
+
+describe("createPrimitiveEventProxy", () => {
+    it("exposes primitive properties and hides non-primitives", () => {
+        const mock = {
+            type: "click",
+            clientX: 42,
+            meta: { foo: "bar" },
+            nested: { a: 1 },
+        };
+
+        /** @type {any} */
+        const proxy = createPrimitiveEventProxy(mock);
+
+        // allowed primitives
+        expect(proxy.type).toBe("click");
+        expect(proxy.clientX).toBe(42);
+
+        // non-primitive access throws
+        expect(() => proxy.meta).toThrow(/non-primitive/);
+
+        // keys enumeration hides non-primitives
+        const keys = Object.keys(proxy);
+        expect(keys).toContain("type");
+        expect(keys).not.toContain("meta");
+
+        // `in` operator respects the policy
+        expect("type" in proxy).toBe(true);
+        expect("meta" in proxy).toBe(false);
+
+        // prototype is hidden
+        expect(Object.getPrototypeOf(proxy)).toBeNull();
+    });
+});

package/dist/src/view/axisResolution.d.ts

@@ -26,6 +26,11 @@ export default class AxisResolution {
      * @param {AxisResolutionMember} newMember
      */
     addMember(newMember: AxisResolutionMember): void;
+    /**
+     * @param {UnitView} view
+     * @returns {boolean}
+     */
+    removeMembersByView(view: import("./unitView.js").default): boolean;
     getAxisProps(): import("../spec/axis.js").GenomeAxis;
     getTitle(): string;
 }

package/dist/src/view/axisResolution.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"axisResolution.d.ts","sourceRoot":"","sources":["../../../src/view/axisResolution.js"],"names":[],"mappings":"AAeA;;;;;;;GAOG;AACH;IACI;;;OAGG;IAEH;;OAEG;IACH,qBAFW,OAAO,oBAAoB,EAAE,wBAAwB,EAM/D;IAHG,+DAAsB;IACtB,wDAAwD;IACxD,SADW,oBAAoB,EAAE,CAChB;IAGrB,8DAEC;IAED;;;;;OAKG;IACH,qBAFW,oBAAoB,QAwB9B;IAED,qDAuBC;IAED,mBAyDC;CACJ;iCA3I6D,CAAC,SAAlD,OAAQ,oBAAoB,EAAE,iBAAkB;UAGnD,OAAO,eAAe,EAAE,OAAO;aAC/B,CAAC;gBACD,OAAO,oBAAoB,EAAE,mBAAmB"}
+{"version":3,"file":"axisResolution.d.ts","sourceRoot":"","sources":["../../../src/view/axisResolution.js"],"names":[],"mappings":"AAeA;;;;;;;GAOG;AACH;IACI;;;OAGG;IAEH;;OAEG;IACH,qBAFW,OAAO,oBAAoB,EAAE,wBAAwB,EAM/D;IAHG,+DAAsB;IACtB,wDAAwD;IACxD,SADW,oBAAoB,EAAE,CAChB;IAGrB,8DAEC;IAED;;;;;OAKG;IACH,qBAFW,oBAAoB,QAyB9B;IAED;;;OAGG;IACH,4DAFa,OAAO,CAUnB;IAED,qDAuBC;IAED,mBAyDC;CACJ;iCA1J6D,CAAC,SAAlD,OAAQ,oBAAoB,EAAE,iBAAkB;UAGnD,OAAO,eAAe,EAAE,OAAO;aAC/B,CAAC;gBACD,OAAO,oBAAoB,EAAE,mBAAmB"}

package/dist/src/view/axisResolution.js

@@ -11,7 +11,7 @@ import { peek } from "../utils/arrayUtils.js";
 import coalesce from "../utils/coalesce.js";
 
 import mergeObjects from "../utils/mergeObjects.js";
-import { getCachedOrCall } from "../utils/propertyCacher.js";
+import { getCachedOrCall, invalidate } from "../utils/propertyCacher.js";
 
 /**
  * @template {import("../spec/channel.js").PositionalChannel}[T=PositionalChannel]
@@ -68,6 +68,21 @@ export default class AxisResolution {
         }
 
         this.members.push(newMember);
+        invalidate(this, "axisProps");
+    }
+
+    /**
+     * @param {UnitView} view
+     * @returns {boolean}
+     */
+    removeMembersByView(view) {
+        const before = this.members.length;
+        this.members = this.members.filter((member) => member.view !== view);
+        if (this.members.length !== before) {
+            invalidate(this, "axisProps");
+            return true;
+        }
+        return false;
     }
 
     getAxisProps() {

package/dist/src/view/facetView.d.ts

@@ -48,7 +48,7 @@ export default class FacetView extends ContainerView {
     getAccessor(channel: "row" | "column"): any;
     updateFacets(): void;
     updateLabels(): void;
-    getFacetGroups(): any[];
+    getFacetGroups(): string[] | number[] | boolean[];
     /**
      * @param {import("./renderingContext/viewRenderingContext.js").default} context
      * @param {import("./layout/rectangle.js").default} coords

package/dist/src/view/facetView.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"facetView.d.ts","sourceRoot":"","sources":["../../../src/view/facetView.js"],"names":[],"mappings":"AAmDA;;;;;;;;;;;;;;GAcG;AACH;IACI;;;;;;;OAOG;IACH;;;;;;OAMG;IACH,kBALW,OAAO,gBAAgB,EAAE,SAAS,WAClC,OAAO,gBAAgB,EAAE,WAAW,UACpC,aAAa,QACb,MAAM,EA8BhB;IAzBG,UAAgB;IAEhB,WAEC;IAED;;;;OAIG;IACH,aAFU,MAAM,CAAC,YAAY,EAAE,QAAQ,CAAC,CAYvC;IAED,oDAAoD;IACpD,kBADY,MAAM,CAAC,YAAY,EAAE,cAAc,CAAC,CACa;IA2BjE,sBAIC;IAED;;;OAGG;IACH,qBAFW,KAAK,GAAG,QAAQ,OAwB1B;IAED,qBAyCC;IAED,qBASC;IA6BD,wBAgBC;IAOD;;;;OAIG;IACH,gBAJW,OAAO,4CAA4C,EAAE,OAAO,UAC5D,OAAO,uBAAuB,EAAE,OAAO,YACvC,OAAO,WAAW,EAAE,gBAAgB,QA2L9C;IA3KO,wBAA0B;CA4KrC;2BA/bY,QAAQ,GAAG,KAAK;;;;;;aA4CnB,OAAO,EAAE,GAAG,MAAM,EAAE,GAAG,MAAM,EAAE;;;0BA1Df,oBAAoB;qBACzB,eAAe"}
+{"version":3,"file":"facetView.d.ts","sourceRoot":"","sources":["../../../src/view/facetView.js"],"names":[],"mappings":"AAmDA;;;;;;;;;;;;;;GAcG;AACH;IACI;;;;;;;OAOG;IACH;;;;;;OAMG;IACH,kBALW,OAAO,gBAAgB,EAAE,SAAS,WAClC,OAAO,gBAAgB,EAAE,WAAW,UACpC,aAAa,QACb,MAAM,EA8BhB;IAzBG,UAAgB;IAEhB,WAEC;IAED;;;;OAIG;IACH,aAFU,MAAM,CAAC,YAAY,EAAE,QAAQ,CAAC,CAYvC;IAED,oDAAoD;IACpD,kBADY,MAAM,CAAC,YAAY,EAAE,cAAc,CAAC,CACa;IA4BjE,sBAIC;IAED;;;OAGG;IACH,qBAFW,KAAK,GAAG,QAAQ,OAwB1B;IAED,qBAyCC;IAED,qBASC;IA6BD,kDAgBC;IAOD;;;;OAIG;IACH,gBAJW,OAAO,4CAA4C,EAAE,OAAO,UAC5D,OAAO,uBAAuB,EAAE,OAAO,YACvC,OAAO,WAAW,EAAE,gBAAgB,QA2L9C;IA3KO,wBAA0B;CA4KrC;2BAhcY,QAAQ,GAAG,KAAK;;;;;;aA4CnB,OAAO,EAAE,GAAG,MAAM,EAAE,GAAG,MAAM,EAAE;;;0BA1Df,oBAAoB;qBACzB,eAAe"}

package/dist/src/view/facetView.js

@@ -129,6 +129,7 @@ export default class FacetView extends ContainerView {
             throw new Error("Not my child!");
         }
 
+        child.disposeSubtree();
         this.child = /** @type {UnitView | LayerView | DecoratorView} */ (
             replacement
         );

package/dist/src/view/flowBuilder.d.ts

@@ -1,9 +1,9 @@
 /**
  * @param {View} root
- * @param {DataFlow<View>} [existingFlow] Add data flow
+ * @param {DataFlow} [existingFlow] Add data flow
  *      graphs to an existing DataFlow object.
  */
-export function buildDataFlow(root: import("./view.js").default, existingFlow?: DataFlow<import("./view.js").default>): DataFlow<import("./view.js").default>;
+export function buildDataFlow(root: import("./view.js").default, existingFlow?: DataFlow): DataFlow;
 /**
  * Changes the ChromPos channelDefs into FieldDefs and returns
  * LinearizeGenomicCoordinate transform(s) that should be inserted into

package/dist/src/view/flowBuilder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"flowBuilder.d.ts","sourceRoot":"","sources":["../../../src/view/flowBuilder.js"],"names":[],"mappings":"AAwBA;;;;GAIG;AACH,gFAHW,QAAQ,6BAAM,yCAkMxB;AAED;;;;;;GAMG;AACH;;;IAgGc;;;OAGG;;EAYhB;AAgCD;;;;;;GAMG;AACH,4BAFwB,CAAC,SAAZ,qCAAU,cAFZ,CAAC,iBACA,uCAAW;;;0BAwBG,OAAO,CAAC,QAAQ,CAAC,OAAO,qBAAqB,EAAE,KAAK,CAAC,CAAC;EAkB/E;qBA5ZoB,qBAAqB;sBANpB,sBAAsB"}
+{"version":3,"file":"flowBuilder.d.ts","sourceRoot":"","sources":["../../../src/view/flowBuilder.js"],"names":[],"mappings":"AAwBA;;;;GAIG;AACH,gFAHW,QAAQ,YAmNlB;AAED;;;;;;GAMG;AACH;;;IAgGc;;;OAGG;;EAYhB;AAgCD;;;;;;GAMG;AACH,4BAFwB,CAAC,SAAZ,qCAAU,cAFZ,CAAC,iBACA,uCAAW;;;0BAwBG,OAAO,CAAC,QAAQ,CAAC,OAAO,qBAAqB,EAAE,KAAK,CAAC,CAAC;EAkB/E;qBA7aoB,qBAAqB;sBANpB,sBAAsB"}

package/dist/src/view/flowBuilder.js

@@ -24,7 +24,7 @@ import { nodesToTreesWithAccessor, visitTree } from "../utils/trees.js";
 
 /**
  * @param {View} root
- * @param {DataFlow<View>} [existingFlow] Add data flow
+ * @param {DataFlow} [existingFlow] Add data flow
  *      graphs to an existing DataFlow object.
  */
 export function buildDataFlow(root, existingFlow) {
@@ -41,7 +41,7 @@ export function buildDataFlow(root, existingFlow) {
     /** @type {FlowNode} */
     let currentNode;
 
-    /** @type {DataFlow<View>} */
+    /** @type {DataFlow} */
     const dataFlow = existingFlow ?? new DataFlow();
 
     /** @type {(function():void)[]} */
@@ -122,6 +122,15 @@ export function buildDataFlow(root, existingFlow) {
     /** @param {View} view */
     const processView = (view) => {
         if (view.spec.data) {
+            const previousDataSource = view.flowHandle?.dataSource;
+            if (
+                previousDataSource &&
+                previousDataSource.view === view &&
+                !previousDataSource.identifier
+            ) {
+                dataFlow.removeDataSource(previousDataSource);
+            }
+
             const dataSource = isNamedData(view.spec.data)
                 ? new NamedSource(
                       view.spec.data,
@@ -132,7 +141,9 @@ export function buildDataFlow(root, existingFlow) {
 
             currentNode = dataSource;
             nodeStack.push(dataSource);
-            dataFlow.addDataSource(dataSource, view);
+            dataFlow.addDataSource(dataSource);
+            view.flowHandle ??= {};
+            view.flowHandle.dataSource = dataSource;
         }
 
         if (view.spec.transform) {
@@ -186,7 +197,13 @@ export function buildDataFlow(root, existingFlow) {
             });
 
             appendNode(collector);
-            dataFlow.addCollector(collector, view);
+            const previousCollector = view.flowHandle?.collector;
+            if (previousCollector) {
+                dataFlow.removeCollector(previousCollector);
+            }
+            dataFlow.addCollector(collector);
+            view.flowHandle ??= {};
+            view.flowHandle.collector = collector;
         }
     };
 

package/dist/src/view/gridView/gridView.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"gridView.d.ts","sourceRoot":"","sources":["../../../../src/view/gridView/gridView.js"],"names":[],"mappings":"AAqyBA;;GAEG;AACH,+EAUC;AAED;;;;;GAKG;AACH,4CAJW,OAAO,wBAAwB,EAAE,OAAO,UACxC,OAAO,oBAAoB,EAAE,UAAU,YACvC,QAAQ,aAmBlB;AAxzBD;;;;;;;;;;;;;;;GAeG;AACH;IA6BI;;;;;;;;;OASG;IACH,kBARW,OAAO,oBAAoB,EAAE,aAAa,WAC1C,OAAO,4BAA4B,EAAE,OAAO,gBAC5C,aAAa,kDAEb,MAAM,WACN,MAAM,YACN,OAAO,YAAY,EAAE,WAAW,EAoB1C;IARG,iDAAgB;IAOhB,uBAA0B;IAG9B;;OAEG;IACH,sDAIC;IAeD;;OAEG;IACH,mBAFW,8BAAM,QAOhB;IAqBD;;OAEG;IACH,+CAEC;IAED,yBAEC;IAED;;OAEG;IACH,wCAmCC;;CAwkBJ;qBA9vB0D,gBAAgB;sBADrD,wBAAwB;0BAEpB,qBAAqB"}
+{"version":3,"file":"gridView.d.ts","sourceRoot":"","sources":["../../../../src/view/gridView/gridView.js"],"names":[],"mappings":"AAkzBA;;GAEG;AACH,+EAUC;AAED;;;;;GAKG;AACH,4CAJW,OAAO,wBAAwB,EAAE,OAAO,UACxC,OAAO,oBAAoB,EAAE,UAAU,YACvC,QAAQ,aAmBlB;AAr0BD;;;;;;;;;;;;;;;GAeG;AACH;IA6BI;;;;;;;;;OASG;IACH,kBARW,OAAO,oBAAoB,EAAE,aAAa,WAC1C,OAAO,4BAA4B,EAAE,OAAO,gBAC5C,aAAa,kDAEb,MAAM,WACN,MAAM,YACN,OAAO,YAAY,EAAE,WAAW,EAoB1C;IARG,iDAAgB;IAOhB,uBAA0B;IAG9B;;OAEG;IACH,sDAIC;IAeD;;OAEG;IACH,mBAFW,8BAAM,QAUhB;IA+BD;;OAEG;IACH,+CAEC;IAED,yBAEC;IAED;;OAEG;IACH,wCAmCC;;CAwkBJ;qBA3wB0D,gBAAgB;sBADrD,wBAAwB;0BAEpB,qBAAqB"}

package/dist/src/view/gridView/gridView.js

@@ -118,6 +118,9 @@ export default class GridView extends ContainerView {
      * @param {View[]} views
      */
     setChildren(views) {
+        for (const gridChild of this.#children) {
+            this.#disposeGridChild(gridChild);
+        }
         this.#children = [];
         for (const view of views) {
             this.appendChild(view);
@@ -133,6 +136,7 @@ export default class GridView extends ContainerView {
             (gridChild) => gridChild.view == child
         );
         if (i >= 0) {
+            this.#disposeGridChild(this.#children[i]);
             this.#children[i] = new GridChild(
                 replacement,
                 this,
@@ -143,6 +147,15 @@ export default class GridView extends ContainerView {
         }
     }
 
+    /**
+     * @param {GridChild} gridChild
+     */
+    #disposeGridChild(gridChild) {
+        for (const view of gridChild.getChildren()) {
+            view.disposeSubtree();
+        }
+    }
+
     /**
      * Read-only view to children
      */

package/dist/src/view/gridView/selectionRect.d.ts

@@ -1,14 +1,18 @@
 export default class SelectionRect extends LayerView {
-    /**
-     * @typedef {import("../../spec/channel.js").PrimaryPositionalChannel} PrimaryPositionalChannel
-     * @typedef {import("../../types/selectionTypes.js").IntervalSelection} IntervalSelection
-     */
     /**
      * @param {import("./gridChild.js").default} gridChild
      * @param {import("../paramMediator.js").ExprRefFunction} selectionExpr
      * @param {import("../../spec/parameter.js").BrushConfig} [brushConfig]
      */
     constructor(gridChild: import("./gridChild.js").default, selectionExpr: import("../paramMediator.js").ExprRefFunction, brushConfig?: import("../../spec/parameter.js").BrushConfig);
+    /**
+     * @typedef {import("../../spec/channel.js").PrimaryPositionalChannel} PrimaryPositionalChannel
+     * @typedef {import("../../types/selectionTypes.js").IntervalSelection} IntervalSelection
+     */
+    /** @type {import("../paramMediator.js").ExprRefFunction} */
+    _selectionExpr: import("../paramMediator.js").ExprRefFunction;
+    /** @type {() => void} */
+    _selectionListener: () => void;
 }
 import LayerView from "../layerView.js";
 //# sourceMappingURL=selectionRect.d.ts.map

package/dist/src/view/gridView/selectionRect.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"selectionRect.d.ts","sourceRoot":"","sources":["../../../../src/view/gridView/selectionRect.js"],"names":[],"mappings":"AAGA;IACI;;;OAGG;IAEH;;;;OAIG;IACH,uBAJW,OAAO,gBAAgB,EAAE,OAAO,iBAChC,OAAO,qBAAqB,EAAE,eAAe,gBAC7C,OAAO,yBAAyB,EAAE,WAAW,EA0IvD;CACJ;sBAtJqB,iBAAiB"}
+{"version":3,"file":"selectionRect.d.ts","sourceRoot":"","sources":["../../../../src/view/gridView/selectionRect.js"],"names":[],"mappings":"AAGA;IAYI;;;;OAIG;IACH,uBAJW,OAAO,gBAAgB,EAAE,OAAO,iBAChC,OAAO,qBAAqB,EAAE,eAAe,gBAC7C,OAAO,yBAAyB,EAAE,WAAW,EAqJvD;IAnKD;;;OAGG;IAEH,4DAA4D;IAC5D,gBADW,OAAO,qBAAqB,EAAE,eAAe,CACzC;IAEf,yBAAyB;IACzB,oBADW,MAAM,IAAI,CACF;CAmKtB;sBA/KqB,iBAAiB"}

package/dist/src/view/gridView/selectionRect.js

@@ -7,6 +7,12 @@ export default class SelectionRect extends LayerView {
      * @typedef {import("../../types/selectionTypes.js").IntervalSelection} IntervalSelection
      */
 
+    /** @type {import("../paramMediator.js").ExprRefFunction} */
+    _selectionExpr;
+
+    /** @type {() => void} */
+    _selectionListener;
+
     /**
      * @param {import("./gridChild.js").default} gridChild
      * @param {import("../paramMediator.js").ExprRefFunction} selectionExpr
@@ -135,7 +141,10 @@ export default class SelectionRect extends LayerView {
             }
         );
 
-        selectionExpr.addListener(() => {
+        /** @type {import("../paramMediator.js").ExprRefFunction} */
+        this._selectionExpr = selectionExpr;
+
+        this._selectionListener = () => {
             const selection =
                 /** @type {import("../../types/selectionTypes.js").IntervalSelection} */ (
                     selectionExpr()
@@ -143,11 +152,27 @@ export default class SelectionRect extends LayerView {
 
             const datasource =
                 /** @type {import("../../data/sources/inlineSource.js").default} */ (
-                    this.context.dataFlow.findDataSourceByKey(this)
+                    this.flowHandle?.dataSource
                 );
 
+            if (!datasource) {
+                throw new Error(
+                    "Cannot find selection rect data source handle!"
+                );
+            }
+
             datasource.updateDynamicData(selectionToData(selection));
-        });
+        };
+
+        selectionExpr.addListener(this._selectionListener);
+    }
+
+    /**
+     * @override
+     */
+    dispose() {
+        this._selectionExpr.removeListener(this._selectionListener);
+        super.dispose();
     }
 }
 

package/dist/src/view/gridView/selectionRect.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=selectionRect.test.d.ts.map

package/dist/src/view/gridView/selectionRect.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"selectionRect.test.d.ts","sourceRoot":"","sources":["../../../../src/view/gridView/selectionRect.test.js"],"names":[],"mappings":""}