chore(UI): remove private icon from apps in dashboard

Kira-Pilot · Kira-Pilot · commit a429bd826f82 · 2023-03-27T00:56:10.000Z
diff --git a/site/src/components/AppLink/AppLink.stories.tsx b/site/src/components/AppLink/AppLink.stories.tsx
@@ -35,6 +35,36 @@ WithIconExternal.args = {
   agent: MockWorkspaceAgent,
 }
 
+export const SharingLevelOwner = Template.bind({})
+SharingLevelOwner.args = {
+  workspace: MockWorkspace,
+  app: {
+    ...MockWorkspaceApp,
+    sharing_level: "owner",
+  },
+  agent: MockWorkspaceAgent,
+}
+
+export const SharingLevelAuthenticated = Template.bind({})
+SharingLevelAuthenticated.args = {
+  workspace: MockWorkspace,
+  app: {
+    ...MockWorkspaceApp,
+    sharing_level: "authenticated",
+  },
+  agent: MockWorkspaceAgent,
+}
+
+export const SharingLevelPublic = Template.bind({})
+SharingLevelPublic.args = {
+  workspace: MockWorkspace,
+  app: {
+    ...MockWorkspaceApp,
+    sharing_level: "public",
+  },
+  agent: MockWorkspaceAgent,
+}
+
 export const WithoutIcon = Template.bind({})
 WithoutIcon.args = {
   workspace: MockWorkspace,
diff --git a/site/src/components/AppLink/AppLink.tsx b/site/src/components/AppLink/AppLink.tsx
@@ -5,6 +5,7 @@ import { makeStyles } from "@material-ui/core/styles"
 import Tooltip from "@material-ui/core/Tooltip"
 import ErrorOutlineIcon from "@material-ui/icons/ErrorOutline"
 import { FC } from "react"
+import { combineClasses } from "util/combineClasses"
 import * as TypesGen from "../../api/typesGenerated"
 import { generateRandomString } from "../../util/random"
 import { BaseIcon } from "./BaseIcon"
@@ -79,15 +80,19 @@ export const AppLink: FC<AppLinkProps> = ({
       "Your admin has not configured subdomain application access"
   }
 
+  const isPrivateApp = app.sharing_level === "owner"
+
   const button = (
     <Button
       size="small"
       startIcon={icon}
-      endIcon={<ShareIcon app={app} />}
+      endIcon={isPrivateApp ? undefined : <ShareIcon app={app} />}
       className={styles.button}
       disabled={!canClick}
     >
-      <span className={styles.appName}>{appDisplayName}</span>
+      <span className={combineClasses({ [styles.appName]: !isPrivateApp })}>
+        {appDisplayName}
+      </span>
     </Button>
   )
 
diff --git a/site/src/components/AppLink/ShareIcon.stories.tsx b/site/src/components/AppLink/ShareIcon.stories.tsx
diff --git a/site/src/components/AppLink/ShareIcon.tsx b/site/src/components/AppLink/ShareIcon.tsx
@@ -1,8 +1,6 @@
 import PublicOutlinedIcon from "@material-ui/icons/PublicOutlined"
-import LockOutlinedIcon from "@material-ui/icons/LockOutlined"
 import GroupOutlinedIcon from "@material-ui/icons/GroupOutlined"
 import LaunchOutlinedIcon from "@material-ui/icons/LaunchOutlined"
-import { FC } from "react"
 import * as TypesGen from "../../api/typesGenerated"
 import Tooltip from "@material-ui/core/Tooltip"
 import { useTranslation } from "react-i18next"
@@ -11,23 +9,29 @@ export interface ShareIconProps {
   app: TypesGen.WorkspaceApp
 }
 
-export const ShareIcon: FC<ShareIconProps> = ({ app }) => {
+export const ShareIcon = ({ app }: ShareIconProps) => {
   const { t } = useTranslation("agent")
-
-  let shareIcon = <LockOutlinedIcon />
-  let shareTooltip = t("shareTooltip.private")
+  if (app.external) {
+    return (
+      <Tooltip title={t("shareTooltip.external")}>
+        <LaunchOutlinedIcon />
+      </Tooltip>
+    )
+  }
   if (app.sharing_level === "authenticated") {
-    shareIcon = <GroupOutlinedIcon />
-    shareTooltip = t("shareTooltip.authenticated")
+    return (
+      <Tooltip title={t("shareTooltip.authenticated")}>
+        <GroupOutlinedIcon />
+      </Tooltip>
+    )
   }
   if (app.sharing_level === "public") {
-    shareIcon = <PublicOutlinedIcon />
-    shareTooltip = t("shareTooltip.public")
-  }
-  if (app.external) {
-    shareIcon = <LaunchOutlinedIcon />
-    shareTooltip = t("shareTooltip.external")
+    return (
+      <Tooltip title={t("shareTooltip.public")}>
+        <PublicOutlinedIcon />
+      </Tooltip>
+    )
   }
 
-  return <Tooltip title={shareTooltip}>{shareIcon}</Tooltip>
+  return null
 }
