@jbrowse/plugin-grid-bookmark 2.17.0 → 2.18.0

package/dist/GridBookmarkWidget/index.js

@@ -30,9 +30,8 @@ exports.default = GridBookmarkWidgetF;
 const react_1 = require("react");
 const configuration_1 = require("@jbrowse/core/configuration");
 const pluggableElementTypes_1 = require("@jbrowse/core/pluggableElementTypes");
-// locals
-const model_1 = __importDefault(require("./model"));
 const Highlight_1 = __importDefault(require("./components/Highlight"));
+const model_1 = __importDefault(require("./model"));
 const configSchema = (0, configuration_1.ConfigurationSchema)('GridBookmarkWidget', {});
 function GridBookmarkWidgetF(pluginManager) {
     pluginManager.addWidgetType(() => {

package/dist/GridBookmarkWidget/model.d.ts

@@ -1,7 +1,7 @@
-import { Instance, SnapshotIn, IMSTArray } from 'mobx-state-tree';
-import PluginManager from '@jbrowse/core/PluginManager';
-import { Region } from '@jbrowse/core/util/types';
-import { LinearGenomeViewModel } from '@jbrowse/plugin-linear-genome-view';
+import type PluginManager from '@jbrowse/core/PluginManager';
+import type { Region } from '@jbrowse/core/util/types';
+import type { LinearGenomeViewModel } from '@jbrowse/plugin-linear-genome-view';
+import type { IMSTArray, Instance, SnapshotIn } from 'mobx-state-tree';
 declare const LabeledRegionModel: import("mobx-state-tree").IModelType<{
     refName: import("mobx-state-tree").ISimpleType<string>;
     start: import("mobx-state-tree").ISimpleType<number>;
@@ -19,10 +19,10 @@ declare const LabeledRegionModel: import("mobx-state-tree").IModelType<{
     setHighlight(color: string): void;
 }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>;
 export interface IExtendedLGV extends LinearGenomeViewModel {
-    showBookmarkHighlights: boolean;
-    showBookmarkLabels: boolean;
-    toggleShowBookmarkHighlights: (arg: boolean) => void;
-    toggleShowBookmarkLabels: (arg: boolean) => void;
+    bookmarkHighlightsVisible: boolean;
+    bookmarkLabelsVisible: boolean;
+    setBookmarkHighlightsVisible: (arg: boolean) => void;
+    setBookmarkLabelsVisible: (arg: boolean) => void;
 }
 export interface ILabeledRegionModel extends SnapshotIn<typeof LabeledRegionModel> {
     refName: string;
@@ -40,22 +40,9 @@ export interface IExtendedLabeledRegionModel extends ILabeledRegionModel {
     id: number;
     correspondingObj: ILabeledRegionModel;
 }
-/**
- * #stateModel GridBookmarkWidgetModel
- */
 export default function f(_pluginManager: PluginManager): import("mobx-state-tree").IModelType<{
-    /**
-     * #property
-     */
     id: import("mobx-state-tree").IOptionalIType<import("mobx-state-tree").ISimpleType<string>, [undefined]>;
-    /**
-     * #property
-     */
     type: import("mobx-state-tree").ISimpleType<"GridBookmarkWidget">;
-    /**
-     * #property
-     * removed by postProcessSnapshot, only loaded from localStorage
-     */
     bookmarks: import("mobx-state-tree").IOptionalIType<import("mobx-state-tree").IArrayType<import("mobx-state-tree").IModelType<{
         refName: import("mobx-state-tree").ISimpleType<string>;
         start: import("mobx-state-tree").ISimpleType<number>;
@@ -76,26 +63,11 @@ export default function f(_pluginManager: PluginManager): import("mobx-state-tre
     selectedBookmarks: IExtendedLabeledRegionModel[];
     selectedAssembliesPre: string[] | undefined;
 } & {
-    /**
-     * #getter
-     */
     readonly bookmarkAssemblies: string[];
-    /**
-     * #getter
-     */
     readonly validAssemblies: Set<string>;
-    /**
-     * #getter
-     */
     readonly areBookmarksHighlightedOnAllOpenViews: boolean;
-    /**
-     * #getter
-     */
     readonly areBookmarksHighlightLabelsOnAllOpenViews: boolean;
 } & {
-    /**
-     * #getter
-     */
     readonly bookmarksWithValidAssemblies: ({
         refName: string;
         start: number;
@@ -126,9 +98,6 @@ export default function f(_pluginManager: PluginManager): import("mobx-state-tre
         setHighlight(color: string): void;
     }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>)[];
 } & {
-    /**
-     * #getter
-     */
     readonly sharedBookmarksModel: {
         sharedBookmarks: (IMSTArray<import("mobx-state-tree").IModelType<{
             refName: import("mobx-state-tree").ISimpleType<string>;
@@ -179,9 +148,6 @@ export default function f(_pluginManager: PluginManager): import("mobx-state-tre
             setHighlight(color: string): void;
         }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>>;
     }, {}, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
-    /**
-     * #getter
-     */
     readonly allBookmarksModel: {
         sharedBookmarks: (IMSTArray<import("mobx-state-tree").IModelType<{
             refName: import("mobx-state-tree").ISimpleType<string>;
@@ -233,64 +199,22 @@ export default function f(_pluginManager: PluginManager): import("mobx-state-tre
         }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>>;
     }, {}, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
 } & {
-    /**
-     * #action
-     */
     setSelectedAssemblies(assemblies?: string[]): void;
 } & {
-    /**
-     * #getter
-     */
     readonly selectedAssemblies: string[];
 } & {
-    /**
-     * #action
-     */
     importBookmarks(regions: Region[]): void;
-    /**
-     * #action
-     */
     addBookmark(region: Region): void;
-    /**
-     * #action
-     */
     removeBookmark(index: number): void;
-    /**
-     * #action
-     */
     updateBookmarkLabel(bookmark: IExtendedLabeledRegionModel, label: string): void;
-    /**
-     * #action
-     */
     updateBookmarkHighlight(bookmark: IExtendedLabeledRegionModel, color: string): void;
-    /**
-     * #action
-     */
     updateBulkBookmarkHighlights(color: string): void;
-    /**
-     * #action
-     */
     setSelectedBookmarks(bookmarks: IExtendedLabeledRegionModel[]): void;
-    /**
-     * #action
-     */
     setBookmarkedRegions(regions: IMSTArray<typeof LabeledRegionModel>): void;
-    /**
-     * #action
-     */
-    setHighlightToggle(toggle: boolean): void;
-    /**
-     * #action
-     */
-    setLabelToggle(toggle: boolean): void;
+    setBookmarkHighlightsVisible(arg: boolean): void;
+    setBookmarkLabelsVisible(arg: boolean): void;
 } & {
-    /**
-     * #action
-     */
     clearAllBookmarks(): void;
-    /**
-     * #action
-     */
     clearSelectedBookmarks(): void;
     removeBookmarkObject(arg: Instance<typeof LabeledRegionModel>): void;
 } & {

package/dist/GridBookmarkWidget/model.js

@@ -1,10 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = f;
-const mobx_state_tree_1 = require("mobx-state-tree");
-const mst_1 = require("@jbrowse/core/util/types/mst");
 const util_1 = require("@jbrowse/core/util");
+const mst_1 = require("@jbrowse/core/util/types/mst");
 const mobx_1 = require("mobx");
+const mobx_state_tree_1 = require("mobx-state-tree");
 const LabeledRegionModel = mobx_state_tree_1.types
     .compose(mst_1.Region, mobx_state_tree_1.types.model('Label', {
     label: mobx_state_tree_1.types.optional(mobx_state_tree_1.types.string, ''),
@@ -24,24 +24,11 @@ const SharedBookmarksModel = mobx_state_tree_1.types.model('SharedBookmarksModel
 const localStorageKeyF = () => typeof window !== 'undefined'
     ? `bookmarks-${[window.location.host + window.location.pathname].join('-')}`
     : 'empty';
-/**
- * #stateModel GridBookmarkWidgetModel
- */
 function f(_pluginManager) {
     return mobx_state_tree_1.types
         .model('GridBookmarkModel', {
-        /**
-         * #property
-         */
         id: mst_1.ElementId,
-        /**
-         * #property
-         */
         type: mobx_state_tree_1.types.literal('GridBookmarkWidget'),
-        /**
-         * #property
-         * removed by postProcessSnapshot, only loaded from localStorage
-         */
         bookmarks: mobx_state_tree_1.types.optional(mobx_state_tree_1.types.array(LabeledRegionModel), () => JSON.parse((0, util_1.localStorageGetItem)(localStorageKeyF()) || '[]')),
     })
         .volatile(() => ({
@@ -49,167 +36,104 @@ function f(_pluginManager) {
         selectedAssembliesPre: undefined,
     }))
         .views(self => ({
-        /**
-         * #getter
-         */
         get bookmarkAssemblies() {
             return [...new Set(self.bookmarks.map(r => r.assemblyName))];
         },
-        /**
-         * #getter
-         */
         get validAssemblies() {
             const { assemblyManager } = (0, util_1.getSession)(self);
             return new Set(this.bookmarkAssemblies.filter(a => assemblyManager.get(a)));
         },
-        /**
-         * #getter
-         */
         get areBookmarksHighlightedOnAllOpenViews() {
             const { views } = (0, util_1.getSession)(self);
-            return views.every(v => 'showBookmarkHighlights' in v ? v.showBookmarkHighlights : true);
+            return views.every(v => 'bookmarkHighlightsVisible' in v ? v.bookmarkHighlightsVisible : true);
         },
-        /**
-         * #getter
-         */
         get areBookmarksHighlightLabelsOnAllOpenViews() {
             const { views } = (0, util_1.getSession)(self);
-            return views.every(v => 'showBookmarkLabels' in v ? v.showBookmarkLabels : true);
+            return views.every(v => 'bookmarkLabelsVisible' in v ? v.bookmarkLabelsVisible : true);
         },
     }))
         .views(self => ({
-        /**
-         * #getter
-         */
         get bookmarksWithValidAssemblies() {
             return self.bookmarks.filter(e => self.validAssemblies.has(e.assemblyName));
         },
     }))
         .views(self => ({
-        /**
-         * #getter
-         */
         get sharedBookmarksModel() {
-            // requires cloning bookmarks with JSON.stringify/parse to avoid duplicate
-            // reference to same object in the same state tree, will otherwise error
-            // when performing share
             return SharedBookmarksModel.create({
                 sharedBookmarks: JSON.parse(JSON.stringify(self.selectedBookmarks)),
             });
         },
-        /**
-         * #getter
-         */
         get allBookmarksModel() {
-            // requires cloning bookmarks with JSON.stringify/parse to avoid duplicate
-            // reference to same object in the same state tree, will otherwise error
-            // when performing share
             return SharedBookmarksModel.create({
                 sharedBookmarks: JSON.parse(JSON.stringify(self.bookmarksWithValidAssemblies)),
             });
         },
     }))
         .actions(self => ({
-        /**
-         * #action
-         */
         setSelectedAssemblies(assemblies) {
             self.selectedAssembliesPre = assemblies;
         },
     }))
         .views(self => ({
-        /**
-         * #getter
-         */
         get selectedAssemblies() {
             var _a, _b;
             return ((_b = (_a = self.selectedAssembliesPre) === null || _a === void 0 ? void 0 : _a.filter(f => self.validAssemblies.has(f))) !== null && _b !== void 0 ? _b : [...self.validAssemblies]);
         },
     }))
         .actions(self => ({
-        /**
-         * #action
-         */
         importBookmarks(regions) {
             self.bookmarks = (0, mobx_state_tree_1.cast)([...self.bookmarks, ...regions]);
         },
-        /**
-         * #action
-         */
         addBookmark(region) {
             self.bookmarks.push(region);
         },
-        /**
-         * #action
-         */
         removeBookmark(index) {
             self.bookmarks.splice(index, 1);
         },
-        /**
-         * #action
-         */
         updateBookmarkLabel(bookmark, label) {
             bookmark.correspondingObj.setLabel(label);
         },
-        /**
-         * #action
-         */
         updateBookmarkHighlight(bookmark, color) {
             bookmark.correspondingObj.setHighlight(color);
         },
-        /**
-         * #action
-         */
         updateBulkBookmarkHighlights(color) {
             self.selectedBookmarks.forEach(bookmark => {
                 this.updateBookmarkHighlight(bookmark, color);
             });
         },
-        /**
-         * #action
-         */
         setSelectedBookmarks(bookmarks) {
             self.selectedBookmarks = bookmarks;
         },
-        /**
-         * #action
-         */
         setBookmarkedRegions(regions) {
             self.bookmarks = (0, mobx_state_tree_1.cast)(regions);
         },
-        /**
-         * #action
-         */
-        setHighlightToggle(toggle) {
+        setBookmarkHighlightsVisible(arg) {
             const { views } = (0, util_1.getSession)(self);
             views.forEach(view => {
-                var _a;
-                // @ts-expect-error
-                (_a = view.toggleShowBookmarkHighlights) === null || _a === void 0 ? void 0 : _a.call(view, toggle);
+                var _a, _b;
+                (_a = view.setBookmarkHighlightsVisible) === null || _a === void 0 ? void 0 : _a.call(view, arg);
+                (_b = view.views) === null || _b === void 0 ? void 0 : _b.map(view => {
+                    var _a;
+                    (_a = view.setBookmarkHighlightsVisible) === null || _a === void 0 ? void 0 : _a.call(view, arg);
+                });
             });
         },
-        /**
-         * #action
-         */
-        setLabelToggle(toggle) {
+        setBookmarkLabelsVisible(arg) {
             const { views } = (0, util_1.getSession)(self);
             views.forEach(view => {
-                var _a;
-                // @ts-expect-error
-                (_a = view.toggleShowBookmarkLabels) === null || _a === void 0 ? void 0 : _a.call(view, toggle);
+                var _a, _b;
+                (_a = view.setBookmarkLabelsVisible) === null || _a === void 0 ? void 0 : _a.call(view, arg);
+                (_b = view.views) === null || _b === void 0 ? void 0 : _b.map(view => {
+                    var _a;
+                    (_a = view.setBookmarkHighlightsVisible) === null || _a === void 0 ? void 0 : _a.call(view, arg);
+                });
             });
         },
     }))
         .actions(self => ({
-        /**
-         * #action
-         */
         clearAllBookmarks() {
             self.setBookmarkedRegions(self.bookmarks.filter(bookmark => !self.validAssemblies.has(bookmark.assemblyName)));
         },
-        /**
-         * #action
-         */
         clearSelectedBookmarks() {
             for (const bookmark of self.selectedBookmarks) {
                 self.bookmarks.remove(bookmark.correspondingObj);

package/dist/GridBookmarkWidget/sessionSharing.d.ts

@@ -3,4 +3,4 @@ export declare function shareSessionToDynamo(session: unknown, url: string, refe
     encryptedSession: string;
     password: string;
 }>;
-export declare function readSessionFromDynamo(baseUrl: string, sessionQueryParam: string, password: string, signal?: AbortSignal): Promise<string>;
+export declare function readSessionFromDynamo(baseUrl: string, sessionQueryParam: string, password: string): Promise<string>;

package/dist/GridBookmarkWidget/sessionSharing.js

@@ -25,9 +25,7 @@ var __importStar = (this && this.__importStar) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.shareSessionToDynamo = shareSessionToDynamo;
 exports.readSessionFromDynamo = readSessionFromDynamo;
-// duplicated from products/jbrowse-web/src/sessionSharing.ts ; could possibly be moved into a higher directory and shared between the two
 const utils_1 = require("./utils");
-// from https://stackoverflow.com/questions/1349404/
 function generateUID(length) {
     return window
         .btoa([...window.crypto.getRandomValues(new Uint8Array(length * 2))]
@@ -55,7 +53,6 @@ function getErrorMsg(err) {
         return err;
     }
 }
-// writes the encrypted session, current datetime, and referer to DynamoDB
 async function shareSessionToDynamo(session, url, referer) {
     const sess = await (0, utils_1.toUrlSafeB64)(JSON.stringify(session));
     const password = generateUID(5);
@@ -80,15 +77,12 @@ async function shareSessionToDynamo(session, url, referer) {
         password,
     };
 }
-async function readSessionFromDynamo(baseUrl, sessionQueryParam, password, signal) {
+async function readSessionFromDynamo(baseUrl, sessionQueryParam, password) {
     const sessionId = sessionQueryParam.split('share-')[1];
     const url = `${baseUrl}?sessionId=${encodeURIComponent(sessionId)}`;
-    const response = await fetch(url, {
-        signal,
-    });
+    const response = await fetch(url);
     if (!response.ok) {
-        const err = await response.text();
-        throw new Error(getErrorMsg(err));
+        throw new Error(getErrorMsg(await response.text()));
     }
     const json = await response.json();
     return decrypt(json.session, password);

package/dist/GridBookmarkWidget/types.d.ts

@@ -1,4 +1,4 @@
-import { Region } from '@jbrowse/core/util/types';
+import type { Region } from '@jbrowse/core/util/types';
 export interface LabeledRegion extends Region {
     label: string;
 }

package/dist/GridBookmarkWidget/utils.d.ts

@@ -1,21 +1,7 @@
-import { AbstractViewModel } from '@jbrowse/core/util/types';
-import { GridBookmarkModel } from './model';
+import type { GridBookmarkModel } from './model';
+import type { AbstractViewModel } from '@jbrowse/core/util/types';
 export declare function navToBookmark(locString: string, assembly: string, views: AbstractViewModel[], model: GridBookmarkModel): Promise<void>;
 export declare function downloadBookmarkFile(fileFormat: string, model: GridBookmarkModel): void;
-/**
- * Pad the end of a base64 string with "=" to make it valid
- * @param b64 - unpadded b64 string
- */
 export declare function b64PadSuffix(b64: string): string;
-/**
- * Decode and inflate a url-safe base64 to a string
- * See {@link https://en.wikipedia.org/wiki/Base64#URL_applications}
- * @param b64 - a base64 string to decode and inflate
- */
 export declare function fromUrlSafeB64(b64: string): Promise<string>;
-/**
- * Compress and encode a string as url-safe base64
- * See {@link https://en.wikipedia.org/wiki/Base64#URL_applications}
- * @param str-  a string to compress and encode
- */
 export declare function toUrlSafeB64(str: string): Promise<string>;

package/dist/GridBookmarkWidget/utils.js

@@ -28,21 +28,15 @@ exports.downloadBookmarkFile = downloadBookmarkFile;
 exports.b64PadSuffix = b64PadSuffix;
 exports.fromUrlSafeB64 = fromUrlSafeB64;
 exports.toUrlSafeB64 = toUrlSafeB64;
-const file_saver_1 = require("file-saver");
 const util_1 = require("@jbrowse/core/util");
+const file_saver_1 = require("file-saver");
 async function navToBookmark(locString, assembly, views, model) {
     const session = (0, util_1.getSession)(model);
     try {
-        // get the focused view
         let view = views.find(view => view.id === session.focusedViewId);
-        // check if the focused view is the appropriate assembly, if not proceed
         if ((view === null || view === void 0 ? void 0 : view.assemblyNames[0]) !== assembly) {
-            view = views.find(elt => 
-            // @ts-expect-error
-            elt.type === 'LinearGenomeView' && elt.assemblyNames[0] === assembly);
+            view = views.find(elt => elt.type === 'LinearGenomeView' && elt.assemblyNames[0] === assembly);
         }
-        // if no view is opened of the selectedAssembly, open a new
-        // view with that assembly
         if (!view) {
             const newViewId = `${model.id}_${assembly}`;
             view = session.addView('LinearGenomeView', {
@@ -85,7 +79,6 @@ function downloadBookmarkFile(fileFormat, model) {
         }
     }
     else {
-        // TSV
         const fileHeader = 'chrom\tstart\tend\tlabel\tassembly_name\tcoord_range\n';
         const fileContents = bookmarksToDownload
             .map(bookmark => {
@@ -102,10 +95,6 @@ function downloadBookmarkFile(fileFormat, model) {
         (0, file_saver_1.saveAs)(blob, fileName);
     }
 }
-/**
- * Pad the end of a base64 string with "=" to make it valid
- * @param b64 - unpadded b64 string
- */
 function b64PadSuffix(b64) {
     let num = 0;
     const mo = b64.length % 4;
@@ -124,11 +113,6 @@ function b64PadSuffix(b64) {
     }
     return b64 + '='.repeat(num);
 }
-/**
- * Decode and inflate a url-safe base64 to a string
- * See {@link https://en.wikipedia.org/wiki/Base64#URL_applications}
- * @param b64 - a base64 string to decode and inflate
- */
 async function fromUrlSafeB64(b64) {
     const originalB64 = b64PadSuffix(b64.replaceAll('-', '+').replaceAll('_', '/'));
     const { toByteArray } = await Promise.resolve().then(() => __importStar(require('base64-js')));
@@ -137,11 +121,6 @@ async function fromUrlSafeB64(b64) {
     const inflated = inflate(bytes);
     return new TextDecoder().decode(inflated);
 }
-/**
- * Compress and encode a string as url-safe base64
- * See {@link https://en.wikipedia.org/wiki/Base64#URL_applications}
- * @param str-  a string to compress and encode
- */
 async function toUrlSafeB64(str) {
     const bytes = new TextEncoder().encode(str);
     const { deflate } = await Promise.resolve().then(() => __importStar(require('pako')));

package/dist/index.d.ts

@@ -1,5 +1,5 @@
 import Plugin from '@jbrowse/core/Plugin';
-import PluginManager from '@jbrowse/core/PluginManager';
+import type PluginManager from '@jbrowse/core/PluginManager';
 export default class GridBookmarkPlugin extends Plugin {
     name: string;
     install(pluginManager: PluginManager): void;

package/dist/index.js

@@ -5,7 +5,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const Plugin_1 = __importDefault(require("@jbrowse/core/Plugin"));
 const util_1 = require("@jbrowse/core/util");
-// icons
 const Bookmark_1 = __importDefault(require("@mui/icons-material/Bookmark"));
 const Bookmarks_1 = __importDefault(require("@mui/icons-material/Bookmarks"));
 const Highlight_1 = __importDefault(require("@mui/icons-material/Highlight"));
@@ -24,31 +23,15 @@ class GridBookmarkPlugin extends Plugin_1.default {
                 const lgv = stateModel;
                 const newStateModel = lgv
                     .props({
-                    /**
-                     * #property
-                     * show the bookmark highlights on this track
-                     */
-                    showBookmarkHighlights: true,
-                    /**
-                     * #property
-                     * show the bookmark labels on this track
-                     */
-                    showBookmarkLabels: true,
+                    bookmarkHighlightsVisible: true,
+                    bookmarkLabelsVisible: true,
                 })
                     .actions(self => ({
-                    /**
-                     * #action
-                     */
-                    toggleShowBookmarkHighlights(toggle) {
-                        self.showBookmarkHighlights =
-                            toggle !== undefined ? toggle : !self.showBookmarkHighlights;
+                    setBookmarkHighlightsVisible(arg) {
+                        self.bookmarkHighlightsVisible = arg;
                     },
-                    /**
-                     * #action
-                     */
-                    toggleShowBookmarkLabels(toggle) {
-                        self.showBookmarkLabels =
-                            toggle !== undefined ? toggle : !self.showBookmarkLabels;
+                    setBookmarkLabelsVisible(arg) {
+                        self.bookmarkLabelsVisible = arg;
                     },
                     activateBookmarkWidget() {
                         const session = (0, util_1.getSession)(self);
@@ -115,18 +98,18 @@ class GridBookmarkPlugin extends Plugin_1.default {
                                             label: 'Toggle bookmark highlights',
                                             icon: Highlight_1.default,
                                             type: 'checkbox',
-                                            checked: self.showBookmarkHighlights,
+                                            checked: self.bookmarkHighlightsVisible,
                                             onClick: () => {
-                                                self.toggleShowBookmarkHighlights();
+                                                self.setBookmarkHighlightsVisible(!self.bookmarkHighlightsVisible);
                                             },
                                         },
                                         {
                                             label: 'Toggle bookmark labels',
                                             icon: Label_1.default,
                                             type: 'checkbox',
-                                            checked: self.showBookmarkLabels,
+                                            checked: self.bookmarkLabelsVisible,
                                             onClick: () => {
-                                                self.toggleShowBookmarkLabels();
+                                                self.setBookmarkLabelsVisible(!self.bookmarkLabelsVisible);
                                             },
                                         },
                                     ],
@@ -159,14 +142,12 @@ class GridBookmarkPlugin extends Plugin_1.default {
                     afterCreate() {
                         document.addEventListener('keydown', e => {
                             const activationSequence = (e.ctrlKey || e.metaKey) && e.shiftKey;
-                            // ctrl+shift+d or cmd+shift+d
                             if (activationSequence && e.code === 'KeyD') {
                                 e.preventDefault();
                                 self.activateBookmarkWidget();
                                 self.bookmarkCurrentRegion();
                                 (0, util_1.getSession)(self).notify('Bookmark created.', 'success');
                             }
-                            // ctrl+shift+m or cmd+shift+m
                             if (activationSequence && e.code === 'KeyM') {
                                 e.preventDefault();
                                 self.navigateNewestBookmark();

package/esm/GridBookmarkWidget/components/AssemblySelector.d.ts

@@ -1,5 +1,5 @@
 import React from 'react';
-import { GridBookmarkModel } from '../model';
+import type { GridBookmarkModel } from '../model';
 declare const AssemblySelector: ({ model, }: {
     model: GridBookmarkModel;
 }) => React.JSX.Element;

package/esm/GridBookmarkWidget/components/AssemblySelector.js

@@ -1,6 +1,6 @@
 import React from 'react';
-import { observer } from 'mobx-react';
 import { Checkbox, FormControl, InputLabel, ListItemText, MenuItem, OutlinedInput, Select, } from '@mui/material';
+import { observer } from 'mobx-react';
 const AssemblySelector = observer(function ({ model, }) {
     const { validAssemblies, selectedAssemblies } = model;
     const noAssemblies = validAssemblies.size === 0;
@@ -14,7 +14,6 @@ const AssemblySelector = observer(function ({ model, }) {
                 model.setSelectedAssemblies([...event.target.value]);
             }, input: React.createElement(OutlinedInput, { label: label }), renderValue: selected => selected.join(', ') },
             React.createElement(MenuItem, { onClickCapture: event => {
-                    // onClickCapture allows us to avoid the parent Select onChange from triggering
                     if (isAllSelected) {
                         model.setSelectedAssemblies([]);
                     }

package/esm/GridBookmarkWidget/components/BookmarkGrid.d.ts

@@ -1,5 +1,5 @@
 import React from 'react';
-import { GridBookmarkModel } from '../model';
+import type { GridBookmarkModel } from '../model';
 declare const BookmarkGrid: ({ model, }: {
     model: GridBookmarkModel;
 }) => React.JSX.Element;