platform: correct vars

solidjs-community · Sep 6, 2023 · 1e3258c · 1e3258c
1 parent 1b286ba
commit 1e3258c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/packages/platform/src/index.ts b/packages/platform/src/index.ts
@@ -48,7 +48,7 @@ export const isOpera: boolean =
 
 /** Browser is Safari */
 export const isSafari: boolean =
-  n.vendor && n.vendor.includes("Apple") && ua && !ua.includes("CriOS") && !ua.includes("FxiOS");
+  !!n.vendor && n.vendor.includes("Apple") && ua && !ua.includes("CriOS") && !ua.includes("FxiOS");
 
 /** Browser is Internet Explorer 6-11 */
 // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
@@ -64,7 +64,7 @@ export const isEdge: boolean = /*#__PURE__*/ /Edg/.test(ua) && isChromium;
 export const isChrome: boolean = isChromium && n.vendor === "Google Inc." && !isOpera && !isEdge;
 
 /** Browser is Brave */
-export const isBrave: boolean = n.brave && n.brave.isBrave && n.brave.isBrave.name === "isBrave";
+export const isBrave: boolean = !!n.brave && n.brave.isBrave && n.brave.isBrave.name === "isBrave";
 
 //
 // Rendering Engines

diff --git a/packages/platform/test/server.test.ts b/packages/platform/test/server.test.ts
@@ -28,6 +28,7 @@ describe("platform", () => {
     expect(platform.isChromium).toBe(false);
     expect(platform.isEdge).toBe(false);
     expect(platform.isChrome).toBe(false);
+    expect(platform.isBrave).toBe(false);
 
     //
     // Rendering Engines