Constrain track height setting

igvteam · Jan 30, 2025 · 080a0bf · 080a0bf
1 parent e6b354e
commit 080a0bf
Showing 1 changed file with 26 additions and 24 deletions.
diff --git a/js/ui/menuUtils.js b/js/ui/menuUtils.js
@@ -290,38 +290,40 @@ function trackHeightMenuItem() {
 
         const callback = () => {
 
-            const number = parseInt(this.browser.inputDialog.value, 10);
+            if (this.browser.inputDialog.value !== undefined) {
 
-            if (undefined !== number) {
+                const number = parseInt(this.browser.inputDialog.value, 10)
 
-                const tracks = [];
-                if (this.trackView.track.selected) {
-                    tracks.push(...(this.trackView.browser.getSelectedTrackViews().map(({track}) => track)));
-                } else {
-                    tracks.push(this);
-                }
-
-                for (const track of tracks) {
-                    // Explicitly setting track height turns off autoHeight
-                    track.trackView.autoHeight = false;
+                if (number > 0){
 
-                    // If explicitly setting the height adjust min or max, if necessary
-                    if (track.minHeight !== undefined && track.minHeight > number) {
-                        track.minHeight = number;
-                    }
-                    if (track.maxHeight !== undefined && track.maxHeight < number) {
-                        track.minHeight = number;
+                    const tracks = [];
+                    if (this.trackView.track.selected) {
+                        tracks.push(...(this.trackView.browser.getSelectedTrackViews().map(({track}) => track)));
+                    } else {
+                        tracks.push(this);
                     }
-                    track.trackView.setTrackHeight(number, true);
 
-                    track.trackView.checkContentHeight();
-                    track.trackView.repaintViews();
+                    for (const track of tracks) {
+                        // Explicitly setting track height turns off autoHeight
+                        track.trackView.autoHeight = false;
 
-                } // for (tracks)
+                        // If explicitly setting the height adjust min or max, if necessary
+                        if (track.minHeight !== undefined && track.minHeight > number) {
+                            track.minHeight = number;
+                        }
+                        if (track.maxHeight !== undefined && track.maxHeight < number) {
+                            track.minHeight = number;
+                        }
+                        track.trackView.setTrackHeight(number, true);
 
-            } // if (undefined !== number)
+                        track.trackView.checkContentHeight();
+                        track.trackView.repaintViews();
+                    } // for (tracks)
 
-        }; // callback
+                } // if ()
+
+            } // if ()
+        }
 
         const config =
             {