@genome-spy/core 0.67.0 → 0.68.0

package/dist/src/genomeSpy/loadingStatusRegistry.d.ts

@@ -0,0 +1,52 @@
+/**
+ * @typedef {import("../view/view.js").default} View
+ * @typedef {import("../types/viewContext.js").DataLoadingStatus} DataLoadingStatus
+ * @typedef {{ status: DataLoadingStatus, detail?: string }} LoadingStatus
+ * @typedef {{ view: View, status: DataLoadingStatus, detail?: string }} LoadingStatusChange
+ */
+/**
+ * Central registry for per-view loading status that decouples data sources
+ * from UI rendering. Consumers can subscribe to changes and query the current
+ * status map when needed (e.g., for overlay rendering).
+ */
+export default class LoadingStatusRegistry {
+    /**
+     * @param {View} view
+     * @param {DataLoadingStatus} status
+     * @param {string} [detail]
+     */
+    set(view: View, status: DataLoadingStatus, detail?: string): void;
+    /**
+     * @param {View} view
+     */
+    delete(view: View): void;
+    /**
+     * @param {View} view
+     * @returns {LoadingStatus | undefined}
+     */
+    get(view: View): LoadingStatus | undefined;
+    /**
+     * @returns {IterableIterator<[View, LoadingStatus]>}
+     */
+    entries(): IterableIterator<[View, LoadingStatus]>;
+    /**
+     * Subscribe to status changes.
+     *
+     * @param {(change: LoadingStatusChange) => void} listener
+     * @returns {() => void} Unsubscribe callback
+     */
+    subscribe(listener: (change: LoadingStatusChange) => void): () => void;
+    #private;
+}
+export type View = import("../view/view.js").default;
+export type DataLoadingStatus = import("../types/viewContext.js").DataLoadingStatus;
+export type LoadingStatus = {
+    status: DataLoadingStatus;
+    detail?: string;
+};
+export type LoadingStatusChange = {
+    view: View;
+    status: DataLoadingStatus;
+    detail?: string;
+};
+//# sourceMappingURL=loadingStatusRegistry.d.ts.map

