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', '⇧');
 }