@jbrowse/plugin-alignments 2.13.1 → 2.14.0

package/dist/AlignmentsFeatureDetail/AlignmentsFeatureDetail.js

@@ -19,13 +19,15 @@ const Formatter_1 = __importDefault(require("./Formatter"));
 const omit = ['clipPos', 'flags'];
 const AlignmentsFeatureDetails = (0, mobx_react_1.observer)(function (props) {
     const { model } = props;
-    const feat = (0, clone_1.default)(model.featureData);
+    const { featureData } = model;
+    const feat = (0, clone_1.default)(featureData);
     const SA = (0, util_1.getTag)('SA', feat);
+    const { flags } = feat;
     return (react_1.default.createElement(material_1.Paper, { "data-testid": "alignment-side-drawer" },
         react_1.default.createElement(BaseFeatureDetail_1.FeatureDetails, { ...props, omit: omit, 
             // @ts-expect-error
             descriptions: { ...tagInfo_1.tags, tags: tagInfo_1.tags }, feature: feat, formatter: (value, key) => key === 'next_segment_position' ? (react_1.default.createElement(PairLink_1.default, { model: model, locString: value })) : (react_1.default.createElement(Formatter_1.default, { value: value })) }),
-        SA ? react_1.default.createElement(SuppAlignments_1.default, { model: model, tag: SA, feature: feat }) : null,
-        feat.flags !== undefined ? react_1.default.createElement(Flags_1.default, { feature: feat, ...props }) : null));
+        SA !== undefined ? (react_1.default.createElement(SuppAlignments_1.default, { model: model, tag: SA, feature: feat })) : null,
+        flags !== undefined ? react_1.default.createElement(Flags_1.default, { feature: feat, ...props }) : null));
 });
 exports.default = AlignmentsFeatureDetails;

package/dist/AlignmentsFeatureDetail/BreakendOptionDialog.js

@@ -51,15 +51,21 @@ const BreakendOptionDialog = (0, mobx_react_1.observer)(function ({ model, handl
     const [mirror, setMirror] = (0, react_1.useState)(true);
     return (react_1.default.createElement(ui_1.Dialog, { open: true, onClose: handleClose, title: "Breakpoint split view options" },
         react_1.default.createElement(material_1.DialogContent, null,
-            react_1.default.createElement(Checkbox2, { checked: copyTracks, onChange: event => setCopyTracks(event.target.checked), label: "Copy tracks into the new view" }),
-            react_1.default.createElement(Checkbox2, { checked: mirror, onChange: event => setMirror(event.target.checked), label: "Mirror tracks vertically in vertically stacked view" })),
+            react_1.default.createElement(Checkbox2, { checked: copyTracks, onChange: event => {
+                    setCopyTracks(event.target.checked);
+                }, label: "Copy tracks into the new view" }),
+            react_1.default.createElement(Checkbox2, { checked: mirror, onChange: event => {
+                    setMirror(event.target.checked);
+                }, label: "Mirror tracks vertically in vertically stacked view" })),
         react_1.default.createElement(material_1.DialogActions, null,
             react_1.default.createElement(material_1.Button, { onClick: () => {
                     const { view } = model;
                     const session = (0, util_1.getSession)(model);
                     try {
                         const viewSnapshot = (0, launchBreakpointSplitView_1.getBreakpointSplitView)({ view, f1, f2 });
-                        const [view1, view2] = viewSnapshot.views;
+                        const views = viewSnapshot.views;
+                        const view1 = views[0];
+                        const view2 = views[1];
                         // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-assertion
                         const viewTracks = (0, mobx_state_tree_1.getSnapshot)(view.tracks);
                         session.addView('BreakpointSplitView', {
@@ -84,6 +90,8 @@ const BreakendOptionDialog = (0, mobx_react_1.observer)(function ({ model, handl
                     }
                     handleClose();
                 }, variant: "contained", color: "primary", autoFocus: true }, "OK"),
-            react_1.default.createElement(material_1.Button, { onClick: () => handleClose(), color: "secondary", variant: "contained" }, "Cancel"))));
+            react_1.default.createElement(material_1.Button, { onClick: () => {
+                    handleClose();
+                }, color: "secondary", variant: "contained" }, "Cancel"))));
 });
 exports.default = BreakendOptionDialog;

package/dist/AlignmentsFeatureDetail/Formatter.js

@@ -40,8 +40,12 @@ function Formatter({ value }) {
         react_1.default.createElement("button", { type: "button", onClick: () => {
                 (0, copy_to_clipboard_1.default)(display);
                 setCopied(true);
-                setTimeout(() => setCopied(false), 700);
+                setTimeout(() => {
+                    setCopied(false);
+                }, 700);
             } }, copied ? 'Copied to clipboard' : 'Copy'),
-        react_1.default.createElement("button", { type: "button", onClick: () => setShow(val => !val) }, show ? 'Show less' : 'Show more'),
+        react_1.default.createElement("button", { type: "button", onClick: () => {
+                setShow(val => !val);
+            } }, show ? 'Show less' : 'Show more'),
         react_1.default.createElement("div", null, show ? display : `${display.slice(0, 100)}...`))) : (react_1.default.createElement("div", null, display));
 }

