fix: disable pointer events on app icons (#5664)

kylecarbs · web-flow · commit 01652e8afb57 · 2023-01-10T21:42:33.000Z
Ben accidentally clicked to open this in a new tab
which seemed kinda janky UX-wise on our part.
diff --git 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 />
   )
