@jbrowse/core 5.0.0-beta.2 → 5.0.0-beta.3

package/esm/pluggableElementTypes/models/BaseDisplayModel.d.ts

@@ -161,7 +161,7 @@ export declare const BaseDisplay: import("@jbrowse/mobx-state-tree").IModelType<
      * pointing at" channel a plugin can subscribe to.
      *
      * Declared here because a cross-display consumer can only read a name the
-     * base declares — the same reason `SyntenyFetchStateMixin.fetchInert` is a
+     * base declares — the same reason `FetchMixin.fetchInert` is a
      * hook rather than a getter each display invents. The container used to
      * read `featureUnderMouse`, which only the wiggle, alignments and
      * Manhattan families spelled that way — canvas said `hoveredFeature`,
@@ -382,7 +382,7 @@ export declare const BaseDisplay: import("@jbrowse/mobx-state-tree").IModelType<
              * pointing at" channel a plugin can subscribe to.
              *
              * Declared here because a cross-display consumer can only read a name the
-             * base declares — the same reason `SyntenyFetchStateMixin.fetchInert` is a
+             * base declares — the same reason `FetchMixin.fetchInert` is a
              * hook rather than a getter each display invents. The container used to
              * read `featureUnderMouse`, which only the wiggle, alignments and
              * Manhattan families spelled that way — canvas said `hoveredFeature`,
@@ -595,7 +595,7 @@ export declare const BaseDisplay: import("@jbrowse/mobx-state-tree").IModelType<
              * pointing at" channel a plugin can subscribe to.
              *
              * Declared here because a cross-display consumer can only read a name the
-             * base declares — the same reason `SyntenyFetchStateMixin.fetchInert` is a
+             * base declares — the same reason `FetchMixin.fetchInert` is a
              * hook rather than a getter each display invents. The container used to
              * read `featureUnderMouse`, which only the wiggle, alignments and
              * Manhattan families spelled that way — canvas said `hoveredFeature`,

package/esm/pluggableElementTypes/models/BaseDisplayModel.js

