From a13cb0ce8e436fe1dba74269059bffd87fc1bc3e Mon Sep 17 00:00:00 2001
From: Adam Gall <galladamj@gmail.com>
Date: Fri, 26 Apr 2024 11:03:37 -0400
Subject: [PATCH] Do Icon cleanup and property consolidation with External
 links too

---
 .../ui/page/Navigation/NavigationExternalLink.tsx         | 8 ++++----
 src/components/ui/page/Navigation/NavigationLinks.tsx     | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/components/ui/page/Navigation/NavigationExternalLink.tsx b/src/components/ui/page/Navigation/NavigationExternalLink.tsx
index 61c6141d05..b91399eaba 100644
--- a/src/components/ui/page/Navigation/NavigationExternalLink.tsx
+++ b/src/components/ui/page/Navigation/NavigationExternalLink.tsx
@@ -1,5 +1,5 @@
 import { Box, Hide, Text, Flex } from '@chakra-ui/react';
-import { ReactElement } from 'react';
+import { Icon } from '@phosphor-icons/react';
 import { useTranslation } from 'react-i18next';
 
 interface INavigationLink {
@@ -8,7 +8,7 @@ interface INavigationLink {
   href: string;
   testId: string;
   routeKey?: string;
-  Icon: ReactElement;
+  NavigationIcon: Icon;
   closeDrawer?: () => void;
 }
 
@@ -16,7 +16,7 @@ export function NavigationExternalLink({
   tooltipKey,
   ariaLabelKey,
   testId,
-  Icon,
+  NavigationIcon,
   routeKey,
   closeDrawer,
   ...rest
@@ -34,7 +34,7 @@ export function NavigationExternalLink({
         rel="noreferrer noopener"
       >
         <Flex>
-          <Box w={6}>{Icon}</Box>
+          <Box w={6}>{<NavigationIcon size={24} />}</Box>
           <Box
             mx={3}
             whiteSpace="nowrap"
diff --git a/src/components/ui/page/Navigation/NavigationLinks.tsx b/src/components/ui/page/Navigation/NavigationLinks.tsx
index 679997d3bd..5c454c2715 100644
--- a/src/components/ui/page/Navigation/NavigationLinks.tsx
+++ b/src/components/ui/page/Navigation/NavigationLinks.tsx
@@ -41,7 +41,7 @@ function ExternalLinks({ closeDrawer }: { closeDrawer?: () => void }) {
           ariaLabelKey="ariaLabelFAQ"
           tooltipKey="faq"
           testId="navigationExternal-faq"
-          Icon={<Question size={24} />}
+          NavigationIcon={Question}
           closeDrawer={closeDrawer}
         />
         <NavigationExternalLink
@@ -49,7 +49,7 @@ function ExternalLinks({ closeDrawer }: { closeDrawer?: () => void }) {
           ariaLabelKey="ariaLabelDiscord"
           tooltipKey="discord"
           testId="navigationExternal-discord"
-          Icon={<DiscordLogo size={24} />}
+          NavigationIcon={DiscordLogo}
           closeDrawer={closeDrawer}
         />
         <NavigationExternalLink
@@ -57,7 +57,7 @@ function ExternalLinks({ closeDrawer }: { closeDrawer?: () => void }) {
           ariaLabelKey="ariaLabelDocumentation"
           tooltipKey="documentation"
           testId="navigationExternal-documentation"
-          Icon={<BookOpen size={24} />}
+          NavigationIcon={BookOpen}
           closeDrawer={closeDrawer}
         />
       </Box>