Remove reload and use link for new tab

code-asher · code-asher · commit b7e7c0d4f18a · 2025-06-18T10:29:24.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 = {
@@ -104,13 +104,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>
