From a6ff90188299468f001cdfac2ad47c783a81e91a Mon Sep 17 00:00:00 2001
From: Kyle Carberry <kyle@carberry.com>
Date: Tue, 10 Jan 2023 21:29:41 +0000
Subject: [PATCH] fix: disable pointer events on app icons

Ben accidentally clicked to open this in a new tab
which seemed kinda janky UX-wise on our part.
---
 site/src/components/AppLink/BaseIcon.tsx | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/site/src/components/AppLink/BaseIcon.tsx b/site/src/components/AppLink/BaseIcon.tsx
index 0df2880d29257..f0ab07658574f 100644
--- a/site/src/components/AppLink/BaseIcon.tsx
+++ b/site/src/components/AppLink/BaseIcon.tsx
@@ -4,7 +4,13 @@ import ComputerIcon from "@material-ui/icons/Computer"
 
 export const BaseIcon: FC<{ app: WorkspaceApp }> = ({ app }) => {
   return app.icon ? (
-    <img alt={`${app.display_name} Icon`} src={app.icon} />
+    <img
+      alt={`${app.display_name} Icon`}
+      src={app.icon}
+      style={{
+        pointerEvents: "none",
+      }}
+    />
   ) : (
     <ComputerIcon />
   )