@@ -94,7 +94,7 @@ function stateModelFactory() {
          * pointing at" channel a plugin can subscribe to.
          *
          * Declared here because a cross-display consumer can only read a name the
-         * base declares — the same reason `SyntenyFetchStateMixin.fetchInert` is a
+         * base declares — the same reason `FetchMixin.fetchInert` is a
          * hook rather than a getter each display invents. The container used to
          * read `featureUnderMouse`, which only the wiggle, alignments and
          * Manhattan families spelled that way — canvas said `hoveredFeature`,

package/esm/pluggableElementTypes/models/BaseTrackModel.js

@@ -277,10 +277,7 @@ export function createBaseTrackModel(pm, trackType, baseTrackConfig) {
             const displayConf = getDisplayConf(displays, newDisplayId);
             // same interception showTrackGeneric makes: a registered-but-unloaded
             // display would otherwise fail as an opaque union mismatch below
-            const displayRecord = pm.resolveDisplayTypeRecord(displayConf.type);
-            if (displayRecord && !displayRecord.isStateModelLoaded) {
-                throw new Error(`state model for display type ${displayConf.type} is not loaded yet — await getDisplayType('${displayConf.type}').loadStateModel() first`);
-            }
+            pm.resolveDisplayTypeRecord(displayConf.type)?.assertStateModelLoaded();
             self.displays[idx] = {
                 ...initialSnapshot,
                 type: displayConf.type,

package/esm/pluggableElementTypes/models/BaseViewModel.d.ts

@@ -69,28 +69,28 @@ declare const BaseViewModel: import("@jbrowse/mobx-state-tree").IModelType<{
     menuItems(): MenuItem[];
     /**
      * #getter
-     * Every track open on this view itself: its own `tracks` array plus any
-     * track containers it owns instead (the synteny view keeps one list per
-     * band on `trackContainers` and its own `tracks` empty). Tracks on nested
-     * views are `allTracks`'s, not this getter's.
+     * The tracks this view puts in the census — its own, not a nested view's.
+     *
+     * Empty here, and overridden by the views that have any: the base cannot
+     * read `self.tracks` for them, because what a view keeps under that name
+     * is the view's business. react-msaview's holds its MSA annotation rows —
+     * `{ReactComponent, model}` objects with no configuration and no displays
+     * — and a base that helped itself to them handed them to the readiness
+     * marker as tracks, which error-paged every session holding an MSA view.
      */
     readonly ownTracks: AbstractTrackModel[];
     /**
      * #getter
-     * This view and every view nested inside it, to any depth — a synteny
-     * stack's genome rows, a breakpoint split view's panels. Each view answers
-     * for its own children, so a consumer never walks the nesting itself:
-     * before this getter, four consumers each carried a copy of the walk and
-     * two of them had drifted (one read `levels`, one read `trackContainers`,
-     * and each was blind to the other's spelling).
-     */
-    readonly allViews: AbstractViewModel[];
-} & {
-    /**
-     * #getter
-     * Every track open on this view or any view nested inside it.
+     * The views nested directly inside this one, which the census counts as
+     * views in their own right — a synteny stack's genome rows, a breakpoint
+     * split view's panels.
+     *
+     * Empty here for the same reason, and the dotplot is why it has to be a
+     * declaration rather than a walk of `views`: that prop name holds its two
+     * 1D *axis* models, which are view-shaped and are not views the user
+     * opened. No structural test separates the two — only the view knows.
      */
-    readonly allTracks: AbstractTrackModel[];
+    readonly ownViews: AbstractViewModel[];
 } & {
     /**
      * #action
@@ -199,28 +199,28 @@ export declare const BaseViewModelWithDisplayedRegions: import("@jbrowse/mobx-st
     menuItems(): MenuItem[];
     /**
      * #getter
-     * Every track open on this view itself: its own `tracks` array plus any
-     * track containers it owns instead (the synteny view keeps one list per
-     * band on `trackContainers` and its own `tracks` empty). Tracks on nested
-     * views are `allTracks`'s, not this getter's.
+     * The tracks this view puts in the census — its own, not a nested view's.
+     *
+     * Empty here, and overridden by the views that have any: the base cannot
+     * read `self.tracks` for them, because what a view keeps under that name
+     * is the view's business. react-msaview's holds its MSA annotation rows —
+     * `{ReactComponent, model}` objects with no configuration and no displays
+     * — and a base that helped itself to them handed them to the readiness
+     * marker as tracks, which error-paged every session holding an MSA view.
      */
     readonly ownTracks: AbstractTrackModel[];
     /**
      * #getter
-     * This view and every view nested inside it, to any depth — a synteny
-     * stack's genome rows, a breakpoint split view's panels. Each view answers
-     * for its own children, so a consumer never walks the nesting itself:
-     * before this getter, four consumers each carried a copy of the walk and
-     * two of them had drifted (one read `levels`, one read `trackContainers`,
-     * and each was blind to the other's spelling).
-     */
-    readonly allViews: AbstractViewModel[];
-} & {
-    /**
-     * #getter
-     * Every track open on this view or any view nested inside it.
+     * The views nested directly inside this one, which the census counts as
+     * views in their own right — a synteny stack's genome rows, a breakpoint
+     * split view's panels.
+     *
+     * Empty here for the same reason, and the dotplot is why it has to be a
+     * declaration rather than a walk of `views`: that prop name holds its two
+     * 1D *axis* models, which are view-shaped and are not views the user
+     * opened. No structural test separates the two — only the view knows.
      */
-    readonly allTracks: AbstractTrackModel[];
+    readonly ownViews: AbstractViewModel[];
 } & {
     /**
      * #action

package/esm/pluggableElementTypes/models/BaseViewModel.js

@@ -96,42 +96,31 @@ const BaseViewModel = types
     },
     /**
      * #getter
-     * Every track open on this view itself: its own `tracks` array plus any
-     * track containers it owns instead (the synteny view keeps one list per
-     * band on `trackContainers` and its own `tracks` empty). Tracks on nested
-     * views are `allTracks`'s, not this getter's.
+     * The tracks this view puts in the census — its own, not a nested view's.
+     *
+     * Empty here, and overridden by the views that have any: the base cannot
+     * read `self.tracks` for them, because what a view keeps under that name
+     * is the view's business. react-msaview's holds its MSA annotation rows —
+     * `{ReactComponent, model}` objects with no configuration and no displays
+     * — and a base that helped itself to them handed them to the readiness
+     * marker as tracks, which error-paged every session holding an MSA view.
      */
     get ownTracks() {
-        const s = self;
-        return [
-            ...(s.tracks ?? []),
-            ...(s.trackContainers ?? []).flatMap(c => c.tracks ?? []),
-        ];
-    },
-    /**
-     * #getter
-     * This view and every view nested inside it, to any depth — a synteny
-     * stack's genome rows, a breakpoint split view's panels. Each view answers
-     * for its own children, so a consumer never walks the nesting itself:
-     * before this getter, four consumers each carried a copy of the walk and
-     * two of them had drifted (one read `levels`, one read `trackContainers`,
-     * and each was blind to the other's spelling).
-     */
-    get allViews() {
-        const s = self;
-        return [
-            self,
-            ...(s.views ?? []).flatMap(v => v.allViews ?? []),
-        ];
+        return [];
     },
-}))
-    .views(self => ({
     /**
      * #getter
-     * Every track open on this view or any view nested inside it.
+     * The views nested directly inside this one, which the census counts as
+     * views in their own right — a synteny stack's genome rows, a breakpoint
+     * split view's panels.
+     *
+     * Empty here for the same reason, and the dotplot is why it has to be a
+     * declaration rather than a walk of `views`: that prop name holds its two
+     * 1D *axis* models, which are view-shaped and are not views the user
+     * opened. No structural test separates the two — only the view knows.
      */
-    get allTracks() {
-        return self.allViews.flatMap(v => v.ownTracks);
+    get ownViews() {
+        return [];
     },
 }))
     .actions(self => ({

package/esm/pluggableElementTypes/models/assertDisplayContract.d.ts

@@ -60,8 +60,9 @@ interface RetryContractHost {
     reloadCounter: number;
     fetchInert: boolean;
     /**
-     * Optional: the comparative family has no two-stage fetch, so nothing there
-     * declares it. A family that grows one declares it and gets the deferral.
+     * Optional: chord and the breakpoint overlay have no two-stage fetch, so
+     * neither declares it. A fetch that grows one declares it and gets the
+     * deferral.
      */
     awaitingPrerequisite?: boolean;
     /**
@@ -92,7 +93,7 @@ export declare function takeFetchStarted(self: object): boolean;
  * has changed. Arc shipped exactly that: a `prepare` declining on `dataCurrent`,
  * with the error clearing on click and no arcs ever coming back. The fix this
  * message asks for is what `GlobalFetchMixin.reload()` now does for that whole
- * family — drop `loadedFetchSignature`, so `dataCurrent` goes false and the
+ * family — drop `loadedFetchKey`, so `dataCurrent` goes false and the
  * gate opens.
  *
  * Detected at the moment it happens rather than statically, because the relation

package/esm/pluggableElementTypes/models/assertDisplayContract.js

@@ -109,7 +109,7 @@ export function takeFetchStarted(self) {
  * has changed. Arc shipped exactly that: a `prepare` declining on `dataCurrent`,
  * with the error clearing on click and no arcs ever coming back. The fix this
  * message asks for is what `GlobalFetchMixin.reload()` now does for that whole
- * family — drop `loadedFetchSignature`, so `dataCurrent` goes false and the
+ * family — drop `loadedFetchKey`, so `dataCurrent` goes false and the
  * gate opens.
  *
  * Detected at the moment it happens rather than statically, because the relation
@@ -222,7 +222,7 @@ export function makeRetryContractCheck(self) {
                     `fetch autorun's gate still declines, so Retry is a dead ` +
                     `button — it clears the error and nothing refetches. reload() has ` +
                     `to invalidate whatever that gate reads, not just bump the ` +
-                    `counter (GlobalFetchMixin.reload drops loadedFetchSignature so ` +
+                    `counter (GlobalFetchMixin.reload drops loadedFetchKey so ` +
                     `dataCurrent goes false). If this display is deliberately not ` +
                     `fetching, say so with fetchInert — the loading scrim and the SVG ` +
                     `export read it too; if this run declined only because a ` +

package/esm/pluggableElementTypes/models/baseTrackConfig.js

@@ -61,7 +61,13 @@ export function preprocessTrackConfigSnapshot(pluginManager, snapshot) {
             const canonical = displayAliasMap.get(d.type);
             return canonical ? { ...d, type: canonical } : d;
         })
-            .filter(d => knownDisplayTypes.has(d.type))
+            .filter(d => {
+            const known = knownDisplayTypes.has(d.type);
+            if (!known) {
+                console.warn(`Dropping display of unknown type "${d.type}" from track "${snap.trackId}": no registered display type has that name`);
+            }
+            return known;
+        })
             .filter(d => {
             const dup = seenTypes.has(d.type);
             seenTypes.add(d.type);

package/esm/ui/BaseExportSvgDialog.d.ts

@@ -9,7 +9,7 @@ export interface BaseExportSvgOptions {
 declare const _default: ({ model, handleClose, exportSvg, children, checkboxes, }: {
     model: IAnyStateTreeNode;
     handleClose: () => void;
-    exportSvg: (opts: BaseExportSvgOptions) => Promise<void>;
+    exportSvg: (opts: BaseExportSvgOptions) => Promise<unknown>;
     children?: React.ReactNode;
     checkboxes?: React.ReactNode;
 }) => import("react").JSX.Element;

package/esm/ui/ErrorMessageStackTraceDialog.js

@@ -15,12 +15,11 @@ export default function ErrorMessageStackTraceDialog({ error, onClose, extra, })
     const errorText = error ? `${error}` : '';
     const stackTrace = formatErrorStack(error);
     const { data: mappedStackTrace, isLoading } = useFetch(['mappedStackTrace', stackTrace], () => mapStackTrace(stackTrace));
-    // The rung actually drawing, not a list of the ones available: it already says
-    // which rungs exist, since Canvas2D means neither and WebGL2 means no WebGPU.
-    // It accounts for a `?renderer=` pin and for the GPU banner's "disable GPU",
-    // which matters most in exactly this dialog — a user reporting a rendering bug
-    // has often already clicked that button, and until this moved to render-core
-    // the report they sent still said WebGL2.
+    // The rung createGpuHal built, not a list of the ones available: it already
+    // says which rungs exist, since Canvas2D means neither and WebGL2 means no
+    // WebGPU. It also honours the GPU banner's "disable GPU", which matters most
+    // in exactly this dialog — a user reporting a rendering bug has often already
+    // clicked it.
     const graphicsInfo = graphicsCapabilities
         ? `Graphics: ${effectiveRenderer(graphicsCapabilities)}`
         : '';

package/esm/ui/ExportSvgDialog.d.ts

@@ -1,7 +1,7 @@
 import type { BaseExportSvgOptions } from './BaseExportSvgDialog.tsx';
 export default function ExportSvgDialog({ model, handleClose, }: {
     model: {
-        exportSvg(opts: BaseExportSvgOptions): Promise<void>;
+        exportSvg(opts: BaseExportSvgOptions): Promise<unknown>;
     };
     handleClose: () => void;
 }): import("react").JSX.Element;

package/esm/ui/MenuTypes.d.ts

@@ -66,7 +66,7 @@ export interface BaseMenuItem {
      */
     endAdornment?: React.ReactNode;
     /**
-     * #menuField the "apply this to all open tracks of this type" pin; set it with a promotable builder
+     * #menuField the "apply this to all open tracks of this type" pin; set it through a row builder's `pin` option
      *
      * The trailing "default for all tracks of this type" pin, as a **description**
      * rather than an element — the renderer builds `PinAdornment` from
@@ -76,8 +76,8 @@ export interface BaseMenuItem {
      * into every host's first paint. It did, until 2026-08-05; see
      * reference/EAGER_BUNDLE.md.
      *
-     * Set it through `promotableToggleItem` / `promotableRadioItems` (or
-     * `promotableRadioItem` for a lone row), not by hand.
+     * Set it through the `pin` option of `checkboxItem` / `toggleItem` /
+     * `radioItem` / `radioItems`, not by hand.
      *
      * A `type: 'custom'` row (`makePromotableSizeMenu`) draws its own pin inside
      * its rendered content and still sets this — the declaration is what lets a

package/esm/ui/PinAdornment.js

@@ -18,40 +18,26 @@ const useStyles = makeStyles()(theme => ({
         display: 'inline-flex',
     },
 }));
-// How the pin names what it acts on. Two shapes, decided by the on-value:
-//
-// - **A boolean on-value carries a state**, and the row's label names the
-//   setting rather than a value ("Show legend", "Show soft clipping"). A pin on
-//   an unchecked such row applies the setting *off*, so the value-shaped copy
-//   below stated the opposite of what the click does — and then, once filled,
-//   claimed the setting was on by default when it had just been turned off.
-// - **Everything else IS what the label says**: a radio option ("Compact"), a
-//   size row whose caller folds the value into the label ("Line width (2px)").
-//
-// One `typeof` rather than a flag on the row, because the two forms of
-// `makePin` are exactly this distinction: a symmetric pin over a `maybeBoolean`
-// slot is the only way a pin's value and its label can disagree.
-function pinPredicate(onValue) {
-    return typeof onValue === 'boolean'
-        ? `${onValue ? 'on' : 'off'} by default`
-        : 'the default';
-}
-// The click, which is not the state the pin draws: an outline pin applies the
-// value to the open tracks, a filled one clears the default it stands for.
+// A toggle pin sits on a checkbox row: the copy names the state the click
+// applies, and the fill mirrors the checkbox. A value pin's on-value IS what
+// the label says — a radio option ("Compact"), a size row whose caller folds
+// the value into the label ("Line width (2px)") — and there the fill means the
+// value is the promoted default, so a click clears it.
 //
 // The aria-label carries "of this type" wherever the tooltip does: it used to
 // stop at "for all open tracks", stating a wider blast radius than the control
 // has.
-function pinCopy(label, onValue, isDefault) {
-    return isDefault
+function pinCopy(label, control) {
+    const { kind, active } = control;
+    return kind === 'toggle'
         ? {
-            title: `${label} is ${pinPredicate(onValue)} for all tracks of this type (click to clear)`,
-            ariaLabel: `clear the default for ${label} for all tracks of this type`,
+            title: `Turn ${label} ${active ? 'off' : 'on'} for all open tracks of this type`,
+            ariaLabel: `turn ${label} ${active ? 'off' : 'on'} for all open tracks of this type`,
         }
-        : typeof onValue === 'boolean'
+        : active
             ? {
-                title: `Turn ${label} ${onValue ? 'on' : 'off'} for all open tracks of this type`,
-                ariaLabel: `turn ${label} ${onValue ? 'on' : 'off'} for all open tracks of this type`,
+                title: `${label} is the default for all tracks of this type (click to clear)`,
+                ariaLabel: `clear the default for ${label} for all tracks of this type`,
             }
             : {
                 title: `Apply ${label} to all open tracks of this type`,
@@ -64,22 +50,18 @@ function pinCopy(label, onValue, isDefault) {
 // ToggleButton (native button a11y + a clear selected tint) with a pin —
 // distinct from the value checkbox.
 //
-// **The click and the state are two different things.** One click writes the
-// value into every open track of this display type and raises a snackbar
-// offering to keep it as the display type's default; the filled pin means that
-// default is in place, so a click on a filled pin clears it and touches no
-// track. A default therefore takes two deliberate clicks, which is what it costs
-// to govern every track of the type opened later (ADR-048). Always shown so the
-// capability is discoverable. stopPropagation keeps the click off the row value
-// / menu dismissal. "of this type" because a promoted default is scoped to the
-// display type (e.g. every LinearAlignmentsDisplay), not literally all tracks.
+// One click writes the value into every open track of this display type and
+// raises a snackbar offering to keep it as the display type's default, so a
+// default takes two deliberate clicks (ADR-048). Always shown so the capability
+// is discoverable. stopPropagation keeps the click off the row value / menu
+// dismissal. "of this type" because a promoted default is scoped to the display
+// type (e.g. every LinearAlignmentsDisplay), not literally all tracks.
 export function PinAdornment({ pin, disabled, }) {
     const { classes } = useStyles();
     const { label, control } = pin;
-    const isDefault = control.active;
-    const { title, ariaLabel } = pinCopy(label, control.onValue, isDefault);
-    return (_jsx(Tooltip, { title: title, children: _jsx("span", { className: classes.wrapper, children: _jsx(ToggleButton, { className: classes.button, value: "default", disabled: disabled, selected: isDefault, color: "primary", size: "small", "aria-label": ariaLabel, onChange: e => {
+    const { title, ariaLabel } = pinCopy(label, control);
+    return (_jsx(Tooltip, { title: title, children: _jsx("span", { className: classes.wrapper, children: _jsx(ToggleButton, { className: classes.button, value: "default", disabled: disabled, selected: control.active, color: "primary", size: "small", "aria-label": ariaLabel, onChange: e => {
                     e.stopPropagation();
                     control.toggle();
-                }, children: isDefault ? (_jsx(PushPinIcon, { fontSize: "small" })) : (_jsx(PushPinOutlinedIcon, { fontSize: "small" })) }) }) }));
+                }, children: control.active ? (_jsx(PushPinIcon, { fontSize: "small" })) : (_jsx(PushPinOutlinedIcon, { fontSize: "small" })) }) }) }));
 }

package/esm/ui/ScrollChrome.d.ts

@@ -0,0 +1,16 @@
+/**
+ * The chrome a display that scrolls its content virtually mounts over its
+ * viewport: `ScrollEdgeShadow` saying content is hidden past an edge, and
+ * `VerticalScrollbar` to reach it. Both read the same three numbers, so a
+ * display hands them over once here. Renders nothing when the content fits.
+ */
+export default function ScrollChrome({ scrollTop, setScrollTop, viewportHeight, contentHeight, controlsId, top, }: {
+    scrollTop: number;
+    setScrollTop: (n: number) => void;
+    viewportHeight: number;
+    contentHeight: number;
+    /** `id` of the scrolled viewport element, for `aria-controls`. */
+    controlsId: string;
+    /** Viewport offset from the top, for a display with a sticky band above it. */
+    top?: number;
+}): import("react").JSX.Element;

package/esm/ui/ScrollChrome.js

@@ -0,0 +1,12 @@
+import { Fragment as _Fragment, jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import ScrollEdgeShadow from './ScrollEdgeShadow.js';
+import VerticalScrollbar from './VerticalScrollbar.js';
+/**
+ * The chrome a display that scrolls its content virtually mounts over its
+ * viewport: `ScrollEdgeShadow` saying content is hidden past an edge, and
+ * `VerticalScrollbar` to reach it. Both read the same three numbers, so a
+ * display hands them over once here. Renders nothing when the content fits.
+ */
+export default function ScrollChrome({ scrollTop, setScrollTop, viewportHeight, contentHeight, controlsId, top = 0, }) {
+    return (_jsxs(_Fragment, { children: [_jsx(ScrollEdgeShadow, { scrollTop: scrollTop, viewportHeight: viewportHeight, contentHeight: contentHeight, top: top }), _jsx(VerticalScrollbar, { scrollTop: scrollTop, setScrollTop: setScrollTop, viewportHeight: viewportHeight, contentHeight: contentHeight, controlsId: controlsId, top: top })] }));
+}

package/esm/ui/ScrollZoomToggle.d.ts

@@ -27,9 +27,10 @@ export interface ScrollZoomToggleModel {
  * of this button pulse with it, which is the preference being session-wide made
  * visible.
  *
- * `iconOnly` is for a header where this sits inside a run of icon buttons (the
- * comparative views), where a single labelled button reads as a mistake rather
- * than as emphasis. The tooltip carries the same words either way.
+ * `iconOnly` is for a header with no room for the words, or one where this sits
+ * inside a run of icon buttons with nothing else labelled — a single labelled
+ * button there reads as a mistake rather than as emphasis. The tooltip carries
+ * the same words either way.
  */
 declare const ScrollZoomToggle: ({ model, iconOnly, }: {
     model: ScrollZoomToggleModel;

package/esm/ui/ScrollZoomToggle.js

@@ -67,9 +67,10 @@ const useStyles = makeStyles()(theme => {
  * of this button pulse with it, which is the preference being session-wide made
  * visible.
  *
- * `iconOnly` is for a header where this sits inside a run of icon buttons (the
- * comparative views), where a single labelled button reads as a mistake rather
- * than as emphasis. The tooltip carries the same words either way.
+ * `iconOnly` is for a header with no room for the words, or one where this sits
+ * inside a run of icon buttons with nothing else labelled — a single labelled
+ * button there reads as a mistake rather than as emphasis. The tooltip carries
+ * the same words either way.
  */
 const ScrollZoomToggle = observer(function ScrollZoomToggle({ model, iconOnly, }) {
     const { classes } = useStyles();

package/esm/ui/SizeSliderRow.js

@@ -33,7 +33,7 @@ export const SizeSliderRow = observer(function SizeSliderRow({ title, getValue,
     const slug = title.toLowerCase().replaceAll(' ', '-');
     return (_jsxs("div", { className: classes.root, children: [_jsxs("div", { className: classes.header, children: [_jsxs(Typography, { variant: "caption", color: "textSecondary", className: classes.label, children: [title, ": ", format(value)] }), _jsx(ResetToDefaultButton, { disabled: isDefault, onClick: () => {
                             onReset();
-                        } }), pin ? (_jsx(PinAdornment, { pin: { control: pin, label: `${title} (${format(value)})` } })) : null] }), _jsx(SingleSlider, { value: toSlider(value), min: toSlider(min), max: toSlider(max), step: sliderStep ?? step, size: "small", "aria-label": title.toLowerCase(), "data-testid": `${slug}-slider`, valueLabelDisplay: "auto", valueLabelFormat: (v) => format(fromSlider(v)), sx: { py: 0.5, display: 'block' }, onChange: v => {
+                        } }), pin ? (_jsx(PinAdornment, { pin: { control: pin, label: `${title} (${format(modelValue)})` } })) : null] }), _jsx(SingleSlider, { value: toSlider(value), min: toSlider(min), max: toSlider(max), step: sliderStep ?? step, size: "small", "aria-label": title.toLowerCase(), "data-testid": `${slug}-slider`, valueLabelDisplay: "auto", valueLabelFormat: (v) => format(fromSlider(v)), sx: { py: 0.5, display: 'block' }, onChange: v => {
                     const n = fromSlider(v);
                     if (commitOnRelease) {
                         setDragValue(n);

package/esm/ui/index.d.ts

@@ -48,6 +48,7 @@ export { default as StatusProgressBar } from './StatusProgressBar.tsx';
 export { default as Menu } from './Menu.tsx';
 export { default as PrerenderedCanvas } from './PrerenderedCanvas.tsx';
 export { default as ResizeHandle } from './ResizeHandle.tsx';
+export { default as ScrollChrome } from './ScrollChrome.tsx';
 export { default as ScrollEdgeShadow } from './ScrollEdgeShadow.tsx';
 export { default as VerticalScrollbar } from './VerticalScrollbar.tsx';
 export { default as SubmitDialog } from './SubmitDialog.tsx';
@@ -78,8 +79,7 @@ export { hoverBoxStyle } from './hoverBoxStyle.ts';
 export { useMouseState, useMouseTracking } from './useMouseTracking.ts';
 export type { MouseState, MouseTracker } from './useMouseTracking.ts';
 export { PinAdornment } from './PinAdornment.tsx';
-export { promotableRadioItem, promotableRadioItems, promotableToggleItem, } from './promotableMenuItems.ts';
-export { displayTypesWithPromotableSlots, promotableSlotsWithoutPin, } from './promotablePinCoverage.ts';
+export { displayTypesWithPromotableSlots, misKindedPins, promotableSlotsWithoutPin, } from './promotablePinCoverage.ts';
 export declare const VIEW_HEADER_HEIGHT = 28;
 export { GRADIENT_LEGEND_HEIGHT, GRADIENT_LEGEND_SVG_AREA_WIDTH, GRADIENT_LEGEND_WIDTH, default as SvgGradientLegend, } from './SvgGradientLegend.tsx';
 export type { GradientLabel, GradientStop } from './SvgGradientLegend.tsx';

package/esm/ui/index.js

@@ -44,6 +44,7 @@ export { default as StatusProgressBar } from './StatusProgressBar.js';
 export { default as Menu } from './Menu.js';
 export { default as PrerenderedCanvas } from './PrerenderedCanvas.js';
 export { default as ResizeHandle } from './ResizeHandle.js';
+export { default as ScrollChrome } from './ScrollChrome.js';
 export { default as ScrollEdgeShadow } from './ScrollEdgeShadow.js';
 export { default as VerticalScrollbar } from './VerticalScrollbar.js';
 export { default as SubmitDialog } from './SubmitDialog.js';
@@ -72,8 +73,7 @@ export { default as Crosshairs } from './Crosshairs.js';
 export { hoverBoxStyle } from './hoverBoxStyle.js';
 export { useMouseState, useMouseTracking } from './useMouseTracking.js';
 export { PinAdornment } from './PinAdornment.js';
-export { promotableRadioItem, promotableRadioItems, promotableToggleItem, } from './promotableMenuItems.js';
-// Not on the React-free `menuItems.ts` entry beside the two builders above: this
+// Not on the React-free `menuItems.ts` entry beside the row builders: this
 // reads a config's slot table, so it pulls `configuration/promotableSlots.ts`
 // in, and that entry exists precisely to stay out of every plugin's eager graph.
 //
@@ -81,7 +81,7 @@ export { promotableRadioItem, promotableRadioItems, promotableToggleItem, } from
 // caller has is "which promotable slots does this menu miss", and answering the
 // half of it that lists what a menu *does* offer needs the schema's slot table
 // to mean anything.
-export { displayTypesWithPromotableSlots, promotableSlotsWithoutPin, } from './promotablePinCoverage.js';
+export { displayTypesWithPromotableSlots, misKindedPins, promotableSlotsWithoutPin, } from './promotablePinCoverage.js';
 export const VIEW_HEADER_HEIGHT = 28;
 export { GRADIENT_LEGEND_HEIGHT, GRADIENT_LEGEND_SVG_AREA_WIDTH, GRADIENT_LEGEND_WIDTH, default as SvgGradientLegend, } from './SvgGradientLegend.js';
 export { LEGEND_ROW_HEIGHT, LEGEND_SWATCH, default as SvgColorLegend, } from './SvgColorLegend.js';

package/esm/ui/legendMenuItem.d.ts

@@ -1,5 +1,5 @@
-import type { Pin } from '../configuration/promotablePin.ts';
 import type { MenuItem } from './MenuTypes.ts';
+import type { CheckboxRowOptions } from './toggleMenuItems.ts';
 /**
  * The "Show legend" checkbox, which seven displays build by hand — Hi-C, LD,
  * alignments, LGVSynteny, multi-wiggle, multi-row features, both multi-sample
@@ -14,25 +14,19 @@ import type { MenuItem } from './MenuTypes.ts';
  * values legitimately differ (a Hi-C color scale is off by default, a variant
  * genotype key on), and their descriptions describe genuinely different legends.
  * What they now share is being *promotable* — pass `opts.pin` and the row gains
- * the pin that applies the current state to every open track of this display
- * type, offering the display-type default as its snackbar's one action. Every
+ * the pin that toggles the legend on every open track of this display type,
+ * offering the new state as the display-type default in its snackbar. Every
  * display whose legend is backed by a config slot passes one, and gets it from
- * `LegendMixin`'s `showLegendDisplayTypeDefault` rather than calling `makePin`
- * itself: the slot is the per-display half, the accessors over it are not.
+ * `LegendMixin`'s `showLegendDisplayTypeDefault` rather than calling
+ * `makeTogglePin` itself: the slot is the per-display half, the accessors over
+ * it are not.
  *
  * `pin` is optional because two callers have no slot to promote: the Manhattan
  * plot's LD legend and `LinearBasicDisplay`'s color key are a volatile and a
  * per-legend `dismissed` flag respectively, neither of which is config at all.
- * They get the plain row, which is what `checkboxItem` builds either way — the
- * promotable form is that same row plus the pin, per `promotableToggleItem`.
  *
- * Deliberately not used by `synteny-core`'s "Show color legend", which sits
- * inside a "Color by..." submenu where the bare word would not say which legend.
+ * `synteny-core`'s color-by legend has no toggle at all: it comes up with the
+ * modes that have a key and is dismissed from its own close button.
  */
 /** #menuBuilder showLegendCheckboxItem | the shared "Show legend" checkbox */
-export declare function showLegendCheckboxItem(checked: boolean, onToggle: () => void, opts?: {
-    helpText?: string;
-    disabled?: boolean;
-    disabledHelpText?: string;
-    pin?: Pin;
-}): MenuItem;
+export declare function showLegendCheckboxItem(checked: boolean, onToggle: () => void, opts?: CheckboxRowOptions): MenuItem;