From a655f03a1e969c2b31bb367161fbf9664f43172d Mon Sep 17 00:00:00 2001
From: Ben Potter <ben@coder.com>
Date: Mon, 6 Feb 2023 10:22:24 -0600
Subject: [PATCH] fix: open terminal and coder_app in a new tab, not window
 (#6044)

* fix!: open coder_app in a new tab, not window

* terminal too
---
 site/src/components/AppLink/AppLink.tsx        | 18 ------------------
 .../components/TerminalLink/TerminalLink.tsx   | 10 ----------
 2 files changed, 28 deletions(-)

diff --git a/site/src/components/AppLink/AppLink.tsx b/site/src/components/AppLink/AppLink.tsx
index 283843a28a..b7c16015fa 100644
--- a/site/src/components/AppLink/AppLink.tsx
+++ b/site/src/components/AppLink/AppLink.tsx
@@ -6,15 +6,9 @@ import Tooltip from "@material-ui/core/Tooltip"
 import ErrorOutlineIcon from "@material-ui/icons/ErrorOutline"
 import { FC } from "react"
 import * as TypesGen from "../../api/typesGenerated"
-import { generateRandomString } from "../../util/random"
 import { BaseIcon } from "./BaseIcon"
 import { ShareIcon } from "./ShareIcon"
 
-const Language = {
-  appTitle: (appName: string, identifier: string): string =>
-    `${appName} - ${identifier}`,
-}
-
 export interface AppLinkProps {
   appsHost?: string
   workspace: TypesGen.Workspace
@@ -98,18 +92,6 @@ export const AppLink: FC<AppLinkProps> = ({
           href={href}
           target="_blank"
           className={canClick ? styles.link : styles.disabledLink}
-          onClick={
-            canClick
-              ? (event) => {
-                  event.preventDefault()
-                  window.open(
-                    href,
-                    Language.appTitle(appDisplayName, generateRandomString(12)),
-                    "width=900,height=600",
-                  )
-                }
-              : undefined
-          }
         >
           {button}
         </Link>
diff --git a/site/src/components/TerminalLink/TerminalLink.tsx b/site/src/components/TerminalLink/TerminalLink.tsx
index b709214648..96c514135b 100644
--- a/site/src/components/TerminalLink/TerminalLink.tsx
+++ b/site/src/components/TerminalLink/TerminalLink.tsx
@@ -5,11 +5,9 @@ import ComputerIcon from "@material-ui/icons/Computer"
 import { FC } from "react"
 import * as TypesGen from "../../api/typesGenerated"
 import { combineClasses } from "../../util/combineClasses"
-import { generateRandomString } from "../../util/random"
 
 export const Language = {
   linkText: "Terminal",
-  terminalTitle: (identifier: string): string => `Terminal - ${identifier}`,
 }
 
 export interface TerminalLinkProps {
@@ -42,14 +40,6 @@ export const TerminalLink: FC<React.PropsWithChildren<TerminalLinkProps>> = ({
       href={href}
       className={combineClasses([styles.link, className])}
       target="_blank"
-      onClick={(event) => {
-        event.preventDefault()
-        window.open(
-          href,
-          Language.terminalTitle(generateRandomString(12)),
-          "width=900,height=600",
-        )
-      }}
     >
       <Button startIcon={<ComputerIcon />} size="small">
         {Language.linkText}