Merge pull request #31 from acdvorak/fix-navigator-warning

Don't `console.warn()` when `navigator` is `undefined`
reaviz · May 8, 2024 · 55a743e · 55a743e
2 parents 031a5db + c10d24d
commit 55a743e
Showing 1 changed file with 15 additions and 8 deletions.
diff --git a/src/utils.ts b/src/utils.ts
@@ -1,15 +1,22 @@
-let isMac: boolean = false;
-try {
-  if (navigator) isMac = navigator.platform.toUpperCase().indexOf('MAC') >= 0;
-} catch (e) {
-  console.warn(e);
+type ModifierKey = '⌘' | 'CTRL';
+
+function isMac(): boolean {
+  try {
+    return navigator.platform.toUpperCase().indexOf('MAC') >= 0;
+  } catch {
+    return false;
+  }
+}
+
+function getModifierKey(): ModifierKey {
+  isMac() ? '⌘' : 'CTRL'
 }
 
-export const MODIFIER_KEY = isMac ? '⌘' : 'CTRL';
+export const MODIFIER_KEY = getModifierKey();
 
 export function getHotkeyText(hotkey: string) {
   return hotkey
-    .replace('modifier', MODIFIER_KEY)
-    .replace('mod', MODIFIER_KEY)
+    .replace('modifier', getModifierKey())
+    .replace('mod', getModifierKey())
     .replace('shift', '⇧');
 }