From 0be7f0886a22c555abba53df19c68f92a57dd2bc Mon Sep 17 00:00:00 2001
From: sprochazka <s.prochazka6@gmail.com>
Date: Wed, 25 Jun 2025 08:08:40 +0200
Subject: [PATCH] Fixed external links in Navigation buttons

---
 .../src/airflow/ui/src/layouts/Nav/NavButton.tsx   | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/airflow-core/src/airflow/ui/src/layouts/Nav/NavButton.tsx b/airflow-core/src/airflow/ui/src/layouts/Nav/NavButton.tsx
index 4ebaa90b7c3ea..6c0eec7f18e7f 100644
--- a/airflow-core/src/airflow/ui/src/layouts/Nav/NavButton.tsx
+++ b/airflow-core/src/airflow/ui/src/layouts/Nav/NavButton.tsx
@@ -45,6 +45,13 @@ export const NavButton = ({ icon, isExternal = false, title, to, ...rest }: NavB
       <Box fontSize="xs">{title}</Box>
     </Button>
   ) : isExternal ? (
+    <Link href={to} px={2} rel="noopener noreferrer" target="_blank">
+      <Button {...styles} variant="ghost" {...rest}>
+        <Box alignSelf="center">{icon}</Box>
+        <Box fontSize="xs">{title}</Box>
+      </Button>
+    </Link>
+  ) : (
     <NavLink to={to}>
       {({ isActive }: { readonly isActive: boolean }) => (
         <Button {...styles} variant={isActive ? "solid" : "ghost"} {...rest}>
@@ -53,11 +60,4 @@ export const NavButton = ({ icon, isExternal = false, title, to, ...rest }: NavB
         </Button>
       )}
     </NavLink>
-  ) : (
-    <Link href={to} px={2} rel="noopener noreferrer" target="_blank">
-      <Button {...styles} variant="ghost" {...rest}>
-        <Box alignSelf="center">{icon}</Box>
-        <Box fontSize="xs">{title}</Box>
-      </Button>
-    </Link>
   );