@jbrowse/plugin-linear-genome-view 2.4.1 → 2.5.0

package/dist/LinearGenomeView/components/util.js

@@ -39,7 +39,7 @@ exports.getRelativeX = getRelativeX;
 function getCytobands(assembly, refName) {
     var _a;
     return (((_a = assembly === null || assembly === void 0 ? void 0 : assembly.cytobands) === null || _a === void 0 ? void 0 : _a.map(f => ({
-        refName: assembly.getCanonicalRefName(f.get('refName')),
+        refName: assembly.getCanonicalRefName(f.get('refName')) || f.get('refName'),
         start: f.get('start'),
         end: f.get('end'),
         type: f.get('type'),

package/dist/LinearGenomeView/components/util.js.map

@@ -1 +1 @@
-{"version":3,"file":"util.js","sourceRoot":"","sources":["../../../src/LinearGenomeView/components/util.ts"],"names":[],"mappings":";;;;;;AAGA,6CAA8D;AAC9D,uFAA6D;AAEtD,KAAK,UAAU,YAAY,CAAC,EACjC,WAAW,EACX,UAAU,EACV,WAAW,EACX,iBAAiB,EACjB,iBAAiB,EACjB,QAAQ,GAQT;;IACC,IAAI,CAAC,iBAAiB,EAAE;QACtB,OAAO,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;KACvC;IAED,MAAM,iBAAiB,GAAG,MAAM,CAAA,iBAAiB,aAAjB,iBAAiB,uBAAjB,iBAAiB,CAAE,MAAM,CACvD;QACE,WAAW;QACX,UAAU;KACX,EACD,WAAW,EACX,iBAAiB,CAClB,CAAA,CAAA;IAED,MAAM,cAAc,GAAG,MAAA,QAAQ,aAAR,QAAQ,uBAAR,QAAQ,CAAE,WAAW,0CACxC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,WAAW,EAAE,CAAC,UAAU,CAAC,WAAW,CAAC,WAAW,EAAE,CAAC,EACtE,KAAK,CAAC,CAAC,EAAE,EAAE,EACX,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,qBAAU,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAA;IAEzC,OAAO,IAAA,aAAM,EACX,CAAC,GAAG,CAAC,cAAc,IAAI,EAAE,CAAC,EAAE,GAAG,CAAC,iBAAiB,IAAI,EAAE,CAAC,CAAC,EACzD,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,KAAK,EAAE,CACnB,CAAA;AACH,CAAC;AArCD,oCAqCC;AAED,6CAA6C;AAC7C,SAAgB,SAAS,CAAC,GAAW,EAAE,KAAa;IAClD,MAAM,SAAS,GAAG,GAAG,CAAC,WAAW,CAAC,KAAK,CAAC,CAAA;IACxC,IAAI,SAAS,KAAK,CAAC,CAAC,EAAE;QACpB,OAAO,CAAC,GAAG,EAAE,EAAE,CAAC,CAAA;KACjB;SAAM;QACL,MAAM,MAAM,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,SAAS,CAAC,CAAA;QACtC,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAA;QACtC,OAAO,CAAC,MAAM,EAAE,KAAK,CAAC,CAAA;KACvB;AACH,CAAC;AATD,8BASC;AAED,SAAgB,YAAY,CAE1B,KAAQ,EAAE,OAA2B;IACrC,OAAO,KAAK,CAAC,OAAO,GAAG,CAAC,CAAA,OAAO,aAAP,OAAO,uBAAP,OAAO,CAAE,qBAAqB,GAAG,IAAI,KAAI,CAAC,CAAC,CAAA;AACrE,CAAC;AAJD,oCAIC;AAED,SAAgB,YAAY,CAAC,QAA8B,EAAE,OAAe;;IAC1E,OAAO,CACL,CAAA,MAAA,QAAQ,aAAR,QAAQ,uBAAR,QAAQ,CAAE,SAAS,0CACf,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACV,OAAO,EAAE,QAAQ,CAAC,mBAAmB,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;QACvD,KAAK,EAAE,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC;QACrB,GAAG,EAAE,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC;QACjB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC;KACpB,CAAC,EACD,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,OAAO,CAAC,KAAI,EAAE,CAC5C,CAAA;AACH,CAAC;AAXD,oCAWC"}
+{"version":3,"file":"util.js","sourceRoot":"","sources":["../../../src/LinearGenomeView/components/util.ts"],"names":[],"mappings":";;;;;;AAGA,6CAA8D;AAC9D,uFAA6D;AAEtD,KAAK,UAAU,YAAY,CAAC,EACjC,WAAW,EACX,UAAU,EACV,WAAW,EACX,iBAAiB,EACjB,iBAAiB,EACjB,QAAQ,GAQT;;IACC,IAAI,CAAC,iBAAiB,EAAE;QACtB,OAAO,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;KACvC;IAED,MAAM,iBAAiB,GAAG,MAAM,CAAA,iBAAiB,aAAjB,iBAAiB,uBAAjB,iBAAiB,CAAE,MAAM,CACvD;QACE,WAAW;QACX,UAAU;KACX,EACD,WAAW,EACX,iBAAiB,CAClB,CAAA,CAAA;IAED,MAAM,cAAc,GAAG,MAAA,QAAQ,aAAR,QAAQ,uBAAR,QAAQ,CAAE,WAAW,0CACxC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,WAAW,EAAE,CAAC,UAAU,CAAC,WAAW,CAAC,WAAW,EAAE,CAAC,EACtE,KAAK,CAAC,CAAC,EAAE,EAAE,EACX,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,qBAAU,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC,CAAA;IAEzC,OAAO,IAAA,aAAM,EACX,CAAC,GAAG,CAAC,cAAc,IAAI,EAAE,CAAC,EAAE,GAAG,CAAC,iBAAiB,IAAI,EAAE,CAAC,CAAC,EACzD,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,KAAK,EAAE,CACnB,CAAA;AACH,CAAC;AArCD,oCAqCC;AAED,6CAA6C;AAC7C,SAAgB,SAAS,CAAC,GAAW,EAAE,KAAa;IAClD,MAAM,SAAS,GAAG,GAAG,CAAC,WAAW,CAAC,KAAK,CAAC,CAAA;IACxC,IAAI,SAAS,KAAK,CAAC,CAAC,EAAE;QACpB,OAAO,CAAC,GAAG,EAAE,EAAE,CAAC,CAAA;KACjB;SAAM;QACL,MAAM,MAAM,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,SAAS,CAAC,CAAA;QACtC,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAA;QACtC,OAAO,CAAC,MAAM,EAAE,KAAK,CAAC,CAAA;KACvB;AACH,CAAC;AATD,8BASC;AAED,SAAgB,YAAY,CAE1B,KAAQ,EAAE,OAA2B;IACrC,OAAO,KAAK,CAAC,OAAO,GAAG,CAAC,CAAA,OAAO,aAAP,OAAO,uBAAP,OAAO,CAAE,qBAAqB,GAAG,IAAI,KAAI,CAAC,CAAC,CAAA;AACrE,CAAC;AAJD,oCAIC;AAED,SAAgB,YAAY,CAAC,QAA8B,EAAE,OAAe;;IAC1E,OAAO,CACL,CAAA,MAAA,QAAQ,aAAR,QAAQ,uBAAR,QAAQ,CAAE,SAAS,0CACf,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACV,OAAO,EACL,QAAQ,CAAC,mBAAmB,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC;QACpE,KAAK,EAAE,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC;QACrB,GAAG,EAAE,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC;QACjB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC;KACpB,CAAC,EACD,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,OAAO,CAAC,KAAI,EAAE,CAC5C,CAAA;AACH,CAAC;AAZD,oCAYC"}

package/dist/LinearGenomeView/model.d.ts

@@ -1,6 +1,7 @@
 import React from 'react';
 import { Region } from '@jbrowse/core/util/types';
 import { MenuItem } from '@jbrowse/core/ui';
+import { ParsedLocString } from '@jbrowse/core/util';
 import BaseResult from '@jbrowse/core/TextSearch/BaseResults';
 import { BlockSet, BaseBlock } from '@jbrowse/core/util/blockTypes';
 import { Instance } from 'mobx-state-tree';
@@ -95,9 +96,6 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     coarseTotalBp: number;
     leftOffset: BpOffset | undefined;
     rightOffset: BpOffset | undefined;
-    searchResults: BaseResult[] | undefined;
-    searchQuery: string | undefined;
-    seqDialogDisplayed: boolean;
 } & {
     /**
      * #getter
@@ -136,10 +134,6 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
      * #getter
      */
     readonly hasDisplayedRegions: boolean;
-    /**
-     * #getter
-     */
-    readonly isSearchDialogDisplayed: boolean;
     /**
      * #getter
      */
@@ -256,17 +250,15 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     zoomTo(bpPerPx: number, offset?: number, centerAtOffset?: boolean): number;
     /**
      * #action
-     * sets offsets used in the get sequence dialog
+     * sets offsets of rubberband, used in the get sequence dialog can call
+     * view.getSelectedRegions(view.leftOffset,view.rightOffset) to compute
+     * the selected regions from the offsets
      */
     setOffsets(left?: BpOffset, right?: BpOffset): void;
     /**
      * #action
      */
-    setSearchResults(results?: BaseResult[], query?: string): void;
-    /**
-     * #action
-     */
-    setGetSequenceDialogOpen(open: boolean): void;
+    setSearchResults(searchResults: BaseResult[], searchQuery: string, assemblyName?: string): void;
     /**
      * #action
      */
@@ -315,10 +307,13 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     /**
      * #method
      * Helper method for the fetchSequence.
-     * Retrieves the corresponding regions that were selected by the rubberband
+     * Retrieves the corresponding regions that were selected by the
+     * rubberband
      *
-     * @param leftOffset - `object as {start, end, index, offset}`, offset = start of user drag
-     * @param rightOffset - `object as {start, end, index, offset}`, offset = end of user drag
+     * @param leftOffset - `object as {start, end, index, offset}`, offset = start
+     * of user drag
+     * @param rightOffset - `object as {start, end, index, offset}`,
+     * offset = end of user drag
      * @returns array of Region[]
      */
     getSelectedRegions(leftOffset?: BpOffset, rightOffset?: BpOffset): {
@@ -336,7 +331,8 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     }[];
     /**
      * #action
-     * schedule something to be run after the next time displayedRegions is set
+     * schedule something to be run after the next time displayedRegions is
+     * set
      */
     afterDisplayedRegionsSet(cb: Function): void;
     /**
@@ -392,8 +388,8 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     readonly anyCytobandsExist: boolean;
     /**
      * #getter
-     * the cytoband is displayed to the right of the chromosome name,
-     * and that offset is calculated manually with this method
+     * the cytoband is displayed to the right of the chromosome name, and
+     * that offset is calculated manually with this method
      */
     readonly cytobandOffset: number;
 } & {
@@ -406,19 +402,18 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     /**
      * #getter
      * static blocks are an important concept jbrowse uses to avoid
-     * re-rendering when you scroll to the side. when you horizontally
-     * scroll to the right, old blocks to the left may be removed, and
-     * new blocks may be instantiated on the right. tracks may use the
-     * static blocks to render their data for the region represented by
-     * the block
+     * re-rendering when you scroll to the side. when you horizontally scroll to the
+     * right, old blocks to the left may be removed, and new blocks may be
+     * instantiated on the right. tracks may use the static blocks to render their
+     * data for the region represented by the block
      */
     readonly staticBlocks: BlockSet;
     /**
      * #getter
      * dynamic blocks represent the exact coordinates of the currently
-     * visible genome regions on the screen. they are similar to static
-     * blocks, but statcic blocks can go offscreen while dynamic blocks
-     * represent exactly what is on screen
+     * visible genome regions on the screen. they are similar to static blocks, but
+     * static blocks can go offscreen while dynamic blocks represent exactly what
+     * is on screen
      */
     readonly dynamicBlocks: BlockSet;
     /**
@@ -446,8 +441,8 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
 } & {
     /**
      * #action
-     * offset is the base-pair-offset in the displayed region, index is the index of the
-     * displayed region in the linear genome view
+     * offset is the base-pair-offset in the displayed region, index is the
+     * index of the displayed region in the linear genome view
      *
      * @param start - object as `{start, end, offset, index}`
      * @param end - object as `{start, end, offset, index}`
@@ -455,18 +450,27 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     moveTo(start?: BpOffset, end?: BpOffset): void;
     /**
      * #action
-     * navigate to the given locstring
+     * Navigate to the given locstring, will change displayed regions if
+     * needed, and wait for assemblies to be initialized
      *
-     * @param locString - e.g. "chr1:1-100"
-     * @param optAssemblyName - (optional) the assembly name to use when navigating to the locstring
+     * @param input - e.g. "chr1:1-100", "chr1:1-100 chr2:1-100", "chr 1 100"
+     * @param optAssemblyName - (optional) the assembly name to use when
+     * navigating to the locstring
+     */
+    navToLocString(input: string, optAssemblyName?: string): Promise<void>;
+    /**
+     * #action
+     * Similar to `navToLocString`, but accepts parsed location objects
+     * instead of strings. Will try to perform `setDisplayedRegions` if
+     * changing regions
      */
-    navToLocString(locString: string, optAssemblyName?: string): Promise<void>;
+    navToLocations(parsedLocStrings: ParsedLocString[], assemblyName?: string): Promise<void>;
     /**
      * #action
      * Navigate to a location based on its refName and optionally start, end,
-     * and assemblyName. Can handle if there are multiple displayedRegions
-     * from same refName. Only navigates to a location if it is entirely
-     * within a displayedRegion. Navigates to the first matching location
+     * and assemblyName. Will not try to change displayed regions, use
+     * `navToLocations` instead. Only navigates to a location if it is
+     * entirely within a displayedRegion. Navigates to the first matching location
      * encountered.
      *
      * Throws an error if navigation was unsuccessful
@@ -476,6 +480,15 @@ export declare function stateModelFactory(pluginManager: PluginManager): import(
     navTo(query: NavLocation): void;
     /**
      * #action
+     * Navigate to a location based on its refName and optionally start, end,
+     * and assemblyName. Will not try to change displayed regions, use
+     * navToLocations instead. Only navigates to a location if it is entirely
+     * within a displayedRegion. Navigates to the first matching location
+     * encountered.
+     *
+     * Throws an error if navigation was unsuccessful
+     *
+     * @param locations - proposed location to navigate to
      */
     navToMultiple(locations: NavLocation[]): void;
 } & {

package/dist/LinearGenomeView/model.js

@@ -56,8 +56,11 @@ const SVGLinearGenomeView_1 = require("./svgcomponents/SVGLinearGenomeView");
 const ExportSvgDialog_1 = __importDefault(require("./components/ExportSvgDialog"));
 const MiniControls_1 = __importDefault(require("./components/MiniControls"));
 const Header_1 = __importDefault(require("./components/Header"));
+const util_2 = require("./util");
 // lazies
 const SequenceSearchDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SequenceSearchDialog'))));
+const GetSequenceDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/GetSequenceDialog'))));
+const SearchResultsDialog = (0, react_1.lazy)(() => Promise.resolve().then(() => __importStar(require('./components/SearchResultsDialog'))));
 function calculateVisibleLocStrings(contentBlocks) {
     if (!contentBlocks.length) {
         return '';
@@ -137,7 +140,8 @@ function stateModelFactory(pluginManager) {
         trackSelectorType: mobx_state_tree_1.types.optional(mobx_state_tree_1.types.enumeration(['hierarchical']), 'hierarchical'),
         /**
          * #property
-         * how to display the track labels, can be "overlapping", "offset", or "hidden"
+         * how to display the track labels, can be "overlapping", "offset", or
+         * "hidden"
          */
         trackLabels: mobx_state_tree_1.types.optional(mobx_state_tree_1.types.string, () => (0, util_1.localStorageGetItem)('lgv-trackLabels') || 'overlapping'),
         /**
@@ -171,9 +175,6 @@ function stateModelFactory(pluginManager) {
         coarseTotalBp: 0,
         leftOffset: undefined,
         rightOffset: undefined,
-        searchResults: undefined,
-        searchQuery: undefined,
-        seqDialogDisplayed: false,
     }))
         .views(self => ({
         /**
@@ -246,12 +247,6 @@ function stateModelFactory(pluginManager) {
         get hasDisplayedRegions() {
             return self.displayedRegions.length > 0;
         },
-        /**
-         * #getter
-         */
-        get isSearchDialogDisplayed() {
-            return self.searchResults !== undefined;
-        },
         /**
          * #getter
          */
@@ -486,7 +481,9 @@ function stateModelFactory(pluginManager) {
         },
         /**
          * #action
-         * sets offsets used in the get sequence dialog
+         * sets offsets of rubberband, used in the get sequence dialog can call
+         * view.getSelectedRegions(view.leftOffset,view.rightOffset) to compute
+         * the selected regions from the offsets
          */
         setOffsets(left, right) {
             self.leftOffset = left;
@@ -495,15 +492,18 @@ function stateModelFactory(pluginManager) {
         /**
          * #action
          */
-        setSearchResults(results, query) {
-            self.searchResults = results;
-            self.searchQuery = query;
-        },
-        /**
-         * #action
-         */
-        setGetSequenceDialogOpen(open) {
-            self.seqDialogDisplayed = open;
+        setSearchResults(searchResults, searchQuery, assemblyName) {
+            (0, util_1.getSession)(self).queueDialog(handleClose => [
+                SearchResultsDialog,
+                {
+                    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                    model: self,
+                    searchResults,
+                    searchQuery,
+                    handleClose,
+                    assemblyName,
+                },
+            ]);
         },
         /**
          * #action
@@ -650,10 +650,13 @@ function stateModelFactory(pluginManager) {
         /**
          * #method
          * Helper method for the fetchSequence.
-         * Retrieves the corresponding regions that were selected by the rubberband
+         * Retrieves the corresponding regions that were selected by the
+         * rubberband
          *
-         * @param leftOffset - `object as {start, end, index, offset}`, offset = start of user drag
-         * @param rightOffset - `object as {start, end, index, offset}`, offset = end of user drag
+         * @param leftOffset - `object as {start, end, index, offset}`, offset = start
+         * of user drag
+         * @param rightOffset - `object as {start, end, index, offset}`,
+         * offset = end of user drag
          * @returns array of Region[]
          */
         getSelectedRegions(leftOffset, rightOffset) {
@@ -672,7 +675,8 @@ function stateModelFactory(pluginManager) {
         },
         /**
          * #action
-         * schedule something to be run after the next time displayedRegions is set
+         * schedule something to be run after the next time displayedRegions is
+         * set
          */
         afterDisplayedRegionsSet(cb) {
             self.afterDisplayedRegionsSetCallbacks.push(cb);
@@ -824,8 +828,8 @@ function stateModelFactory(pluginManager) {
         },
         /**
          * #getter
-         * the cytoband is displayed to the right of the chromosome name,
-         * and that offset is calculated manually with this method
+         * the cytoband is displayed to the right of the chromosome name, and
+         * that offset is calculated manually with this method
          */
         get cytobandOffset() {
             return this.showCytobands
@@ -981,11 +985,10 @@ function stateModelFactory(pluginManager) {
             /**
              * #getter
              * static blocks are an important concept jbrowse uses to avoid
-             * re-rendering when you scroll to the side. when you horizontally
-             * scroll to the right, old blocks to the left may be removed, and
-             * new blocks may be instantiated on the right. tracks may use the
-             * static blocks to render their data for the region represented by
-             * the block
+             * re-rendering when you scroll to the side. when you horizontally scroll to the
+             * right, old blocks to the left may be removed, and new blocks may be
+             * instantiated on the right. tracks may use the static blocks to render their
+             * data for the region represented by the block
              */
             get staticBlocks() {
                 const ret = (0, calculateStaticBlocks_1.default)(self);
@@ -999,9 +1002,9 @@ function stateModelFactory(pluginManager) {
             /**
              * #getter
              * dynamic blocks represent the exact coordinates of the currently
-             * visible genome regions on the screen. they are similar to static
-             * blocks, but statcic blocks can go offscreen while dynamic blocks
-             * represent exactly what is on screen
+             * visible genome regions on the screen. they are similar to static blocks, but
+             * static blocks can go offscreen while dynamic blocks represent exactly what
+             * is on screen
              */
             get dynamicBlocks() {
                 return (0, calculateDynamicBlocks_1.default)(self);
@@ -1062,8 +1065,8 @@ function stateModelFactory(pluginManager) {
         .actions(self => ({
         /**
          * #action
-         * offset is the base-pair-offset in the displayed region, index is the index of the
-         * displayed region in the linear genome view
+         * offset is the base-pair-offset in the displayed region, index is the
+         * index of the displayed region in the linear genome view
          *
          * @param start - object as `{start, end, offset, index}`
          * @param end - object as `{start, end, offset, index}`
@@ -1073,76 +1076,38 @@ function stateModelFactory(pluginManager) {
         },
         /**
          * #action
-         * navigate to the given locstring
+         * Navigate to the given locstring, will change displayed regions if
+         * needed, and wait for assemblies to be initialized
          *
-         * @param locString - e.g. "chr1:1-100"
-         * @param optAssemblyName - (optional) the assembly name to use when navigating to the locstring
+         * @param input - e.g. "chr1:1-100", "chr1:1-100 chr2:1-100", "chr 1 100"
+         * @param optAssemblyName - (optional) the assembly name to use when
+         * navigating to the locstring
          */
-        async navToLocString(locString, optAssemblyName) {
+        async navToLocString(input, optAssemblyName) {
             const { assemblyNames } = self;
             const { assemblyManager } = (0, util_1.getSession)(self);
             const { isValidRefName } = assemblyManager;
-            await (0, mobx_1.when)(() => self.volatileWidth !== undefined);
             const assemblyName = optAssemblyName || assemblyNames[0];
-            let parsedLocStrings;
-            const inputs = locString
-                .split(/(\s+)/)
-                .map(f => f.trim())
-                .filter(f => !!f);
             if (assemblyName) {
+                // wait before isValidRefName can be called
                 await assemblyManager.waitForAssembly(assemblyName);
             }
-            // first try interpreting as a whitespace-separated sequence of
-            // multiple locstrings
-            try {
-                parsedLocStrings = inputs.map(loc => (0, util_1.parseLocString)(loc, ref => isValidRefName(ref, assemblyName)));
-            }
-            catch (e) {
-                // if this fails, try interpreting as a whitespace-separated refname,
-                // start, end if start and end are integer inputs
-                const [refName, start, end] = inputs;
-                if (`${e}`.match(/Unknown reference sequence/) &&
-                    Number.isInteger(+start) &&
-                    Number.isInteger(+end)) {
-                    parsedLocStrings = [
-                        (0, util_1.parseLocString)(refName + ':' + start + '..' + end, ref => isValidRefName(ref, assemblyName)),
-                    ];
-                }
-                else {
-                    throw e;
-                }
-            }
-            const locations = await Promise.all(parsedLocStrings === null || parsedLocStrings === void 0 ? void 0 : parsedLocStrings.map(async (region) => {
-                const asmName = region.assemblyName || assemblyName;
-                const asm = await assemblyManager.waitForAssembly(asmName);
-                const { refName } = region;
-                if (!asm) {
-                    throw new Error(`assembly ${asmName} not found`);
-                }
-                const { regions } = asm;
-                if (!regions) {
-                    throw new Error(`regions not loaded yet for ${asmName}`);
-                }
-                const canonicalRefName = asm.getCanonicalRefName(region.refName);
-                if (!canonicalRefName) {
-                    throw new Error(`Could not find refName ${refName} in ${asm.name}`);
-                }
-                const parentRegion = regions.find(r => r.refName === canonicalRefName);
-                if (!parentRegion) {
-                    throw new Error(`Could not find refName ${refName} in ${asmName}`);
-                }
-                return {
-                    ...region,
-                    assemblyName: asmName,
-                    parentRegion,
-                };
-            }));
+            return this.navToLocations((0, util_2.parseLocStrings)(input, assemblyName, isValidRefName), assemblyName);
+        },
+        /**
+         * #action
+         * Similar to `navToLocString`, but accepts parsed location objects
+         * instead of strings. Will try to perform `setDisplayedRegions` if
+         * changing regions
+         */
+        async navToLocations(parsedLocStrings, assemblyName) {
+            const { assemblyManager } = (0, util_1.getSession)(self);
+            await (0, mobx_1.when)(() => self.volatileWidth !== undefined);
+            const locations = await (0, util_2.generateLocations)(parsedLocStrings, assemblyManager, assemblyName);
             if (locations.length === 1) {
                 const loc = locations[0];
-                self.setDisplayedRegions([
-                    { reversed: loc.reversed, ...loc.parentRegion },
-                ]);
-                const { start, end, parentRegion } = loc;
+                const { reversed, parentRegion, start, end } = loc;
+                self.setDisplayedRegions([{ reversed, ...parentRegion }]);
                 this.navTo({
                     ...loc,
                     start: (0, util_1.clamp)(start !== null && start !== void 0 ? start : 0, 0, parentRegion.end),
@@ -1159,9 +1124,9 @@ function stateModelFactory(pluginManager) {
         /**
          * #action
          * Navigate to a location based on its refName and optionally start, end,
-         * and assemblyName. Can handle if there are multiple displayedRegions
-         * from same refName. Only navigates to a location if it is entirely
-         * within a displayedRegion. Navigates to the first matching location
+         * and assemblyName. Will not try to change displayed regions, use
+         * `navToLocations` instead. Only navigates to a location if it is
+         * entirely within a displayedRegion. Navigates to the first matching location
          * encountered.
          *
          * Throws an error if navigation was unsuccessful
@@ -1173,105 +1138,62 @@ function stateModelFactory(pluginManager) {
         },
         /**
          * #action
+         * Navigate to a location based on its refName and optionally start, end,
+         * and assemblyName. Will not try to change displayed regions, use
+         * navToLocations instead. Only navigates to a location if it is entirely
+         * within a displayedRegion. Navigates to the first matching location
+         * encountered.
+         *
+         * Throws an error if navigation was unsuccessful
+         *
+         * @param locations - proposed location to navigate to
          */
         navToMultiple(locations) {
-            const firstLocation = locations[0];
-            let { refName } = firstLocation;
-            const { start, end, assemblyName = self.assemblyNames[0], } = firstLocation;
-            if (start !== undefined && end !== undefined && start > end) {
-                throw new Error(`start "${start + 1}" is greater than end "${end}"`);
+            if (locations.some(l => l.start !== undefined && l.end !== undefined && l.start > l.end)) {
+                throw new Error('found start greater than end');
             }
-            const session = (0, util_1.getSession)(self);
-            const { assemblyManager } = session;
-            const assembly = assemblyManager.get(assemblyName);
-            if (assembly) {
-                const canonicalRefName = assembly.getCanonicalRefName(refName);
-                if (canonicalRefName) {
-                    refName = canonicalRefName;
-                }
+            const f1 = locations[0];
+            const f2 = locations[locations.length - 1];
+            const a = self.assemblyNames[0];
+            const { assemblyManager } = (0, util_1.getSession)(self);
+            const assembly1 = assemblyManager.get(f1.assemblyName || a);
+            const assembly2 = assemblyManager.get(f2.assemblyName || a);
+            const ref1 = (assembly1 === null || assembly1 === void 0 ? void 0 : assembly1.getCanonicalRefName(f1.refName)) || f1.refName;
+            const ref2 = (assembly2 === null || assembly2 === void 0 ? void 0 : assembly2.getCanonicalRefName(f2.refName)) || f2.refName;
+            const r1 = self.displayedRegions.find(r => r.refName === ref1);
+            const r2 = (0, util_1.findLast)(self.displayedRegions, r => r.refName === ref2);
+            if (!r1) {
+                throw new Error(`could not find a region with refName "${ref1}"`);
             }
-            let s = start;
-            let e = end;
-            let refNameMatched = false;
-            const predicate = (r) => {
-                if (refName === r.refName) {
-                    refNameMatched = true;
-                    if (s === undefined) {
-                        s = r.start;
-                    }
-                    if (e === undefined) {
-                        e = r.end;
-                    }
-                    if (s >= r.start && s <= r.end && e <= r.end && e >= r.start) {
-                        return true;
-                    }
-                    s = start;
-                    e = end;
-                }
-                return false;
-            };
-            const lastIndex = (0, util_1.findLastIndex)(self.displayedRegions, predicate);
-            let index;
-            while (index !== lastIndex) {
-                try {
-                    const previousIndex = index;
-                    index = self.displayedRegions
-                        .slice(previousIndex === undefined ? 0 : previousIndex + 1)
-                        .findIndex(predicate);
-                    if (previousIndex !== undefined) {
-                        index += previousIndex + 1;
-                    }
-                    if (!refNameMatched) {
-                        throw new Error(`could not find a region with refName "${refName}"`);
-                    }
-                    if (s === undefined) {
-                        throw new Error(`could not find a region with refName "${refName}" that contained an end position ${e}`);
-                    }
-                    if (e === undefined) {
-                        throw new Error(`could not find a region with refName "${refName}" that contained a start position ${s + 1}`);
-                    }
-                    if (index === -1) {
-                        throw new Error(`could not find a region that completely contained "${(0, util_1.assembleLocString)(firstLocation)}"`);
-                    }
-                    if (locations.length === 1) {
-                        const f = self.displayedRegions[index];
-                        this.moveTo({ index, offset: f.reversed ? f.end - e : s - f.start }, { index, offset: f.reversed ? f.end - s : e - f.start });
-                        return;
-                    }
-                    let idx = 0;
-                    let start = 0;
-                    let end = 0;
-                    for (idx; idx < locations.length; idx++) {
-                        const location = locations[idx];
-                        const region = self.displayedRegions[index + idx];
-                        start = location.start || region.start;
-                        end = location.end || region.end;
-                        if (location.refName !== region.refName) {
-                            throw new Error(`Entered location ${(0, util_1.assembleLocString)(location)} does not match with displayed regions`);
-                        }
-                    }
-                    idx -= 1;
-                    const startDisplayedRegion = self.displayedRegions[index];
-                    const endDisplayedRegion = self.displayedRegions[index + idx];
-                    this.moveTo({
-                        index,
-                        offset: startDisplayedRegion.reversed
-                            ? startDisplayedRegion.end - e
-                            : s - startDisplayedRegion.start,
-                    }, {
-                        index: index + idx,
-                        offset: endDisplayedRegion.reversed
-                            ? endDisplayedRegion.end - start
-                            : end - endDisplayedRegion.start,
-                    });
-                    return;
-                }
-                catch (error) {
-                    if (index === lastIndex) {
-                        throw error;
-                    }
-                }
+            if (!r2) {
+                throw new Error(`could not find a region with refName "${ref2}"`);
+            }
+            const s1 = f1.start === undefined ? r1.start : f1.start;
+            const e1 = f1.end === undefined ? r1.end : f1.end;
+            const s2 = f2.start === undefined ? r2.start : f2.start;
+            const e2 = f2.end === undefined ? r2.end : f2.end;
+            const index = self.displayedRegions.findIndex(r => ref1 === r.refName &&
+                s1 >= r.start &&
+                s1 <= r.end &&
+                e1 <= r.end &&
+                e1 >= r.start);
+            const index2 = self.displayedRegions.findIndex(r => ref2 === r.refName &&
+                s2 >= r.start &&
+                s2 <= r.end &&
+                e2 <= r.end &&
+                e2 >= r.start);
+            if (index === -1 || index2 === -1) {
+                throw new Error(`could not find a region that contained "${locations.map(l => (0, util_1.assembleLocString)(l))}"`);
             }
+            const sd = self.displayedRegions[index];
+            const ed = self.displayedRegions[index2];
+            this.moveTo({
+                index,
+                offset: sd.reversed ? sd.end - e1 : s1 - sd.start,
+            }, {
+                index: index2,
+                offset: ed.reversed ? ed.end - s2 : e2 - ed.start,
+            });
         },
     }))
         .views(self => ({
@@ -1283,15 +1205,16 @@ function stateModelFactory(pluginManager) {
                 {
                     label: 'Zoom to region',
                     icon: ZoomIn_1.default,
-                    onClick: () => {
-                        const { leftOffset, rightOffset } = self;
-                        self.moveTo(leftOffset, rightOffset);
-                    },
+                    onClick: () => self.moveTo(self.leftOffset, self.rightOffset),
                 },
                 {
                     label: 'Get sequence',
                     icon: MenuOpen_1.default,
-                    onClick: () => self.setGetSequenceDialogOpen(true),
+                    onClick: () => (0, util_1.getSession)(self).queueDialog(handleClose => [
+                        GetSequenceDialog,
+                        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                        { model: self, handleClose },
+                    ]),
                 },
             ];
         },
@@ -1315,7 +1238,7 @@ function stateModelFactory(pluginManager) {
                 coord,
                 regionNumber,
             });
-            if (centerPx) {
+            if (centerPx !== undefined) {
                 self.scrollTo(Math.round(centerPx.offsetPx - self.width / 2));
             }
         },