1 files changed, 35 insertions, 38 deletions

diff --git a/src/client/views/GlobalKeyHandler.ts b/src/client/views/GlobalKeyHandler.ts
index 2f64ea28c..5c923d301 100644
--- a/src/client/views/GlobalKeyHandler.ts
+++ b/src/client/views/GlobalKeyHandler.ts
@@ -67,7 +67,6 @@ export class KeyManager {
         // accumulate buffer of characters to insert in a new text note.  once the note is created, it will stop keyboard events from reaching this function.
         if (FormattedTextBox.SelectOnLoadChar) FormattedTextBox.SelectOnLoadChar = FormattedTextBox.SelectOnLoadChar + (e.key === 'Enter' ? '\n' : e.key);
         e.key === 'Control' && (CtrlKey = true);
-        //if (!Doc.noviceMode && e.key.toLocaleLowerCase() === "shift") DocServer.UPDATE_SERVER_CACHE(true);
         const keyname = e.key && e.key.toLowerCase();
         this.handleGreedy(keyname);
 
@@ -89,10 +88,7 @@ export class KeyManager {
         control.preventDefault && e.preventDefault();
     });
 
-    private handleGreedy = action((keyname: string) => {
-        switch (keyname) {
-        }
-    });
+    private handleGreedy = action((/* keyname: string */) => {});
 
     nudge = (x: number, y: number, label: string) => {
         const nudgeable = SelectionManager.Views.some(dv => dv.CollectionFreeFormDocumentView?.nudge);
@@ -105,7 +101,7 @@ export class KeyManager {
             case 'u':
                 if (document.activeElement?.tagName !== 'INPUT' && document.activeElement?.tagName !== 'TEXTAREA') {
                     const ungroupings = SelectionManager.Views;
-                    UndoManager.RunInBatch(() => ungroupings.map(dv => (dv.layoutDoc.group = undefined)), 'ungroup');
+                    UndoManager.RunInBatch(() => ungroupings.forEach(dv => { dv.layoutDoc.group = undefined; }), 'ungroup');
                     SelectionManager.DeselectAll();
                 }
                 break;
@@ -124,41 +120,41 @@ export class KeyManager {
             case ' ':
                 // MarqueeView.DragMarquee = !MarqueeView.DragMarquee; // bcz: this needs a better disclosure UI
                 break;
-            case 'escape':
-                DocumentLinksButton.StartLink = undefined;
-                DocumentLinksButton.StartLinkView = undefined;
-                InkStrokeProperties.Instance._controlButton = false;
-                Doc.ActiveTool = InkTool.None;
-                DragManager.CompleteWindowDrag?.(true);
-                var doDeselect = true;
-                if (SnappingManager.IsDragging) {
-                    DragManager.AbortDrag();
-                } 
-                if (CollectionDockingView.Instance?.HasFullScreen) {
-                    CollectionDockingView.Instance?.CloseFullScreen();
-                } 
-                if (CollectionStackedTimeline.SelectingRegions.size) {
-                    CollectionStackedTimeline.StopSelecting();
-                    doDeselect = false;
-                } else {
-                    doDeselect = !ContextMenu.Instance.closeMenu();
-                }
-                if (doDeselect) {
-                    SelectionManager.DeselectAll();
-                    LightboxView.Instance.SetLightboxDoc(undefined);
+            case 'escape': {
+                    DocumentLinksButton.StartLink = undefined;
+                    DocumentLinksButton.StartLinkView = undefined;
+                    InkStrokeProperties.Instance._controlButton = false;
+                    Doc.ActiveTool = InkTool.None;
+                    DragManager.CompleteWindowDrag?.(true);
+                    let doDeselect = true;
+                    if (SnappingManager.IsDragging) {
+                        DragManager.AbortDrag();
+                    } 
+                    if (CollectionDockingView.Instance?.HasFullScreen) {
+                        CollectionDockingView.Instance?.CloseFullScreen();
+                    } 
+                    if (CollectionStackedTimeline.SelectingRegions.size) {
+                        CollectionStackedTimeline.StopSelecting();
+                        doDeselect = false;
+                    } else {
+                        doDeselect = !ContextMenu.Instance.closeMenu();
+                    }
+                    if (doDeselect) {
+                        SelectionManager.DeselectAll();
+                        LightboxView.Instance.SetLightboxDoc(undefined);
+                    }
+                    // DictationManager.Controls.stop();
+                    GoogleAuthenticationManager.Instance.cancel();
+                    SharingManager.Instance.close();
+                    if (!GroupManager.Instance.isOpen) SettingsManager.Instance.close();
+                    GroupManager.Instance.close();
+                    window.getSelection()?.empty();
+                    document.body.focus();
                 }
-                // DictationManager.Controls.stop();
-                GoogleAuthenticationManager.Instance.cancel();
-                SharingManager.Instance.close();
-                if (!GroupManager.Instance.isOpen) SettingsManager.Instance.close();
-                GroupManager.Instance.close();
-                window.getSelection()?.empty();
-                document.body.focus();
                 break;
-            case 'enter': {
-                //DocumentDecorations.Instance.onCloseClick(false);
+            case 'enter':
+                // DocumentDecorations.Instance.onCloseClick(false);
                 break;
-            }
             case 'delete':
             case 'backspace':
                 if (document.activeElement?.tagName !== 'INPUT' && document.activeElement?.tagName !== 'TEXTAREA') {
@@ -173,6 +169,7 @@ export class KeyManager {
             case 'arrowright': return this.nudge(1,0, 'nudge right');
             case 'arrowup':    return this.nudge(0, -1, 'nudge up');
             case 'arrowdown':  return this.nudge(0, 1, 'nudge down');
+            default:
         } // prettier-ignore
 
         return {