package/dist/AlignmentsFeatureDetail/SuppAlignmentsLocStrings.js

@@ -15,7 +15,7 @@ function SuppAlignmentsLocStrings({ tag, model, }) {
         react_1.default.createElement("ul", null, tag
             .split(';')
             .filter(SA => !!SA)
-            .map((SA, index) => {
+            .map((SA, idx) => {
             const [saRef, saStart, saStrand, saCigar] = SA.split(',');
             const saLength = (0, MismatchParser_1.getLengthOnRef)(saCigar);
             const extra = Math.floor(saLength / 5);
@@ -27,7 +27,9 @@ function SuppAlignmentsLocStrings({ tag, model, }) {
             const displayStart = start.toLocaleString('en-US');
             const displayEnd = end.toLocaleString('en-US');
             const displayString = `${saRef}:${displayStart}-${displayEnd} (${saStrand}) [${saLength}bp]`;
-            return (react_1.default.createElement("li", { key: `${locString}-${index}` },
+            return (
+            /* biome-ignore lint/suspicious/noArrayIndexKey: */
+            react_1.default.createElement("li", { key: `${locString}-${idx}` },
                 react_1.default.createElement(material_1.Link, { href: "#", onClick: async (event) => {
                         event.preventDefault();
                         // eslint-disable-next-line @typescript-eslint/no-floating-promises

package/dist/AlignmentsFeatureDetail/getSAFeatures.js

@@ -12,9 +12,9 @@ async function getSAFeatures({ view, feature, }) {
     const SA = (0, util_2.getTag)(feature, 'SA') || '';
     const readName = feature.get('name');
     const clipPos = (0, MismatchParser_1.getClip)(cigar, 1);
-    // get the canonical refname for the read because if the
-    // read.get('refName') is chr1 and the actual fasta refName is 1 then no
-    // tracks can be opened on the top panel of the linear read vs ref
+    // get the canonical refname for the read because if the read.get('refName')
+    // is chr1 and the actual fasta refName is 1 then no tracks can be opened on
+    // the top panel of the linear read vs ref
     const assembly = await assemblyManager.waitForAssembly(view.assemblyNames[0]);
     if (!assembly) {
         throw new Error('assembly not found');
@@ -30,7 +30,7 @@ async function getSAFeatures({ view, feature, }) {
     };
     const features = [feat, ...suppAlns];
     features.forEach((f, idx) => {
-        f.refName = (assembly === null || assembly === void 0 ? void 0 : assembly.getCanonicalRefName(f.refName)) || f.refName;
+        f.refName = assembly.getCanonicalRefName(f.refName) || f.refName;
         f.syntenyId = idx;
         f.mate.syntenyId = idx;
         f.mate.uniqueId = `${f.uniqueId}_mate`;

package/dist/AlignmentsFeatureDetail/launchBreakpointSplitView.js

@@ -15,7 +15,7 @@ function getBreakpointSplitView({ f1, f2, view, }) {
     const topRegion = assembly.regions.find(f => f.refName === f1.refName);
     const bottomRegion = assembly.regions.find(f => f.refName === f2.refName);
     if (!topRegion || !bottomRegion) {
-        throw new Error(`unable to find the refName for the top or bottom of the breakpoint view`);
+        throw new Error('unable to find the refName for the top or bottom of the breakpoint view');
     }
     const topMarkedRegion = [{ ...topRegion }, { ...topRegion }];
     const bottomMarkedRegion = [{ ...bottomRegion }, { ...bottomRegion }];
@@ -44,6 +44,6 @@ function getBreakpointSplitView({ f1, f2, view, }) {
                 offsetPx: (bottomRegion.start + e) / bpPerPx,
             },
         ],
-        displayName: `breakend split detail`,
+        displayName: 'breakend split detail',
     };
 }

package/dist/BamAdapter/BamAdapter.js

@@ -13,8 +13,8 @@ const rxjs_2 = require("rxjs");
 // locals
 const BamSlightlyLazyFeature_1 = __importDefault(require("./BamSlightlyLazyFeature"));
 class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
-    // derived classes may not use the same configuration so a custom
-    // configure method allows derived classes to override this behavior
+    // derived classes may not use the same configuration so a custom configure
+    // method allows derived classes to override this behavior
     async configurePre() {
         const bamLocation = this.getConf('bamLocation');
         const location = this.getConf(['index', 'location']);
@@ -25,10 +25,7 @@ class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
             bamFilehandle: (0, io_1.openLocation)(bamLocation, pm),
             csiFilehandle: csi ? (0, io_1.openLocation)(location, pm) : undefined,
             baiFilehandle: !csi ? (0, io_1.openLocation)(location, pm) : undefined,
-            // chunkSizeLimit and fetchSizeLimit are more troublesome than
-            // helpful, and have given overly large values on the ultra long
-            // nanopore reads even with 500MB limits, so disabled with infinity
-            yieldThreadTime: Infinity,
+            yieldThreadTime: Number.POSITIVE_INFINITY,
         });
         const adapterConfig = this.getConf('sequenceAdapter');
         if (adapterConfig && this.getSubAdapter) {
@@ -38,13 +35,11 @@ class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
                 sequenceAdapter: dataAdapter,
             };
         }
-        else {
-            return { bam };
-        }
+        return { bam };
     }
     async configure() {
         if (!this.configureP) {
-            this.configureP = this.configurePre().catch(e => {
+            this.configureP = this.configurePre().catch((e) => {
                 this.configureP = undefined;
                 throw e;
             });
@@ -79,7 +74,7 @@ class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
     }
     async setup(opts) {
         if (!this.setupP) {
-            this.setupP = this.setupPre(opts).catch(e => {
+            this.setupP = this.setupPre(opts).catch((e) => {
                 this.setupP = undefined;
                 throw e;
             });
@@ -142,10 +137,11 @@ class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
                         continue;
                     }
                     if (tagFilter) {
-                        const v = record.get(tagFilter.tag);
-                        if (!(tagFilter.value === '*'
-                            ? v !== undefined
-                            : `${v}` === tagFilter.value)) {
+                        const readVal = record.get(tagFilter.tag);
+                        const filterVal = tagFilter.value;
+                        if (filterVal === '*'
+                            ? readVal !== undefined
+                            : `${readVal}` !== `${filterVal}`) {
                             continue;
                         }
                     }
@@ -166,9 +162,7 @@ class BamAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
             const fetchSizeLimit = this.getConf('fetchSizeLimit');
             return { bytes, fetchSizeLimit };
         }
-        else {
-            return super.getMultiRegionFeatureDensityStats(regions, opts);
-        }
+        return super.getMultiRegionFeatureDensityStats(regions, opts);
     }
     freeResources( /* { region } */) { }
     // depends on setup being called before the BAM constructor

package/dist/BamAdapter/BamSlightlyLazyFeature.js

@@ -62,7 +62,6 @@ class BamSlightlyLazyFeature {
     id() {
         return `${this.adapter.id}-${this.record.id()}`;
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     get(field) {
         const methodName = `_get_${field}`;
         // @ts-expect-error

package/dist/CramAdapter/CramAdapter.d.ts

@@ -29,7 +29,7 @@ export default class CramAdapter extends BaseFeatureDataAdapter {
     getRefNames(opts?: BaseOptions): Promise<string[]>;
     refNameToId(refName: string): number | undefined;
     refIdToName(refId: number): string | undefined;
-    refIdToOriginalName(refId: number): string;
+    refIdToOriginalName(refId: number): string | undefined;
     getFeatures(region: Region & {
         originalRefName?: string;
     }, opts?: BaseOptions & {

package/dist/CramAdapter/CramAdapter.js

@@ -44,7 +44,7 @@ class CramAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
     }
     async configure() {
         if (!this.configureP) {
-            this.configureP = this.configurePre().catch(e => {
+            this.configureP = this.configurePre().catch((e) => {
                 this.configureP = undefined;
                 throw e;
             });
@@ -118,7 +118,7 @@ class CramAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
     }
     async setup(opts) {
         if (!this.setupP) {
-            this.setupP = this.setupPre(opts).catch(e => {
+            this.setupP = this.setupPre(opts).catch((e) => {
                 this.setupP = undefined;
                 throw e;
             });
@@ -177,12 +177,13 @@ class CramAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
                         continue;
                     }
                     if (tagFilter) {
-                        const v = tagFilter.tag === 'RG'
+                        const readVal = tagFilter.tag === 'RG'
                             ? (_a = samHeader.readGroups) === null || _a === void 0 ? void 0 : _a[record.readGroupId]
                             : record.tags[tagFilter.tag];
-                        if (!(tagFilter.value === '*'
-                            ? v !== undefined
-                            : `${v}` === tagFilter.value)) {
+                        const filterVal = tagFilter.value;
+                        if (filterVal === '*'
+                            ? readVal !== undefined
+                            : `${readVal}` !== `${filterVal}`) {
                             continue;
                         }
                     }

package/dist/CramAdapter/CramTestAdapters.js

@@ -19,7 +19,11 @@ function parseSmallFasta(text) {
         const [id, ...descriptionLines] = defLine.split(' ');
         const description = descriptionLines.join(' ');
         const sequence = seqLines.join('').replaceAll(/\s/g, '');
-        return { id, description, sequence };
+        return {
+            id: id,
+            description,
+            sequence,
+        };
     });
 }
 class FetchableSmallFasta {
@@ -71,7 +75,9 @@ class SequenceAdapter extends BaseAdapter_1.BaseFeatureDataAdapter {
                 }));
                 observer.complete();
             })
-                .catch(e => observer.error(e));
+                .catch((e) => {
+                observer.error(e);
+            });
             return { unsubscribe: () => { } };
         });
     }

package/dist/GuessAlignmentsTypes/index.js

@@ -16,7 +16,7 @@ function GuessAlignmentsTypesF(pluginManager) {
             if (regexGuess.test(fileName) && !adapterHint) {
                 return obj;
             }
-            else if (adapterHint === adapterName) {
+            if (adapterHint === adapterName) {
                 return obj;
             }
             return adapterGuesser(file, index, adapterHint);
@@ -39,7 +39,7 @@ function GuessAlignmentsTypesF(pluginManager) {
             if (regexGuess.test(fileName) && !adapterHint) {
                 return obj;
             }
-            else if (adapterHint === adapterName) {
+            if (adapterHint === adapterName) {
                 return obj;
             }
             return adapterGuesser(file, index, adapterHint);

package/dist/LinearAlignmentsDisplay/components/AlignmentsDisplay.d.ts

@@ -1,6 +1,6 @@
 import React from 'react';
-import { AlignmentsDisplayModel } from '../models/model';
+import { LinearAlignmentsDisplayModel } from '../models/model';
 declare const AlignmentsDisplay: ({ model, }: {
-    model: AlignmentsDisplayModel;
+    model: LinearAlignmentsDisplayModel;
 }) => React.JSX.Element | null;
 export default AlignmentsDisplay;

package/dist/LinearAlignmentsDisplay/models/model.d.ts

@@ -4,6 +4,7 @@ import { AnyConfigurationModel, AnyConfigurationSchemaType } from '@jbrowse/core
 import PluginManager from '@jbrowse/core/PluginManager';
 import { MenuItem } from '@jbrowse/core/ui';
 import { FeatureDensityStats } from '@jbrowse/core/data_adapters/BaseAdapter';
+import { IFilter } from '../../shared';
 /**
  * #stateModel LinearAlignmentsDisplay
  * extends
@@ -46,7 +47,7 @@ declare function stateModelFactory(pluginManager: PluginManager, configSchema: A
             error: unknown;
             message: string | undefined;
         }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
-        onHorizontalScroll?: Function;
+        onHorizontalScroll?: () => void;
         blockState?: Record<string, any>;
     }>;
     readonly DisplayBlurb: React.FC<{
@@ -59,26 +60,14 @@ declare function stateModelFactory(pluginManager: PluginManager, configSchema: A
             error: unknown;
             message: string | undefined;
         } & import("mobx-state-tree").IStateTreeNode<import("mobx-state-tree").IModelType<{
-            id: import("mobx-state-tree").IOptionalIType<import(
-            /**
-             * #action
-             */
-            "mobx-state-tree").ISimpleType<string>, [undefined]>;
-            type: import("mobx-state-tree" /**
-             * #getter
-             */).ISimpleType<string>;
-            rpcDriverName: import("mobx-state-tree" /**
-             * #getter
-             */).IMaybe<import("mobx-state-tree").ISimpleType<string>>;
+            id: import("mobx-state-tree").IOptionalIType<import("mobx-state-tree").ISimpleType<string>, [undefined]>;
+            type: import("mobx-state-tree").ISimpleType<string>;
+            rpcDriverName: import("mobx-state-tree").IMaybe<import("mobx-state-tree").ISimpleType<string>>;
         }, {
             rendererTypeName: string;
             error: unknown;
             message: string | undefined;
-        }, import("mobx-state-tree")._NotCustomized, import(
-        /**
-         * #getter
-         */
-        "mobx-state-tree")._NotCustomized>>;
+        }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
     }> | null;
     readonly adapterConfig: any;
     readonly parentTrack: any;
@@ -131,7 +120,7 @@ declare function stateModelFactory(pluginManager: PluginManager, configSchema: A
      */
     readonly features: any;
     /**
-     * #getter
+     * #getteralignmentsdisplaymodel
      */
     readonly DisplayBlurb: any;
     /**
@@ -159,6 +148,10 @@ declare function stateModelFactory(pluginManager: PluginManager, configSchema: A
      * #action
      */
     setHeight(n: number): number;
+    /**
+     * #action
+     */
+    setFilterBy(filter: IFilter): void;
     /**
      * #action
      */
@@ -195,5 +188,5 @@ declare function stateModelFactory(pluginManager: PluginManager, configSchema: A
     lowerPanelType: import("mobx-state-tree").IType<string | undefined, string, string>;
 }>>, import("mobx-state-tree")._NotCustomized>;
 export default stateModelFactory;
-export type AlignmentsDisplayStateModel = ReturnType<typeof stateModelFactory>;
-export type AlignmentsDisplayModel = Instance<AlignmentsDisplayStateModel>;
+export type LinearAlignmentsDisplayStateModel = ReturnType<typeof stateModelFactory>;
+export type LinearAlignmentsDisplayModel = Instance<LinearAlignmentsDisplayStateModel>;

package/dist/LinearAlignmentsDisplay/models/model.js

@@ -116,7 +116,7 @@ function stateModelFactory(pluginManager, configSchema) {
             return self.PileupDisplay.features;
         },
         /**
-         * #getter
+         * #getteralignmentsdisplaymodel
          */
         get DisplayBlurb() {
             var _a;
@@ -173,6 +173,13 @@ function stateModelFactory(pluginManager, configSchema) {
             self.heightPreConfig = Math.max(n, minDisplayHeight);
             return self.heightPreConfig;
         },
+        /**
+         * #action
+         */
+        setFilterBy(filter) {
+            self.PileupDisplay.setFilterBy(filter);
+            self.SNPCoverageDisplay.setFilterBy(filter);
+        },
         /**
          * #action
          */
@@ -244,7 +251,9 @@ function stateModelFactory(pluginManager, configSchema) {
                     type: 'radio',
                     label: d.displayName,
                     checked: d.name === self.PileupDisplay.type,
-                    onClick: () => self.setLowerPanelType(d.name),
+                    onClick: () => {
+                        self.setLowerPanelType(d.name);
+                    },
                 }));
                 return [
                     ...superTrackMenuItems(),
@@ -260,7 +269,7 @@ function stateModelFactory(pluginManager, configSchema) {
                     },
                     {
                         type: 'subMenu',
-                        label: `Replace lower panel with...`,
+                        label: 'Replace lower panel with...',
                         subMenu: extra,
                     },
                 ];
@@ -268,6 +277,7 @@ function stateModelFactory(pluginManager, configSchema) {
         };
     })
         .preProcessSnapshot(snap => {
+        // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
         if (!snap) {
             return snap;
         }

package/dist/LinearPileupDisplay/SharedLinearPileupDisplayMixin.d.ts

@@ -138,7 +138,7 @@ export declare function SharedLinearPileupDisplayMixin(configSchema: AnyConfigur
         readName: import("mobx-state-tree").IMaybe<import("mobx-state-tree").ISimpleType<string>>;
         tagFilter: import("mobx-state-tree").IMaybe<import("mobx-state-tree").IModelType<{
             tag: import("mobx-state-tree").ISimpleType<string>;
-            value: import("mobx-state-tree").ISimpleType<string>;
+            value: import("mobx-state-tree").IMaybe<import("mobx-state-tree").ISimpleType<string>>;
         }, {}, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
     }, {}, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>, [undefined]>;
     /**
@@ -168,7 +168,7 @@ export declare function SharedLinearPileupDisplayMixin(configSchema: AnyConfigur
             error: unknown;
             message: string | undefined;
         }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
-        onHorizontalScroll?: Function;
+        onHorizontalScroll?: () => void;
         blockState?: Record<string, any>;
     }>;
     readonly DisplayBlurb: import("react").FC<{
@@ -181,7 +181,9 @@ export declare function SharedLinearPileupDisplayMixin(configSchema: AnyConfigur
             error: unknown;
             message: string | undefined;
         } & import("mobx-state-tree").IStateTreeNode<import("mobx-state-tree").IModelType<{
-            id: import("mobx-state-tree").IOptionalIType<import("mobx-state-tree").ISimpleType<string>, [undefined]>;
+            id: import("mobx-state-tree" /**
+             * #property
+             */).IOptionalIType<import("mobx-state-tree").ISimpleType<string>, [undefined]>;
             type: import("mobx-state-tree").ISimpleType<string>;
             rpcDriverName: import("mobx-state-tree").IMaybe<import("mobx-state-tree").ISimpleType<string>>;
         }, {
@@ -284,7 +286,7 @@ export declare function SharedLinearPileupDisplayMixin(configSchema: AnyConfigur
     /**
      * #action
      */
-    setMaxHeight(n: number): void;
+    setMaxHeight(n?: number): void;
     /**
      * #action
      */
@@ -417,6 +419,7 @@ export declare function SharedLinearPileupDisplayMixin(configSchema: AnyConfigur
         icon: import("@mui/material/OverridableComponent").OverridableComponent<import("@mui/material").SvgIconTypeMap<{}, "svg">> & {
             muiName: string;
         };
+        priority: number;
         onClick: () => void;
     })[];
 } & {

package/dist/LinearPileupDisplay/SharedLinearPileupDisplayMixin.js

@@ -45,11 +45,11 @@ const LinearPileupDisplayBlurb_1 = __importDefault(require("./components/LinearP
 const shared_1 = require("../shared");
 const util_2 = require("../util");
 const color_1 = require("../shared/color");
-// async
-const FilterByTagDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('../shared/FilterByTag'))));
-const ColorByTagDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/ColorByTag'))));
-const SetFeatureHeightDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SetFeatureHeight'))));
-const SetMaxHeightDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SetMaxHeight'))));
+// lazies
+const FilterByTagDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('../shared/FilterByTagDialog'))));
+const ColorByTagDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/ColorByTagDialog'))));
+const SetFeatureHeightDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SetFeatureHeightDialog'))));
+const SetMaxHeightDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SetMaxHeightDialog'))));
 // using a map because it preserves order
 const rendererTypes = new Map([
     ['pileup', 'PileupRenderer'],
@@ -291,18 +291,14 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                             icon: MenuOpen_1.default,
                             onClick: () => {
                                 self.clearFeatureSelection();
-                                if (feat) {
-                                    self.selectFeature(feat);
-                                }
+                                self.selectFeature(feat);
                             },
                         },
                         {
                             label: 'Copy info to clipboard',
                             icon: Icons_1.ContentCopy,
                             onClick: () => {
-                                if (feat) {
-                                    self.copyFeatureToClipboard(feat);
-                                }
+                                self.copyFeatureToClipboard(feat);
                             },
                         },
                     ]
@@ -395,27 +391,39 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                 return [
                     {
                         label: 'Normal',
-                        onClick: () => self.setColorScheme({ type: 'normal' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'normal' });
+                        },
                     },
                     {
                         label: 'Mapping quality',
-                        onClick: () => self.setColorScheme({ type: 'mappingQuality' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'mappingQuality' });
+                        },
                     },
                     {
                         label: 'Strand',
-                        onClick: () => self.setColorScheme({ type: 'strand' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'strand' });
+                        },
                     },
                     {
                         label: 'Per-base quality',
-                        onClick: () => self.setColorScheme({ type: 'perBaseQuality' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'perBaseQuality' });
+                        },
                     },
                     {
                         label: 'Per-base lettering',
-                        onClick: () => self.setColorScheme({ type: 'perBaseLettering' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'perBaseLettering' });
+                        },
                     },
                     {
                         label: 'First-of-pair strand',
-                        onClick: () => self.setColorScheme({ type: 'stranded' }),
+                        onClick: () => {
+                            self.setColorScheme({ type: 'stranded' });
+                        },
                     },
                     {
                         label: 'Color by tag...',
@@ -435,8 +443,9 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                 return [
                     ...superTrackMenuItems(),
                     {
-                        label: 'Filter by',
+                        label: 'Filter by...',
                         icon: ClearAll_1.default,
+                        priority: -1,
                         onClick: () => {
                             (0, util_1.getSession)(self).queueDialog(doneCallback => [
                                 FilterByTagDialog,
@@ -445,7 +454,8 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                         },
                     },
                     {
-                        label: 'Set feature height',
+                        label: 'Set feature height...',
+                        priority: -1,
                         subMenu: [
                             {
                                 label: 'Normal',
@@ -500,7 +510,11 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                 const { colorBy, tagsReady } = self;
                 const { staticBlocks } = view;
                 if ((colorBy === null || colorBy === void 0 ? void 0 : colorBy.tag) && !tagsReady) {
-                    const vals = await (0, shared_1.getUniqueTagValues)(self, colorBy, staticBlocks);
+                    const vals = await (0, shared_1.getUniqueTagValues)({
+                        self,
+                        tag: colorBy.tag,
+                        blocks: staticBlocks,
+                    });
                     self.updateColorTagMap(vals);
                 }
                 self.setTagsReady(true);
@@ -530,7 +544,8 @@ function SharedLinearPileupDisplayMixin(configSchema) {
                             // feature.id that was returned e.g. that the user hasn't
                             // moused over to a new position during the async operation
                             // above
-                            if (self.featureIdUnderMouse === (feature === null || feature === void 0 ? void 0 : feature.uniqueId)) {
+                            if (feature &&
+                                self.featureIdUnderMouse === feature.uniqueId) {
                                 self.setFeatureUnderMouse(new util_1.SimpleFeature(feature));
                             }
                         }

package/dist/LinearPileupDisplay/components/{ColorByModifications.js → ColorByModificationsDialog.js}

@@ -34,6 +34,8 @@ const ColorByModificationsDialog = (0, mobx_react_1.observer)(function ({ model,
                         model.setColorScheme({ type: 'methylation' });
                         handleClose();
                     } }, "Methylation"),
-                react_1.default.createElement(material_1.Button, { variant: "contained", color: "secondary", onClick: () => handleClose() }, "Cancel")))));
+                react_1.default.createElement(material_1.Button, { variant: "contained", color: "secondary", onClick: () => {
+                        handleClose();
+                    } }, "Cancel")))));
 });
 exports.default = ColorByModificationsDialog;