chore: refactor constant IS_MAC in a function isMac to prevent SS…

…R errors
josdejong · Jun 4, 2024 · 40a6fbe · 40a6fbe
1 parent 804b5fd
commit 40a6fbe
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/lib/components/modals/CopyPasteModal.svelte b/src/lib/components/modals/CopyPasteModal.svelte
@@ -2,14 +2,14 @@
 
 <script lang="ts">
   import { getContext } from 'svelte'
-  import { IS_MAC } from '$lib/utils/navigatorUtils.js'
+  import { isMac } from '$lib/utils/navigatorUtils.js'
   import { onEscape } from '$lib/actions/onEscape.js'
   import Header from './Header.svelte'
   import type { Context } from 'svelte-simple-modal'
 
   const { close } = getContext<Context>('simple-modal')
 
-  const ctrl = IS_MAC ? '⌘' : 'Ctrl'
+  const ctrl = isMac() ? '⌘' : 'Ctrl'
 </script>
 
 <div class="jse-modal jse-copy-paste" use:onEscape={() => close()}>

diff --git a/src/lib/utils/navigatorUtils.ts b/src/lib/utils/navigatorUtils.ts
@@ -8,7 +8,8 @@ declare global {
   }
 }
 
-export const IS_MAC =
-  navigator?.platform?.toUpperCase().includes('MAC') ??
-  navigator?.userAgentData?.platform?.toUpperCase().includes('MAC') ??
-  false
+export function isMac() {
+  return navigator?.platform?.toUpperCase().includes('MAC') ??
+    navigator?.userAgentData?.platform?.toUpperCase().includes('MAC') ??
+    false
+}