From c87f42ee43be3e1bb3c8c5fe05654736bd8d9db5 Mon Sep 17 00:00:00 2001
From: Zhenya Goroh <zhenyagoroh@gmail.com>
Date: Tue, 8 Oct 2024 21:46:26 +0300
Subject: [PATCH] Added rel=noopener noreferrer to some links

---
 src/components/branding-assets.tsx |  2 +-
 src/components/download.tsx        |  3 +-
 src/components/features.tsx        |  7 +++--
 src/components/footer.tsx          | 44 +++++++++++++++++++++++++-----
 src/components/mobile-nav.tsx      |  4 ++-
 src/components/navigation.tsx      |  8 +++++-
 src/components/release-note.tsx    | 10 +++++++
 7 files changed, 64 insertions(+), 14 deletions(-)

diff --git a/src/components/branding-assets.tsx b/src/components/branding-assets.tsx
index 110f744..328d92f 100644
--- a/src/components/branding-assets.tsx
+++ b/src/components/branding-assets.tsx
@@ -75,7 +75,7 @@ export function BrandingAssets() {
 						CC BY-SA 4.0
 					</a>
 					. Thanks to{" "}
-					<a href="https://www.onnno.nl/" target="_blank" className="text-blue-500">
+					<a href="https://www.onnno.nl/" rel="noopener noreferrer" target="_blank" className="text-blue-500">
 						Donno (mr. Logos)
 					</a>{" "}
 					for the assets.
diff --git a/src/components/download.tsx b/src/components/download.tsx
index fee9ef2..690c380 100644
--- a/src/components/download.tsx
+++ b/src/components/download.tsx
@@ -227,10 +227,11 @@ export default function DownloadPage() {
 								the web with Zen!
 							</p>
 							<div className="mt-5 flex items-center font-bold">
-								<a target="_blank" href="https://github.com/zen-browser">Source Code</a>
+								<a target="_blank" rel="noopener noreferrer" href="https://github.com/zen-browser">Source Code</a>
 								<a
 									target="_blank"
 									className="ml-5"
+									rel="noopener noreferrer"
 									href="https://patreon.com/zen_browser?utm_medium=unknown&utm_source=join_link&utm_campaign=creatorshare_creator&utm_content=copyLink"
 								>
 									Donate
diff --git a/src/components/features.tsx b/src/components/features.tsx
index 713383f..7ec9721 100644
--- a/src/components/features.tsx
+++ b/src/components/features.tsx
@@ -140,7 +140,7 @@ export default function Features() {
 							browser.
 						</p>
 						<div className="relative">
-							<a href="https://github.com/zen-browser" target="_blank">
+							<a href="https://github.com/zen-browser" rel="noopener noreferrer" target="_blank">
 								<Button className="mt-8">GitHub</Button>
 							</a>
 						</div>
@@ -474,6 +474,7 @@ export default function Features() {
 							<a
 								href="https://github.com/zen-browser/desktop?tab=readme-ov-file#compatibility"
 								target="_blank"
+								rel="noopener noreferrer"
 								className="tems-center m-auto flex h-fit w-fit animate-fade-in items-center rounded-full border-2 border-white bg-surface p-4 opacity-0 shadow [--animation-delay:300ms]"
 							>
 								See what version of Firefox Zen uses{" "}
@@ -547,12 +548,12 @@ export default function Features() {
 						cause.
 					</p>
 					<div className="relative mt-8 flex">
-						<a href="https://patreon.com/zen_browser" target="_blank">
+						<a href="https://patreon.com/zen_browser" rel="noopener noreferrer" target="_blank">
 							<Button data-umami-event="patreon-feature" variant="ghost">
 								Patreon <ExternalLinkIcon className="ml-4 h-4 w-4 opacity-50" />
 							</Button>
 						</a>
-						<a href="https://ko-fi.com/zen_browser" target="_blank">
+						<a href="https://ko-fi.com/zen_browser" rel="noopener noreferrer" target="_blank">
 							<Button
 								data-umami-event="ko-fi-feature"
 								className="ml-8"
diff --git a/src/components/footer.tsx b/src/components/footer.tsx
index 6874e34..efeaece 100644
--- a/src/components/footer.tsx
+++ b/src/components/footer.tsx
@@ -20,6 +20,7 @@ export default function Footer() {
 						<a
 							target="_blank"
 							data-umami-event="footer-status"
+							rel="noopener noreferrer"
 							href="https://uptime.zen-browser.app/"
 							className="mt-2 flex w-fit cursor-pointer items-center rounded-md bg-surface px-3 py-2 shadow"
 						>
@@ -35,16 +36,35 @@ export default function Footer() {
 						</a>
 						<h2 className="text-md mt-6 font-bold opacity-80">Follow Us</h2>
 						<div className="mt-4 flex opacity-70">
-							<a target="_blank" href="https://github.com/zen-browser">
+							<a
+								target="_blank"
+								rel="noopener noreferrer"
+								href="https://github.com/zen-browser"
+							>
 								<GitHubLogoIcon className="h-5 w-5" />
 							</a>
-							<a target="_blank" href="https://discord.gg/zen-browser" className="ml-5">
+							<a
+								target="_blank"
+								rel="noopener noreferrer"
+								href="https://discord.gg/zen-browser"
+								className="ml-5"
+							>
 								<DiscordLogoIcon className="h-5 w-5" />
 							</a>
-							<a target="_blank" href="https://fosstodon.org/@zenbrowser" className="ml-5">
+							<a
+								target="_blank"
+								rel="noopener noreferrer"
+								href="https://fosstodon.org/@zenbrowser"
+								className="ml-5"
+							>
 								<MastodonLogo className="h-5 w-5" />
 							</a>
-							<a target="_blank" href="https://twitter.com/zen_browser" className="ml-5">
+							<a
+								target="_blank"
+								rel="noopener noreferrer"
+								href="https://twitter.com/zen_browser"
+								className="ml-5"
+							>
 								<TwitterLogoIcon className="h-5 w-5" />
 							</a>
 						</div>
@@ -81,7 +101,10 @@ export default function Footer() {
 								</a>
 							</li>
 							<li className="mt-2 font-normal">
-								<a target="_blank" href="https://github.com/zen-browser/desktop/issues">
+								<a
+									target="_blank"
+									href="https://github.com/zen-browser/desktop/issues"
+								>
 									Report an Issue
 								</a>
 							</li>
@@ -108,10 +131,17 @@ export default function Footer() {
 								<a href="/branding-assets">Branding Assets</a>
 							</li>
 							<li className="mt-2">
-								<a target="_blank" href="https://github.com/zen-browser/desktop">Source Code</a>
+								<a
+									target="_blank"
+									href="https://github.com/zen-browser/desktop"
+								>
+									Source Code
+								</a>
 							</li>
 							<li className="mt-2">
-								<a target="_blank" href="https://docs.zen-browser.app">Documentation</a>
+								<a target="_blank" href="https://docs.zen-browser.app">
+									Documentation
+								</a>
 							</li>
 							<li className="mt-2">
 								<a href="/release-notes">Release Notes</a>
diff --git a/src/components/mobile-nav.tsx b/src/components/mobile-nav.tsx
index 9332546..6a1f0bf 100644
--- a/src/components/mobile-nav.tsx
+++ b/src/components/mobile-nav.tsx
@@ -90,18 +90,20 @@ export function MobileNav() {
 						</MobileLink>
 						<MobileLink
 							href="https://patreon.com/zen_browser?utm_medium=unknown&utm_source=join_link&utm_campaign=creatorshare_creator&utm_content=copyLink"
+							rel="noopener noreferrer"
 							onOpenChange={setOpen}
 							target="_blank"
 						>
 							<div>Donate {"<"}3</div>
 							<p className="text-xs opacity-60">Support the project</p>
 						</MobileLink>
-						{components.map(({ title, href, description, isTargetBlank }) => (
+						{components.map(({ title, href, description, isTargetBlank, rel }) => (
 							<MobileLink
 								href={href}
 								key={href}
 								target={isTargetBlank ? "_blank" : "_self"}
 								onOpenChange={setOpen}
+								rel={rel}
 							>
 								<div>{title}</div>
 								<p className="text-xs opacity-60">{description}</p>
diff --git a/src/components/navigation.tsx b/src/components/navigation.tsx
index 771bdeb..d679b0e 100644
--- a/src/components/navigation.tsx
+++ b/src/components/navigation.tsx
@@ -22,6 +22,7 @@ export const components: {
 	href: string;
 	description: string;
 	isTargetBlank?: boolean;
+	rel?: "noopener noreferrer" 
 }[] = [
 	{
 		title: "Privacy Policy",
@@ -35,6 +36,7 @@ export const components: {
 		description:
 			"Join our Discord server to chat with the community and get support.",
 		isTargetBlank: true,
+		rel: "noopener noreferrer"
 	},
 	{
 		title: "Source Code",
@@ -42,6 +44,7 @@ export const components: {
 		description:
 			"View the source code on GitHub and contribute to the project.",
 		isTargetBlank: true,
+		rel: "noopener noreferrer"
 	},
 	{
 		title: "Branding Assets",
@@ -169,6 +172,7 @@ export function Navigation() {
 									title="Patreon"
 									href="https://patreon.com/zen_browser?utm_medium=unknown&utm_source=join_link&utm_campaign=creatorshare_creator&utm_content=copyLink"
 									target="_blank"
+									rel="noopener noreferrer"
 								>
 									Support us on Patreon and get exclusive rewards and keep the
 									project alive.
@@ -177,6 +181,7 @@ export function Navigation() {
 									title="Ko-Fi"
 									href="https://ko-fi.com/zen_browser?utm_medium=unknown&utm_source=join_link&utm_campaign=creatorshare_creator&utm_content=copyLink"
 									target="_blank"
+									rel="noopener noreferrer"
 								>
 									Ko-fi is a way to support us with a one-time donation and help
 									us keep the project alive.
@@ -189,12 +194,13 @@ export function Navigation() {
 						<NavigationMenuContent>
 							<ul className="grid w-[400px] gap-3 p-4 md:w-[500px] md:grid-cols-2 lg:w-[600px]">
 								{components.map(
-									({ description, href, title, isTargetBlank }) => (
+									({ description, href, title, isTargetBlank, rel }) => (
 										<ListItem
 											key={title}
 											title={title}
 											href={href}
 											target={isTargetBlank ? "_blank" : "_self"}
+											rel={rel}
 										>
 											{description}
 										</ListItem>
diff --git a/src/components/release-note.tsx b/src/components/release-note.tsx
index 6bcd67c..6e369a1 100644
--- a/src/components/release-note.tsx
+++ b/src/components/release-note.tsx
@@ -20,6 +20,8 @@ function DateContainer({ data }: { data: ReleaseNote }) {
 			}).format("MMMM Do, YYYY")}
 			<div className="mt-2 flex items-center text-blue-500 opacity-60">
 				<a
+					rel="noopener noreferrer"
+					target="_blank"
 					href={`https://github.com/zen-browser/desktop/releases/tag/${data.version}`}
 				>
 					GitHub Release
@@ -28,6 +30,8 @@ function DateContainer({ data }: { data: ReleaseNote }) {
 					<>
 						<span className="mx-1 text-muted-foreground">•</span>
 						<a
+							rel="noopener noreferrer"
+							target="_blank"
 							href={`https://github.com/zen-browser/desktop/actions/runs/${data.workflowId}`}
 						>
 							Workflow Run
@@ -43,6 +47,8 @@ function DateContainer({ data }: { data: ReleaseNote }) {
 						<p className="mt-2">
 							Consider joining our{" "}
 							<a
+								rel="noopener noreferrer"
+								target="_blank"
 								href="https://discord.gg/zen-browser"
 								className="text-blue-500"
 							>
@@ -85,6 +91,8 @@ export default function ReleaseNoteElement({ data }: { data: ReleaseNote }) {
 				<p className="text-md mt-4 text-muted-foreground">
 					If you encounter any issues, please report them on{" "}
 					<a
+						rel="noopener noreferrer"
+						target="_blank"
 						href="https://github.com/zen-browser/desktop/issues/"
 						className="text-underline text-blue-500"
 					>
@@ -148,6 +156,8 @@ export default function ReleaseNoteElement({ data }: { data: ReleaseNote }) {
 											<span className="ml-1">{fix.description}</span>
 											{fix.issue && (
 												<a
+													rel="noopener noreferrer"
+													target="_blank"
 													href={`https://github.com/zen-browser/desktop/issues/${fix.issue}`}
 													className="text-underline ml-1 text-blue-500"
 												>