fix: disable pointer events on app icons

kylecarbs · kylecarbs · commit e396cce43caf · 2023-01-10T21:29:41.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,9 @@ 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 />
   )
