@jbrowse/plugin-dotplot-view 2.10.3 → 2.11.0

package/dist/DotplotReadVsRef/DotplotReadVsRef.js

@@ -95,7 +95,7 @@ function onClick(feature, self) {
     }
     catch (e) {
         console.error(e);
-        session.notify(`${e}`, 'error');
+        session.notifyError(`${e}`, e);
     }
 }
 exports.onClick = onClick;

package/dist/DotplotView/components/DotplotView.js

@@ -178,15 +178,15 @@ const DotplotViewInternal = (0, mobx_react_1.observer)(function ({ model, }) {
     // detect a mouseup after a mousedown was submitted, autoremoves mouseup once
     // that single mouseup is set
     (0, react_1.useEffect)(() => {
-        if (mousedown && !mouseup) {
-            function globalMouseUp(event) {
-                if (Math.abs(xdistance) > 3 && Math.abs(ydistance) > 3 && validSelect) {
-                    setMouseUpClient([event.clientX, event.clientY]);
-                }
-                else {
-                    setMouseDownClient(undefined);
-                }
+        function globalMouseUp(event) {
+            if (Math.abs(xdistance) > 3 && Math.abs(ydistance) > 3 && validSelect) {
+                setMouseUpClient([event.clientX, event.clientY]);
+            }
+            else {
+                setMouseDownClient(undefined);
             }
+        }
+        if (mousedown && !mouseup) {
             window.addEventListener('mouseup', globalMouseUp, true);
             return () => {
                 window.removeEventListener('mouseup', globalMouseUp, true);

package/dist/DotplotView/components/ExportSvgDialog.js

@@ -32,6 +32,10 @@ function LoadingMessage() {
         react_1.default.createElement(material_1.CircularProgress, { size: 20, style: { marginRight: 20 } }),
         react_1.default.createElement(material_1.Typography, { display: "inline" }, "Creating SVG")));
 }
+function TextField2({ children, ...rest }) {
+    return (react_1.default.createElement("div", null,
+        react_1.default.createElement(material_1.TextField, { ...rest }, children)));
+}
 function useSvgLocal(key, val) {
     return (0, util_1.useLocalStorage)('svg-' + key, val);
 }
@@ -46,9 +50,8 @@ function ExportSvgDialog({ model, handleClose, }) {
     return (react_1.default.createElement(ui_1.Dialog, { open: true, onClose: handleClose, title: "Export SVG" },
         react_1.default.createElement(material_1.DialogContent, null,
             error ? (react_1.default.createElement(ui_1.ErrorMessage, { error: error })) : loading ? (react_1.default.createElement(LoadingMessage, null)) : null,
-            react_1.default.createElement(material_1.TextField, { helperText: "filename", value: filename, onChange: event => setFilename(event.target.value) }),
-            react_1.default.createElement("br", null),
-            session.allThemes ? (react_1.default.createElement(material_1.TextField, { select: true, label: "Theme", value: themeName, onChange: event => setThemeName(event.target.value) }, Object.entries(session.allThemes()).map(([key, val]) => (react_1.default.createElement(material_1.MenuItem, { key: key, value: key }, 
+            react_1.default.createElement(TextField2, { helperText: "filename", value: filename, onChange: event => setFilename(event.target.value) }),
+            session.allThemes ? (react_1.default.createElement(TextField2, { select: true, label: "Theme", value: themeName, onChange: event => setThemeName(event.target.value) }, Object.entries(session.allThemes()).map(([key, val]) => (react_1.default.createElement(material_1.MenuItem, { key: key, value: key }, 
             // @ts-expect-error
             val.name || '(Unknown name)'))))) : null,
             offscreenCanvas ? (react_1.default.createElement(material_1.FormControlLabel, { control: react_1.default.createElement(material_1.Checkbox, { checked: rasterizeLayers, onChange: () => setRasterizeLayers(val => !val) }), label: "Rasterize canvas based tracks? File may be much larger if this is turned off" })) : (react_1.default.createElement(material_1.Typography, null, "Note: rasterizing layers not yet supported in this browser, so SVG size may be large"))),
@@ -68,6 +71,8 @@ function ExportSvgDialog({ model, handleClose, }) {
                     catch (e) {
                         console.error(e);
                         setError(e);
+                    }
+                    finally {
                         setLoading(false);
                     }
                 } }, "Submit"))));

package/dist/DotplotView/components/Header.d.ts

@@ -5,6 +5,6 @@ declare const DotplotHeader: ({ model, selection, }: {
     selection?: {
         width: number;
         height: number;
-    } | undefined;
+    };
 }) => React.JSX.Element;
 export default DotplotHeader;

package/dist/DotplotView/model.d.ts

@@ -89,9 +89,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -100,12 +98,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -177,9 +170,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -188,12 +179,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -230,9 +216,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
     } & {
         setMinimized(flag: boolean): void;
         showDisplay(displayId: string, initialSnapshot?: {} | undefined): void;
-        hideDisplay(displayId: string): number; /**
-         * #property
-         */
+        hideDisplay(displayId: string): number;
         replaceDisplay(oldId: string, newId: string, initialSnapshot?: {} | undefined): void;
     } & {
         trackMenuItems(): (import("@jbrowse/core/ui").MenuDivider | import("@jbrowse/core/ui").MenuSubHeader | import("@jbrowse/core/ui").NormalMenuItem | import("@jbrowse/core/ui").CheckboxMenuItem | import("@jbrowse/core/ui").RadioMenuItem | import("@jbrowse/core/ui").SubMenuItem | {
@@ -385,9 +369,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -396,12 +378,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -472,9 +449,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -483,12 +458,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;

package/dist/DotplotView/svgcomponents/SVGDotplotView.js

@@ -17,7 +17,7 @@ const SVGBackground_1 = __importDefault(require("./SVGBackground"));
 async function renderToSvg(model, opts) {
     var _a;
     await (0, mobx_1.when)(() => model.initialized);
-    const { themeName = 'default', Wrapper = ({ children }) => react_1.default.createElement(react_1.default.Fragment, null, children) } = opts;
+    const { themeName = 'default', Wrapper = ({ children }) => children } = opts;
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     const { createRootFn } = (0, mobx_state_tree_1.getRoot)(model);
     const session = (0, util_1.getSession)(model);

package/dist/LaunchDotplotView.js

@@ -23,7 +23,7 @@ function LaunchDotplotView(pluginManager) {
             }
         }
         catch (e) {
-            session.notify(`${e}`, 'error');
+            session.notifyError(`${e}`, e);
             throw e;
         }
     });

package/esm/DotplotReadVsRef/DotplotReadVsRef.js

@@ -92,6 +92,6 @@ export function onClick(feature, self) {
     }
     catch (e) {
         console.error(e);
-        session.notify(`${e}`, 'error');
+        session.notifyError(`${e}`, e);
     }
 }

package/esm/DotplotView/components/DotplotView.js

@@ -150,15 +150,15 @@ const DotplotViewInternal = observer(function ({ model, }) {
     // detect a mouseup after a mousedown was submitted, autoremoves mouseup once
     // that single mouseup is set
     useEffect(() => {
-        if (mousedown && !mouseup) {
-            function globalMouseUp(event) {
-                if (Math.abs(xdistance) > 3 && Math.abs(ydistance) > 3 && validSelect) {
-                    setMouseUpClient([event.clientX, event.clientY]);
-                }
-                else {
-                    setMouseDownClient(undefined);
-                }
+        function globalMouseUp(event) {
+            if (Math.abs(xdistance) > 3 && Math.abs(ydistance) > 3 && validSelect) {
+                setMouseUpClient([event.clientX, event.clientY]);
+            }
+            else {
+                setMouseDownClient(undefined);
             }
+        }
+        if (mousedown && !mouseup) {
             window.addEventListener('mouseup', globalMouseUp, true);
             return () => {
                 window.removeEventListener('mouseup', globalMouseUp, true);

package/esm/DotplotView/components/ExportSvgDialog.js

@@ -7,6 +7,10 @@ function LoadingMessage() {
         React.createElement(CircularProgress, { size: 20, style: { marginRight: 20 } }),
         React.createElement(Typography, { display: "inline" }, "Creating SVG")));
 }
+function TextField2({ children, ...rest }) {
+    return (React.createElement("div", null,
+        React.createElement(TextField, { ...rest }, children)));
+}
 function useSvgLocal(key, val) {
     return useLocalStorage('svg-' + key, val);
 }
@@ -21,9 +25,8 @@ export default function ExportSvgDialog({ model, handleClose, }) {
     return (React.createElement(Dialog, { open: true, onClose: handleClose, title: "Export SVG" },
         React.createElement(DialogContent, null,
             error ? (React.createElement(ErrorMessage, { error: error })) : loading ? (React.createElement(LoadingMessage, null)) : null,
-            React.createElement(TextField, { helperText: "filename", value: filename, onChange: event => setFilename(event.target.value) }),
-            React.createElement("br", null),
-            session.allThemes ? (React.createElement(TextField, { select: true, label: "Theme", value: themeName, onChange: event => setThemeName(event.target.value) }, Object.entries(session.allThemes()).map(([key, val]) => (React.createElement(MenuItem, { key: key, value: key }, 
+            React.createElement(TextField2, { helperText: "filename", value: filename, onChange: event => setFilename(event.target.value) }),
+            session.allThemes ? (React.createElement(TextField2, { select: true, label: "Theme", value: themeName, onChange: event => setThemeName(event.target.value) }, Object.entries(session.allThemes()).map(([key, val]) => (React.createElement(MenuItem, { key: key, value: key }, 
             // @ts-expect-error
             val.name || '(Unknown name)'))))) : null,
             offscreenCanvas ? (React.createElement(FormControlLabel, { control: React.createElement(Checkbox, { checked: rasterizeLayers, onChange: () => setRasterizeLayers(val => !val) }), label: "Rasterize canvas based tracks? File may be much larger if this is turned off" })) : (React.createElement(Typography, null, "Note: rasterizing layers not yet supported in this browser, so SVG size may be large"))),
@@ -43,6 +46,8 @@ export default function ExportSvgDialog({ model, handleClose, }) {
                     catch (e) {
                         console.error(e);
                         setError(e);
+                    }
+                    finally {
                         setLoading(false);
                     }
                 } }, "Submit"))));

package/esm/DotplotView/components/Header.d.ts

@@ -5,6 +5,6 @@ declare const DotplotHeader: ({ model, selection, }: {
     selection?: {
         width: number;
         height: number;
-    } | undefined;
+    };
 }) => React.JSX.Element;
 export default DotplotHeader;

package/esm/DotplotView/model.d.ts

@@ -89,9 +89,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -100,12 +98,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -177,9 +170,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -188,12 +179,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -230,9 +216,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
     } & {
         setMinimized(flag: boolean): void;
         showDisplay(displayId: string, initialSnapshot?: {} | undefined): void;
-        hideDisplay(displayId: string): number; /**
-         * #property
-         */
+        hideDisplay(displayId: string): number;
         replaceDisplay(oldId: string, newId: string, initialSnapshot?: {} | undefined): void;
     } & {
         trackMenuItems(): (import("@jbrowse/core/ui").MenuDivider | import("@jbrowse/core/ui").MenuSubHeader | import("@jbrowse/core/ui").NormalMenuItem | import("@jbrowse/core/ui").CheckboxMenuItem | import("@jbrowse/core/ui").RadioMenuItem | import("@jbrowse/core/ui").SubMenuItem | {
@@ -385,9 +369,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -396,12 +378,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;
@@ -472,9 +449,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
              * #property
              */
             coord: number;
-            regionNumber?: number | undefined; /**
-             * #property
-             */
+            regionNumber?: number | undefined;
         }): number | undefined;
     } & {
         setFeatures(features: import("@jbrowse/core/util").Feature[]): void;
@@ -483,12 +458,7 @@ export default function stateModelFactory(pm: PluginManager): import("mobx-state
         zoomIn(): void;
         zoomTo(bpPerPx: number, offset?: number | undefined): number;
         scrollTo(offsetPx: number): number;
-        centerAt(coord: number, refName: string | undefined, regionNumber: number): void; /**
-         * #property
-         * this represents tracks specific to this view specifically used
-         * for read vs ref dotplots where this track would not really apply
-         * elsewhere
-         */
+        centerAt(coord: number, refName: string | undefined, regionNumber: number): void;
         scroll(distance: number): number;
     } & {
         moveTo(start?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined, end?: import("@jbrowse/core/util/Base1DUtils").BpOffset | undefined): void;

package/esm/DotplotView/svgcomponents/SVGDotplotView.js

@@ -11,7 +11,7 @@ import SVGBackground from './SVGBackground';
 export async function renderToSvg(model, opts) {
     var _a;
     await when(() => model.initialized);
-    const { themeName = 'default', Wrapper = ({ children }) => React.createElement(React.Fragment, null, children) } = opts;
+    const { themeName = 'default', Wrapper = ({ children }) => children } = opts;
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     const { createRootFn } = getRoot(model);
     const session = getSession(model);

package/esm/LaunchDotplotView.js

@@ -21,7 +21,7 @@ export default function LaunchDotplotView(pluginManager) {
             }
         }
         catch (e) {
-            session.notify(`${e}`, 'error');
+            session.notifyError(`${e}`, e);
             throw e;
         }
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jbrowse/plugin-dotplot-view",
-  "version": "2.10.3",
+  "version": "2.11.0",
   "description": "JBrowse 2 dotplot view",
   "keywords": [
     "jbrowse",
@@ -38,7 +38,7 @@
   "dependencies": {
     "@floating-ui/react": "^0.26.3",
     "@mui/icons-material": "^5.0.1",
-    "@mui/x-data-grid": "^6.0.1",
+    "@mui/x-data-grid": "^7.0.0",
     "@types/file-saver": "^2.0.1",
     "@types/normalize-wheel": "^1.0.0",
     "clone": "^2.1.2",
@@ -63,5 +63,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "c8fc800cd17decd72b2e971c7a6add3b95214e72"
+  "gitHead": "3d43a820b9274a6160aa4dc15616147f390d9094"
 }