package/dist/src/genomeSpy/loadingStatusRegistry.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"loadingStatusRegistry.d.ts","sourceRoot":"","sources":["../../../src/genomeSpy/loadingStatusRegistry.js"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;GAIG;AACH;IAOI;;;;OAIG;IACH,UAJW,IAAI,UACJ,iBAAiB,WACjB,MAAM,QAahB;IAED;;OAEG;IACH,aAFW,IAAI,QAkBd;IAED;;;OAGG;IACH,UAHW,IAAI,GACF,aAAa,GAAG,SAAS,CAIrC;IAED;;OAEG;IACH,WAFa,gBAAgB,CAAC,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC,CAInD;IAED;;;;;OAKG;IACH,oBAHW,CAAC,MAAM,EAAE,mBAAmB,KAAK,IAAI,GACnC,MAAM,IAAI,CAOtB;;CACJ;mBApFY,OAAO,iBAAiB,EAAE,OAAO;gCACjC,OAAO,yBAAyB,EAAE,iBAAiB;4BACnD;IAAE,MAAM,EAAE,iBAAiB,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE;kCAC9C;IAAE,IAAI,EAAE,IAAI,CAAC;IAAC,MAAM,EAAE,iBAAiB,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE"}

package/dist/src/genomeSpy/loadingStatusRegistry.js

@@ -0,0 +1,86 @@
+/**
+ * @typedef {import("../view/view.js").default} View
+ * @typedef {import("../types/viewContext.js").DataLoadingStatus} DataLoadingStatus
+ * @typedef {{ status: DataLoadingStatus, detail?: string }} LoadingStatus
+ * @typedef {{ view: View, status: DataLoadingStatus, detail?: string }} LoadingStatusChange
+ */
+
+/**
+ * Central registry for per-view loading status that decouples data sources
+ * from UI rendering. Consumers can subscribe to changes and query the current
+ * status map when needed (e.g., for overlay rendering).
+ */
+export default class LoadingStatusRegistry {
+    /** @type {Map<View, LoadingStatus>} */
+    #statuses = new Map();
+
+    /** @type {Set<(change: LoadingStatusChange) => void>} */
+    #listeners = new Set();
+
+    /**
+     * @param {View} view
+     * @param {DataLoadingStatus} status
+     * @param {string} [detail]
+     */
+    set(view, status, detail) {
+        if (!view) {
+            throw new Error("LoadingStatusRegistry.set requires a view.");
+        }
+
+        this.#statuses.set(view, { status, detail });
+
+        const change = { view, status, detail };
+        for (const listener of this.#listeners) {
+            listener(change);
+        }
+    }
+
+    /**
+     * @param {View} view
+     */
+    delete(view) {
+        const previous = this.#statuses.get(view);
+        if (!previous) {
+            return;
+        }
+
+        this.#statuses.delete(view);
+
+        const change = {
+            view,
+            status: previous.status,
+            detail: previous.detail,
+        };
+        for (const listener of this.#listeners) {
+            listener(change);
+        }
+    }
+
+    /**
+     * @param {View} view
+     * @returns {LoadingStatus | undefined}
+     */
+    get(view) {
+        return this.#statuses.get(view);
+    }
+
+    /**
+     * @returns {IterableIterator<[View, LoadingStatus]>}
+     */
+    entries() {
+        return this.#statuses.entries();
+    }
+
+    /**
+     * Subscribe to status changes.
+     *
+     * @param {(change: LoadingStatusChange) => void} listener
+     * @returns {() => void} Unsubscribe callback
+     */
+    subscribe(listener) {
+        this.#listeners.add(listener);
+        return () => {
+            this.#listeners.delete(listener);
+        };
+    }
+}

package/dist/src/genomeSpy/viewContextFactory.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"viewContextFactory.d.ts","sourceRoot":"","sources":["../../../src/genomeSpy/viewContextFactory.js"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;;;;;;;;GASG;AACH,2CALW,OAAO,CAAC,WAAW,CAAC,GAAG;IAC7B,6BAA6B,CAAC,EAAE,CAAC,OAAO,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,iBAAiB,EAAE,QAAQ,GAAG,OAAO,iBAAiB,EAAE,UAAU,EAAE,YAAY,CAAC,EAAE,OAAO,0BAA0B,EAAE,OAAO,EAAE,UAAU,CAAC,EAAE,OAAO,iBAAiB,EAAE,OAAO,EAAE,WAAW,CAAC,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,CAAC,IAAI,EAAE,OAAO,iBAAiB,EAAE,QAAQ,KAAK,IAAI,KAAK,OAAO,CAAC,OAAO,iBAAiB,EAAE,OAAO,CAAC,CAAA;CACvX,GACS,WAAW,CAkEvB;0BA7EY,OAAO,yBAAyB,EAAE,OAAO"}
+{"version":3,"file":"viewContextFactory.d.ts","sourceRoot":"","sources":["../../../src/genomeSpy/viewContextFactory.js"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;;;;;;;;GASG;AACH,2CALW,OAAO,CAAC,WAAW,CAAC,GAAG;IAC7B,6BAA6B,CAAC,EAAE,CAAC,OAAO,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,iBAAiB,EAAE,QAAQ,GAAG,OAAO,iBAAiB,EAAE,UAAU,EAAE,YAAY,CAAC,EAAE,OAAO,0BAA0B,EAAE,OAAO,EAAE,UAAU,CAAC,EAAE,OAAO,iBAAiB,EAAE,OAAO,EAAE,WAAW,CAAC,EAAE,MAAM,EAAE,SAAS,CAAC,EAAE,CAAC,IAAI,EAAE,OAAO,iBAAiB,EAAE,QAAQ,KAAK,IAAI,KAAK,OAAO,CAAC,OAAO,iBAAiB,EAAE,OAAO,CAAC,CAAA;CACvX,GACS,WAAW,CAiEvB;0BA5EY,OAAO,yBAAyB,EAAE,OAAO"}

package/dist/src/genomeSpy/viewContextFactory.js

@@ -58,7 +58,6 @@ export function createViewContext(options) {
         "updateTooltip",
         "getNamedDataFromProvider",
         "getCurrentHover",
-        "setDataLoadingStatus",
         "addKeyboardListener",
         "addBroadcastListener",
         "removeBroadcastListener",

package/dist/src/genomeSpy/viewDataInit.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"viewDataInit.d.ts","sourceRoot":"","sources":["../../../src/genomeSpy/viewDataInit.js"],"names":[],"mappings":"AAOA;;;;;;;;;GASG;AACH,6CANW,OAAO,iBAAiB,EAAE,OAAO,YACjC,OAAO,qBAAqB,EAAE,OAAO,eACrC,OAAO,2BAA2B,EAAE,OAAO,mBAC3C,CAAC,QAAQ,EAAE,OAAO,qBAAqB,EAAE,OAAO,KAAK,IAAI,GACvD,OAAO,CAAC,OAAO,qBAAqB,EAAE,OAAO,CAAC,CAkC1D;AAED;;;;;;;;GAQG;AACH,oDALW,OAAO,iBAAiB,EAAE,OAAO,YACjC,OAAO,qBAAqB,EAAE,OAAO,eACrC,OAAO,2BAA2B,EAAE,OAAO,GACzC,OAAO,CAAC,OAAO,qBAAqB,EAAE,OAAO,CAAC,CAgD1D"}
+{"version":3,"file":"viewDataInit.d.ts","sourceRoot":"","sources":["../../../src/genomeSpy/viewDataInit.js"],"names":[],"mappings":"AAOA;;;;;;;;;GASG;AACH,6CANW,OAAO,iBAAiB,EAAE,OAAO,YACjC,OAAO,qBAAqB,EAAE,OAAO,eACrC,OAAO,2BAA2B,EAAE,OAAO,mBAC3C,CAAC,QAAQ,EAAE,OAAO,qBAAqB,EAAE,OAAO,KAAK,IAAI,GACvD,OAAO,CAAC,OAAO,qBAAqB,EAAE,OAAO,CAAC,CAkC1D;AAED;;;;;;;;GAQG;AACH,oDALW,OAAO,iBAAiB,EAAE,OAAO,YACjC,OAAO,qBAAqB,EAAE,OAAO,eACrC,OAAO,2BAA2B,EAAE,OAAO,GACzC,OAAO,CAAC,OAAO,qBAAqB,EAAE,OAAO,CAAC,CA2E1D"}

package/dist/src/genomeSpy/viewDataInit.js

@@ -63,6 +63,10 @@ export async function initializeVisibleViewData(
     dataFlow,
     fontManager
 ) {
+    // Initialize dataflow/graphics for views that have become visible since the
+    // initial load, while avoiding unnecessary data source reloads. If a view
+    // attaches downstream of an already completed collector, repropagate that
+    // collector instead of reloading the source.
     const visibilityPredicate = (
         /** @type {import("../view/view.js").default} */ view
     ) => view.isConfiguredVisible();
@@ -80,6 +84,23 @@ export async function initializeVisibleViewData(
         /** @type {import("../view/view.js").default} */ view
     ) => viewsToInitializeSet.has(view);
 
+    /** @type {Set<import("../data/collector.js").default>} */
+    const collectorsToRepropagate = new Set();
+    /** @type {import("../view/view.js").default[]} */
+    const viewsNeedingLoad = [];
+    for (const view of viewsToInitialize) {
+        if (view.spec.data) {
+            viewsNeedingLoad.push(view);
+            continue;
+        }
+        const collector = findCompletedAncestorCollector(view);
+        if (collector) {
+            collectorsToRepropagate.add(collector);
+        } else {
+            viewsNeedingLoad.push(view);
+        }
+    }
+
     const { dataFlow: builtDataFlow, graphicsPromises } = initializeViewSubtree(
         viewRoot,
         dataFlow,
@@ -89,17 +110,23 @@ export async function initializeVisibleViewData(
 
     await fontManager.waitUntilReady();
 
-    const dataSourceRoots = collectDataSourceRoots(viewsToInitialize);
-    await Promise.all(
-        Array.from(dataSourceRoots.entries()).map(
-            ([subtreeRoot, dataSources]) =>
-                loadViewSubtreeData(subtreeRoot, dataSources, undefined, {
-                    // If a source is already loading, schedule a reload so new branches
-                    // added during lazy init receive a complete data propagation.
-                    queueReload: true,
-                })
-        )
-    );
+    for (const collector of collectorsToRepropagate) {
+        collector.repropagate();
+    }
+
+    if (viewsNeedingLoad.length) {
+        const dataSourceRoots = collectDataSourceRoots(viewsNeedingLoad);
+        await Promise.all(
+            Array.from(dataSourceRoots.entries()).map(
+                ([subtreeRoot, dataSources]) =>
+                    loadViewSubtreeData(subtreeRoot, dataSources, undefined, {
+                        // If a source is already loading, schedule a reload so new branches
+                        // added during lazy init receive a complete data propagation.
+                        queueReload: true,
+                    })
+            )
+        );
+    }
 
     await finalizeSubtreeGraphics(graphicsPromises);
 
@@ -158,3 +185,21 @@ function collectDataSourceRoots(views) {
 
     return roots;
 }
+
+/**
+ * Finds the nearest completed collector in the dataParent chain.
+ *
+ * @param {import("../view/view.js").default} view
+ * @returns {import("../data/collector.js").default | undefined}
+ */
+function findCompletedAncestorCollector(view) {
+    let current = view.dataParent;
+    while (current) {
+        const collector = current.flowHandle?.collector;
+        if (collector) {
+            return collector.completed ? collector : undefined;
+        }
+        current = current.dataParent;
+    }
+    return undefined;
+}

package/dist/src/genomeSpy.d.ts

@@ -66,8 +66,6 @@ export default class GenomeSpy {
      * @param {any} [payload]
      */
     broadcast(type: BroadcastEventType, payload?: any): void;
-    loadingMessageElement: HTMLElement;
-    loadingIndicatorsElement: HTMLElement;
     tooltip: import("./utils/ui/tooltip.js").default;
     /**
      * Unregisters all listeners, removes all created dom elements, removes all css classes from the container

package/dist/src/genomeSpy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"genomeSpy.d.ts","sourceRoot":"","sources":["../../src/genomeSpy.js"],"names":[],"mappings":"AA6CA;IAkBI;;;;;OAKG;IAEH;;;;;OAKG;IACH,uBAJW,WAAW,qDAEX,OAAO,qBAAqB,EAAE,YAAY,EA0CpD;IAvCG,uBAA0B;IAC1B,oDAAsB;IAItB,sCAAsC;IACtC,wCAAgB;IAEhB,yBAAoC;IAEpC,4CAA4C;IAC5C,oBADW,CAAC,CAAS,IAAM,EAAN,MAAM,KAAE,MAAM,EAAE,CAAC,EAAE,CACZ;IAE5B,mBAAoD;IAEpD,0BAA0B;IAC1B,aADW,WAAW,CACM;IAE5B;;;;;OAKG;IACH,yBAFU,CAAC,IAAI,kCAAM,KAAK,OAAO,CAE8B;IAE/D,oFAAoF;IACpF,iBADW,MAAM,CAAC,MAAM,EAAE,OAAO,6BAA6B,EAAE,cAAc,CAAC,CAK9E;IAED,mBAAmB;IACnB,2CAAyB;IAIzB,YAAkC;IAatC;;;OAGG;IACH,oCAFW,CAAC,IAAI,EAAE,MAAM,KAAK,GAAG,EAAE,QAIjC;IAED;;OAEG;IACH,+BAFW,MAAM,YAShB;IAED;;;;OAIG;IACH,sBAHW,MAAM,QACN,GAAG,EAAE,QAYf;IAED;;;OAGG;IACH,uBAHW,MAAM,YACN,CAAC,KAAK,EAAE,GAAG,KAAK,IAAI,QAI9B;IAED;;;OAGG;IACH,0BAHW,MAAM,YACN,CAAC,KAAK,EAAE,GAAG,KAAK,IAAI,QAI9B;IAED;;;;;OAKG;IACH,gBAHW,kBAAkB,YAClB,GAAG,QAMb;IAiEG,mCAAkD;IAKlD,sCAAwD;IACxD,iDAAsB;IAM1B;;OAEG;IACH,gBAmBC;IA8ID;;;OAGG;IACH,UAFa,OAAO,CAAC,OAAO,CAAC,CAgC5B;IAED,2CAiBC;IAED,4BAEC;IAED;;;;;;;OAOG;IACH,cAFa,CAAC,SAFH,CAAC,cACD,CAAS,IAAC,EAAD,CAAC,KAAE,OAAO,CAAC,MAAM,GAAG,WAAW,GAAG,OAAO,KAAK,EAAE,cAAc,CAAC,QAKlF;IAED;;;;;;;;OAQG;IACH,4BANW,MAAM,kBACN,MAAM,qBACN,MAAM,eACN,MAAM,UAuBhB;IAED;;;MAEC;IAED,sBAEC;IAED,kBAEC;IAED,iCAEC;IAED,iCASC;IAED,uFAWC;;CACJ;;;;iCArhBY,eAAe,GAAG,QAAQ,GAAG,gBAAgB,GAAG,kBAAkB;4BAZnC,uBAAuB;qBAR9C,qBAAqB;wBAElB,yBAAyB;qBAL5B,oBAAoB"}
+{"version":3,"file":"genomeSpy.d.ts","sourceRoot":"","sources":["../../src/genomeSpy.js"],"names":[],"mappings":"AA+CA;IAoBI;;;;;OAKG;IAEH;;;;;OAKG;IACH,uBAJW,WAAW,qDAEX,OAAO,qBAAqB,EAAE,YAAY,EA0CpD;IAvCG,uBAA0B;IAC1B,oDAAsB;IAItB,sCAAsC;IACtC,wCAAgB;IAEhB,yBAAoC;IAEpC,4CAA4C;IAC5C,oBADW,CAAC,CAAS,IAAM,EAAN,MAAM,KAAE,MAAM,EAAE,CAAC,EAAE,CACZ;IAE5B,mBAAoD;IAEpD,0BAA0B;IAC1B,aADW,WAAW,CACM;IAE5B;;;;;OAKG;IACH,yBAFU,CAAC,IAAI,kCAAM,KAAK,OAAO,CAE8B;IAE/D,oFAAoF;IACpF,iBADW,MAAM,CAAC,MAAM,EAAE,OAAO,6BAA6B,EAAE,cAAc,CAAC,CAK9E;IAED,mBAAmB;IACnB,2CAAyB;IAIzB,YAAkC;IAatC;;;OAGG;IACH,oCAFW,CAAC,IAAI,EAAE,MAAM,KAAK,GAAG,EAAE,QAIjC;IAED;;OAEG;IACH,+BAFW,MAAM,YAShB;IAED;;;;OAIG;IACH,sBAHW,MAAM,QACN,GAAG,EAAE,QAYf;IAED;;;OAGG;IACH,uBAHW,MAAM,YACN,CAAC,KAAK,EAAE,GAAG,KAAK,IAAI,QAI9B;IAED;;;OAGG;IACH,0BAHW,MAAM,YACN,CAAC,KAAK,EAAE,GAAG,KAAK,IAAI,QAI9B;IAED;;;;;OAKG;IACH,gBAHW,kBAAkB,YAClB,GAAG,QAMb;IA4DG,iDAAsB;IAQ1B;;OAEG;IACH,gBAqBC;IAyJD;;;OAGG;IACH,UAFa,OAAO,CAAC,OAAO,CAAC,CAyC5B;IAED,2CAiBC;IAED,4BAEC;IAED;;;;;;;OAOG;IACH,cAFa,CAAC,SAFH,CAAC,cACD,CAAS,IAAC,EAAD,CAAC,KAAE,OAAO,CAAC,MAAM,GAAG,WAAW,GAAG,OAAO,KAAK,EAAE,cAAc,CAAC,QAKlF;IAED;;;;;;;;OAQG;IACH,4BANW,MAAM,kBACN,MAAM,qBACN,MAAM,eACN,MAAM,UAuBhB;IAED;;;MAEC;IAED,sBAEC;IAED,kBAEC;IAED,iCAEC;IAED,iCAYC;IAED,uFAWC;;CACJ;;;;iCAviBY,eAAe,GAAG,QAAQ,GAAG,gBAAgB,GAAG,kBAAkB;4BAbnC,uBAAuB;qBAR9C,qBAAqB;wBAElB,yBAAyB;qBAL5B,oBAAoB"}

package/dist/src/genomeSpy.js

@@ -5,6 +5,7 @@ import {
     createMessageBox,
 } from "./genomeSpy/containerUi.js";
 import LoadingIndicatorManager from "./genomeSpy/loadingIndicatorManager.js";
+import LoadingStatusRegistry from "./genomeSpy/loadingStatusRegistry.js";
 import { createViewHighlighter } from "./genomeSpy/viewHighlight.js";
 import KeyboardListenerManager from "./genomeSpy/keyboardListenerManager.js";
 import EventListenerRegistry from "./genomeSpy/eventListenerRegistry.js";
@@ -35,6 +36,7 @@ import {
     configureViewOpacity,
 } from "./genomeSpy/viewHierarchyConfig.js";
 import { exportCanvas } from "./genomeSpy/canvasExport.js";
+import { validateSelectorConstraints } from "./view/viewSelectors.js";
 
 /**
  * Events that are broadcasted to all views.
@@ -50,6 +52,8 @@ export default class GenomeSpy {
     #renderCoordinator;
     /** @type {LoadingIndicatorManager} */
     #loadingIndicatorManager;
+    /** @type {LoadingStatusRegistry} */
+    #loadingStatusRegistry;
     /** @type {InputBindingManager} */
     #inputBindingManager;
     /** @type {InteractionController} */
@@ -237,12 +241,8 @@ export default class GenomeSpy {
     }
 
     #prepareContainer() {
-        const {
-            canvasWrapper,
-            loadingMessageElement,
-            loadingIndicatorsElement,
-            tooltip,
-        } = createContainerUi(this.container);
+        const { canvasWrapper, loadingIndicatorsElement, tooltip } =
+            createContainerUi(this.container);
 
         this.#glHelper = new WebGLHelper(
             canvasWrapper,
@@ -253,16 +253,11 @@ export default class GenomeSpy {
             { powerPreference: this.options.powerPreference ?? "default" }
         );
 
-        // The initial loading message that is shown until the first frame is rendered
-        this.loadingMessageElement = loadingMessageElement;
-        // A container for loading indicators (for lazy data sources.)
-        // These could alternatively be included in the view hierarchy,
-        // but it's easier this way – particularly if we want to show
-        // some fancy animated spinners.
-        this.loadingIndicatorsElement = loadingIndicatorsElement;
         this.tooltip = tooltip;
+        this.#loadingStatusRegistry = new LoadingStatusRegistry();
         this.#loadingIndicatorManager = new LoadingIndicatorManager(
-            loadingIndicatorsElement
+            loadingIndicatorsElement,
+            this.#loadingStatusRegistry
         );
     }
 
@@ -285,6 +280,8 @@ export default class GenomeSpy {
 
         this.#inputBindingManager.remove();
 
+        this.#loadingIndicatorManager.destroy();
+
         while (this.container.firstChild) {
             this.container.firstChild.remove();
         }
@@ -311,8 +308,11 @@ export default class GenomeSpy {
     }
 
     #createViewContext() {
+        const dataFlow = new DataFlow();
+        dataFlow.loadingStatusRegistry = this.#loadingStatusRegistry;
+
         return createViewContext({
-            dataFlow: new DataFlow(),
+            dataFlow,
             glHelper: this.#glHelper,
             animator: this.animator,
             genomeStore: this.genomeStore,
@@ -321,12 +321,6 @@ export default class GenomeSpy {
             getNamedDataFromProvider: this.getNamedDataFromProvider.bind(this),
             getCurrentHover: () =>
                 this.#interactionController.getCurrentHover(),
-            setDataLoadingStatus: (view, status, detail) =>
-                this.#loadingIndicatorManager.setDataLoadingStatus(
-                    view,
-                    status,
-                    detail
-                ),
             addKeyboardListener: (type, listener) => {
                 // TODO: Listeners should be called only when the mouse pointer is inside the
                 // container or the app covers the full document.
@@ -378,6 +372,8 @@ export default class GenomeSpy {
             VIEW_ROOT_NAME
         );
 
+        this.#loadingStatusRegistry.set(this.viewRoot, "loading");
+
         this.#canvasWrapper.style.flexGrow =
             this.viewRoot.getSize().height.grow > 0 ? "1" : "0";
 
@@ -385,6 +381,7 @@ export default class GenomeSpy {
 
         configureViewHierarchy(this.viewRoot);
         configureViewOpacity(this.viewRoot);
+        this.#logSelectorConstraintWarnings();
 
         // We should now have a complete view hierarchy. Let's update the canvas size
         // and ensure that the loading message is visible.
@@ -405,6 +402,17 @@ export default class GenomeSpy {
         this.#setupDpr();
     }
 
+    #logSelectorConstraintWarnings() {
+        const issues = validateSelectorConstraints(this.viewRoot);
+        if (!issues.length) {
+            return;
+        }
+
+        for (const issue of issues) {
+            console.warn("Selector constraints warning:", issue.message);
+        }
+    }
+
     /**
      * @param {import("./types/viewContext.js").default} context
      */
@@ -435,6 +443,7 @@ export default class GenomeSpy {
      * @returns {Promise<boolean>} true if the launch was successful
      */
     async launch() {
+        let launched = false;
         try {
             this.#prepareContainer();
 
@@ -445,6 +454,7 @@ export default class GenomeSpy {
             this.computeLayout();
             this.animator.requestRender();
 
+            launched = true;
             return true;
         } catch (reason) {
             const message = `${
@@ -456,13 +466,20 @@ export default class GenomeSpy {
                 createMessageBox(this.container, message);
             }
 
+            if (this.viewRoot) {
+                this.#loadingStatusRegistry.set(
+                    this.viewRoot,
+                    "error",
+                    message
+                );
+            }
+
             return false;
         } finally {
             this.#canvasWrapper.classList.remove("loading");
-            // Transition listener doesn't appear to work on observablehq
-            window.setTimeout(() => {
-                this.loadingMessageElement.style.display = "none";
-            }, 2000);
+            if (launched && this.viewRoot) {
+                this.#loadingStatusRegistry.set(this.viewRoot, "complete");
+            }
         }
     }
 
@@ -552,7 +569,10 @@ export default class GenomeSpy {
         /** @type {UnitView[]} */
         const views = [];
         this.viewRoot.visit((view) => {
-            if (view instanceof UnitView && view.getDataAccessor("search")) {
+            if (
+                view instanceof UnitView &&
+                view.getSearchAccessors().length > 0
+            ) {
                 views.push(view);
             }
         });

package/dist/src/marks/mark.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"mark.d.ts","sourceRoot":"","sources":["../../../src/marks/mark.js"],"names":[],"mappings":"AA0DA,mCAAoC,sBAAsB,CAAC;AAC3D,mCAAoC,sBAAsB,CAAC;AAE3D,uCAAwC,oBAAoB,CAAC;AAE7D;;;;;;;;;;;;;GAaG;AAEH;;GAEG;AACH,0BAF0B,CAAC,SAAd,mCAAW;IAuBpB;;OAEG;IACH,sBAFW,OAAO,qBAAqB,EAAE,OAAO,EA4G/C;IAzGG,gDAAwB;IAExB,8EAA8E;IAC9E,UADW,OAAO,CAAC,MAAM,uCAAU,OAAO,qBAAqB,EAAE,OAAO,CAAC,CAAC,CACjD;IAIzB;;;OAGG;IACH,sBAHU,OAAO,SAAS,EAAE,UAAU,GAAG;QAAE,iBAAiB,CAAC,EAAE,MAAM,CAAA;KAAE,CAG5C;IAE3B;;;;;;;OAOG;IACH,2BAHU,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAGG;IAEhC;;;OAGG;IACH,uBAHU,OAAO,SAAS,EAAE,WAAW,CAGX;IAE5B;;;OAGG;IACH,2BAHU,OAAO,SAAS,EAAE,eAAe,CAGX;IAEhC;;;OAGG;IACH,2BAHU,OAAO,SAAS,EAAE,gBAAgB,CAGZ;IAEhC;;;;;OAKG;IACH,2BAHU,OAAO,SAAS,EAAE,gBAAgB,CAGZ;IAEhC;;;;;OAKG;IACH,uCAA+B;IAE/B;;;;;OAKG;IACH,4BAA6B;IAE7B,kFAAkF;IAClF,UADW,QAAQ,CAAC,GAAG,CAAC,CACM;IAG9B,qBAqBE;IAEF;;;;;;OAMG;IACH,qBAHU,CAAC,CAQV;IAGL;;;OAGG;IACH,0CAHW,OAAO,CAAC,CAAC,CAAC,QAQpB;IAED,sBAEC;IAED;;;OAGG;IACH,0BAFa,WAAW,CAIvB;IAED;;;;;OAKG;IACH,2BAHa,OAAO,oBAAoB,EAAE,OAAO,EAAE,CAMlD;IAED;;OAEG;IACH,wBAFa,sCAAS,CAarB;IAED;;OAEG;IACH,4DAcC;IAED;;;;;OAKG;IACH,oGAEC;IAED;;;;;;OAMG;IACH,oDAHW,CAAC,MAAM,CAAC,CAAC,EAAE,QAqCrB;IAED;;;;OAIG;IACH,sDAiDC;IAED,wDAEC;IAED,uDAEC;IAED,uBAEC;IAED;;;OAGG;IACH,2BAEC;IAED;;OAEG;IACH,oCAEC;IAED;;OAEG;IACH,2BAEC;IAED,sEAeC;IAED;;;;;;OAMG;IAEH,6CANW,MAAM,kBACN,MAAM,iBACN,MAAM,EAAE,QAselB;IALG;;;;;;MAIC;IAGL;;;;;;OAMG;IACH,uCA6CC;IAED;;;;;;OAMG;IACH,+CAHW,MAAM,GACJ,CAAS,IAAG,EAAH,GAAG,KAAE,IAAI,CAe9B;IAED;;;;;;;;;;OAUG;IACH,mCALa,CAAC,eACH,MAAM,aACN,CAAC,aACD,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,yCAAU,KAAK,GAAG,QA8BzC;IAED;;OAEG;IACH,2BA+BC;IAED,gBAMC;IAED;;;OAGG;IACH,6BAFW,GAAG,QAyCb;IAED,yBAAyB;IACzB,uDAEC;IAED,yBAAyB;IACzB,iCAEC;IAED,gCAEC;IAED,+BAEC;IAED,yCAEC;IAED;;OAEG;IACH,gCAgBC;IAED;;OAEG;IACH,4CAOC;IAED;;;;;;;;OAQG;IAEH,uBAJW,OAAO,uBAAuB,EAAE,sBAAsB,GACpD,CAAC,MAAM,IAAI,CAAC,EAAE,CA4E1B;IAED;;;;;;OAMG;IACH,qCAJW,oBAAoB,GAClB,OAAO,CAqCnB;IAED;;;;;;;OAOG;IACH,gBAJW,oBAAoB,GAClB,MAAW,IAAI,CAM3B;IAED;;;;OAIG;IACH,2BAJW,YAAY,WACZ,OAAO,WAAW,EAAE,oBAAoB,GACtC,MAAW,IAAI,CAmE3B;IAED;;;;;;;;OAQG;IACH,wBANW;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAC,OAC/B,MAAM,UACN,OAAO,6BAA6B,EAAE,OAAO,aAC7C,OAAO,6BAA6B,EAAE,OAAO,GAC3C,OAAO,CA+GnB;IAED;;;;;;;;;OASG;IACH,qBAJW,MAAM,KACN,OAAO,oBAAoB,EAAE,MAAM,GACjC,GAAG,CAIf;;CACJ;+BAp8CY,OAAO,uBAAuB,EAAE,gBAAgB;;;;;;wBAEnD,OAAO;;mCAEJ,gBAAgB,GAAG,qBAAqB;oCAG1C,MAAM,SACN,MAAM;0BAEJ,YAAY,GAAG,UAAU,GAAG,WAAW;AA47CpD;;;GAGG;AACH,uBAFa,CAAC;IAGV,cAEC;IAkBD;;;OAGG;IACH,2BAFW,GAAG,CAAC,CAAC,EAAE,OAAO,yBAAyB,EAAE,UAAU,CAAC,QAiB9D;CACJ;0BApgDyB,WAAW"}
+{"version":3,"file":"mark.d.ts","sourceRoot":"","sources":["../../../src/marks/mark.js"],"names":[],"mappings":"AA0DA,mCAAoC,sBAAsB,CAAC;AAC3D,mCAAoC,sBAAsB,CAAC;AAE3D,uCAAwC,oBAAoB,CAAC;AAE7D;;;;;;;;;;;;;GAaG;AAEH;;GAEG;AACH,0BAF0B,CAAC,SAAd,mCAAW;IAuBpB;;OAEG;IACH,sBAFW,OAAO,qBAAqB,EAAE,OAAO,EA4G/C;IAzGG,gDAAwB;IAExB,8EAA8E;IAC9E,UADW,OAAO,CAAC,MAAM,uCAAU,OAAO,qBAAqB,EAAE,OAAO,CAAC,CAAC,CACjD;IAIzB;;;OAGG;IACH,sBAHU,OAAO,SAAS,EAAE,UAAU,GAAG;QAAE,iBAAiB,CAAC,EAAE,MAAM,CAAA;KAAE,CAG5C;IAE3B;;;;;;;OAOG;IACH,2BAHU,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAGG;IAEhC;;;OAGG;IACH,uBAHU,OAAO,SAAS,EAAE,WAAW,CAGX;IAE5B;;;OAGG;IACH,2BAHU,OAAO,SAAS,EAAE,eAAe,CAGX;IAEhC;;;OAGG;IACH,2BAHU,OAAO,SAAS,EAAE,gBAAgB,CAGZ;IAEhC;;;;;OAKG;IACH,2BAHU,OAAO,SAAS,EAAE,gBAAgB,CAGZ;IAEhC;;;;;OAKG;IACH,uCAA+B;IAE/B;;;;;OAKG;IACH,4BAA6B;IAE7B,kFAAkF;IAClF,UADW,QAAQ,CAAC,GAAG,CAAC,CACM;IAG9B,qBAqBE;IAEF;;;;;;OAMG;IACH,qBAHU,CAAC,CAQV;IAGL;;;OAGG;IACH,0CAHW,OAAO,CAAC,CAAC,CAAC,QAQpB;IAED,sBAEC;IAED;;;OAGG;IACH,0BAFa,WAAW,CAIvB;IAED;;;;;OAKG;IACH,2BAHa,OAAO,oBAAoB,EAAE,OAAO,EAAE,CAMlD;IAED;;OAEG;IACH,wBAFa,sCAAS,CAarB;IAED;;OAEG;IACH,4DAcC;IAED;;;;;OAKG;IACH,oGAEC;IAED;;;;;;OAMG;IACH,oDAHW,CAAC,MAAM,CAAC,CAAC,EAAE,QAqCrB;IAED;;;;OAIG;IACH,sDAiDC;IAED,wDAEC;IAED,uDAEC;IAED,uBAEC;IAED;;;OAGG;IACH,2BAEC;IAED;;OAEG;IACH,oCAEC;IAED;;OAEG;IACH,2BAEC;IAED,sEAeC;IAED;;;;;;OAMG;IAEH,6CANW,MAAM,kBACN,MAAM,iBACN,MAAM,EAAE,QAselB;IALG;;;;;;MAIC;IAGL;;;;;;OAMG;IACH,uCAwDC;IAED;;;;;;OAMG;IACH,+CAHW,MAAM,GACJ,CAAS,IAAG,EAAH,GAAG,KAAE,IAAI,CAe9B;IAED;;;;;;;;;;OAUG;IACH,mCALa,CAAC,eACH,MAAM,aACN,CAAC,aACD,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,yCAAU,KAAK,GAAG,QA8BzC;IAED;;OAEG;IACH,2BA+BC;IAED,gBAMC;IAED;;;OAGG;IACH,6BAFW,GAAG,QAyCb;IAED,yBAAyB;IACzB,uDAEC;IAED,yBAAyB;IACzB,iCAEC;IAED,gCAEC;IAED,+BAEC;IAED,yCAEC;IAED;;OAEG;IACH,gCAgBC;IAED;;OAEG;IACH,4CAOC;IAED;;;;;;;;OAQG;IAEH,uBAJW,OAAO,uBAAuB,EAAE,sBAAsB,GACpD,CAAC,MAAM,IAAI,CAAC,EAAE,CA4E1B;IAED;;;;;;OAMG;IACH,qCAJW,oBAAoB,GAClB,OAAO,CAiCnB;IAED;;;;;;;OAOG;IACH,gBAJW,oBAAoB,GAClB,MAAW,IAAI,CAM3B;IAED;;;;OAIG;IACH,2BAJW,YAAY,WACZ,OAAO,WAAW,EAAE,oBAAoB,GACtC,MAAW,IAAI,CAmE3B;IAED;;;;;;;;OAQG;IACH,wBANW;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAC,OAC/B,MAAM,UACN,OAAO,6BAA6B,EAAE,OAAO,aAC7C,OAAO,6BAA6B,EAAE,OAAO,GAC3C,OAAO,CA+GnB;IAED;;;;;;;;;OASG;IACH,qBAJW,MAAM,KACN,OAAO,oBAAoB,EAAE,MAAM,GACjC,GAAG,CAIf;;CACJ;+BA38CY,OAAO,uBAAuB,EAAE,gBAAgB;;;;;;wBAEnD,OAAO;;mCAEJ,gBAAgB,GAAG,qBAAqB;oCAG1C,MAAM,SACN,MAAM;0BAEJ,YAAY,GAAG,UAAU,GAAG,WAAW;AAm8CpD;;;GAGG;AACH,uBAFa,CAAC;IAGV,cAEC;IAkBD;;;OAGG;IACH,2BAFW,GAAG,CAAC,CAAC,EAAE,OAAO,yBAAyB,EAAE,UAAU,CAAC,QAiB9D;CACJ;0BA3gDyB,WAAW"}

package/dist/src/marks/mark.js

@@ -940,6 +940,17 @@ export default class Mark {
      * initiated. The idea is to allow for parallel background compilation.
      */
     finalizeGraphicsInitialization() {
+        // Allow duplicate finalization calls when multiple init paths overlap.
+        if (this.programInfo) {
+            return;
+        }
+
+        if (!this.programStatus) {
+            throw new Error(
+                "No program status found! " + this.unitView.getPathString()
+            );
+        }
+
         const error = this.programStatus.getProgramErrors();
         if (error) {
             if (error.detail) {
@@ -1290,21 +1301,17 @@ export default class Mark {
         const locationSetter = this.programInfo.uniformSetters.uSampleFacet;
 
         if (opts && locationSetter) {
-            const pos = opts.locSize ? opts.locSize.location : 0.0;
-            const height = opts.locSize ? opts.locSize.size : 1.0;
+            const scale = opts.pixelToUnit;
+            const pos = opts.locSize.location * scale;
+            const height = opts.locSize.size * scale;
 
             if (pos > 1.0 || pos + height < 0.0) {
                 // Not visible
                 return false;
             }
 
-            const targetPos = opts.targetLocSize
-                ? opts.targetLocSize.location
-                : pos;
-            const targetHeight = opts.targetLocSize
-                ? opts.targetLocSize.size
-                : height;
-
+            // No target locSize in the current API. Keep a consistent shader path
+            // by repeating the current position/height.
             // Use WebGL directly, because twgl uses gl.uniform4fv, which has an
             // inferior performance. Based on profiling, this optimization gives
             // a significant performance boost.
@@ -1313,8 +1320,8 @@ export default class Mark {
                 locationSetter.location, // TODO: Make a twgl pull request to fix typing
                 pos,
                 height,
-                targetPos,
-                targetHeight
+                pos,
+                height
             );
         }
 

package/dist/src/marks/markUtils.js

@@ -38,7 +38,7 @@ export function fixPositional(encoding, channel) {
         if (!secondary) {
             if (primary.type == "quantitative") {
                 // Bar plot, anchor the other end to zero
-                secondary = { datum: 0, contributesToScaleDomain: false };
+                secondary = { datum: 0, domainInert: true };
             } else {
                 secondary = { ...primary };
 

package/dist/src/scale/scale.d.ts

@@ -6,5 +6,10 @@ export function configureScale(_: any, scale: any, logger: any): void;
  * @returns {import("../encoder/encoder.js").VegaScale}
  */
 export default function createScale(_: import("../spec/scale.js").Scale, logger: any): import("../encoder/encoder.js").VegaScale;
-export function configureDomain(scale: any, _: any, logger: any): any;
+export function configureDomain(scale: any, _: any, logger: any): {
+    domain: any;
+    count: any;
+    ordinalUnknown: any;
+    applyOrdinalUnknown: boolean;
+};
 //# sourceMappingURL=scale.d.ts.map

package/dist/src/scale/scale.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"scale.d.ts","sourceRoot":"","sources":["../../../src/scale/scale.js"],"names":[],"mappings":"AAoHA,sEAmBC;AAED;;;;;GAKG;AACH,uCAJW,OAAO,kBAAkB,EAAE,KAAK,UAChC,GAAC,GACC,OAAO,uBAAuB,EAAE,SAAS,CAoBrD;AAgCD,sEAuEC"}
+{"version":3,"file":"scale.d.ts","sourceRoot":"","sources":["../../../src/scale/scale.js"],"names":[],"mappings":"AAgIA,sEAuBC;AAED;;;;;GAKG;AACH,uCAJW,OAAO,kBAAkB,EAAE,KAAK,UAChC,GAAC,GACC,OAAO,uBAAuB,EAAE,SAAS,CAoBrD;AA4CD;;;;;EAuGC"}