Skip to content

Commit 5d56bbe

Browse files
committed
Remove ExternalLinkHandler — it was preventing links from opening in browser
The handler called preventDefault() then window.open(), which is less reliable in PWA webviews than the browser's native target="_blank" handling. All external links already have target="_blank" rel="noopener noreferrer", so the handler was redundant at best and counterproductive at worst. https://claude.ai/code/session_016tHCpYCgfzkirpLBvmpWZ9
1 parent 6110551 commit 5d56bbe

2 files changed

Lines changed: 0 additions & 44 deletions

File tree

src/app/layout.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import { Source_Serif_4 } from "next/font/google";
33
import "@/styles/globals.css";
44
import Header from "@/components/Header";
55
import Footer from "@/components/Footer";
6-
import ExternalLinkHandler from "@/components/ExternalLinkHandler";
76

87
const sourceSerif = Source_Serif_4({
98
subsets: ["latin"],
@@ -58,7 +57,6 @@ export default function RootLayout({
5857
}}
5958
/>
6059
<a href="#main-content" className="skip-link">Skip to content</a>
61-
<ExternalLinkHandler />
6260
<Header />
6361
<main id="main-content">{children}</main>
6462
<Footer />

src/components/ExternalLinkHandler.tsx

Lines changed: 0 additions & 42 deletions
This file was deleted.

0 commit comments

Comments
 (0)