Remove reload and use link for new tab

code-asher · code-asher · commit 14674b4d4ba9 · 2025-06-18T10:46:38.000-08:00
diff --git a/site/src/pages/TaskPage/TaskAppIframe.tsx b/site/src/pages/TaskPage/TaskAppIframe.tsx
@@ -12,10 +12,10 @@ import {
 	HouseIcon,
 	RotateCwIcon,
 } from "lucide-react";
-import { openAppInNewWindow } from "modules/apps/apps";
 import { useAppLink } from "modules/apps/useAppLink";
 import type { Task } from "modules/tasks/tasks";
 import { type FC, useRef } from "react";
+import { Link as RouterLink } from "react-router-dom";
 import { cn } from "utils/cn";
 
 type TaskAppIFrameProps = {
@@ -64,20 +64,6 @@ export const TaskAppIFrame: FC<TaskAppIFrameProps> = ({
 	return (
 		<div className={cn([active ? "flex" : "hidden", "w-full h-full flex-col"])}>
 			<div className="bg-surface-tertiary flex items-center p-2 py-1 gap-1">
-				<Button
-					size="icon"
-					variant="subtle"
-					onClick={(e) => {
-						e.preventDefault();
-						if (frameRef.current?.contentWindow) {
-							frameRef.current.contentWindow.location.reload();
-						}
-					}}
-				>
-					<RotateCwIcon />
-					<span className="sr-only">Refresh</span>
-				</Button>
-
 				<Button
 					size="icon"
 					variant="subtle"
@@ -104,13 +90,11 @@ export const TaskAppIFrame: FC<TaskAppIFrameProps> = ({
 						</Button>
 					</DropdownMenuTrigger>
 					<DropdownMenuContent align="end">
-						<DropdownMenuItem
-							onClick={() => {
-								openAppInNewWindow(frameSrc);
-							}}
-						>
-							<ExternalLinkIcon />
-							Open app in new tab
+						<DropdownMenuItem asChild>
+							<RouterLink to={frameSrc} target="_blank">
+								<ExternalLinkIcon />
+								Open app in new tab
+							</RouterLink>
 						</DropdownMenuItem>
 					</DropdownMenuContent>
 				</DropdownMenu>
