From b5d43c8d1a9918ffacbd61db712b4bbede3f07a0 Mon Sep 17 00:00:00 2001
From: Summer Cook <summer@scientist.com>
Date: Wed, 15 Nov 2023 09:18:37 -0600
Subject: [PATCH 1/2] add target option to links

---
 src/components/Link/Link.jsx          |  5 ++++-
 src/components/Link/Link.stories.jsx  |  8 ++++++++
 src/compounds/LinkGroup/LinkGroup.jsx | 12 +++++++++++-
 src/resources/args.js                 |  5 +++++
 4 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/src/components/Link/Link.jsx b/src/components/Link/Link.jsx
index 76bdeb92..fcfa5003 100644
--- a/src/components/Link/Link.jsx
+++ b/src/components/Link/Link.jsx
@@ -3,12 +3,13 @@ import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'
 import PropTypes from 'prop-types'
 import './link.css'
 
-const Link = ({ addClass, href, icon, label, style, ...props }) => {
+const Link = ({ addClass, href, icon, label, style, target, ...props }) => {
   return (
     <a
       href={href}
       className={`link ${addClass}`}
       style={style}
+      target={target}
       {...props}
     >
       {icon && (
@@ -31,12 +32,14 @@ Link.propTypes = {
   icon: PropTypes.string,
   label: PropTypes.string.isRequired,
   style: PropTypes.shape({}),
+  target: PropTypes.string,
 }
 
 Link.defaultProps = {
   addClass: '',
   icon: '',
   style: {},
+  target: ''
 }
 
 export default Link
diff --git a/src/components/Link/Link.stories.jsx b/src/components/Link/Link.stories.jsx
index 182070da..d2faaabb 100644
--- a/src/components/Link/Link.stories.jsx
+++ b/src/components/Link/Link.stories.jsx
@@ -36,3 +36,11 @@ NoLabel.args = {
   label: '',
   style: {},
 }
+
+export const ExternalLinkInNewTab = Template.bind({})
+ExternalLinkInNewTab.args = {
+  href: 'https://www.google.com',
+  label: 'I should open in a new tab',
+  style: {},
+  target: '_blank'
+}
diff --git a/src/compounds/LinkGroup/LinkGroup.jsx b/src/compounds/LinkGroup/LinkGroup.jsx
index 85cc74fd..568a9797 100644
--- a/src/compounds/LinkGroup/LinkGroup.jsx
+++ b/src/compounds/LinkGroup/LinkGroup.jsx
@@ -6,7 +6,16 @@ import './link-group.css'
 const LinkGroup = ({ header, headerStyle, links, linkStyle }) => (
   <div className='link-group-container'>
     <p className='link-group-header mb-2' style={headerStyle}>{header}</p>
-    {links.map(({ name, url }) => <Link href={url} key={name} label={name} addClass='link-group-links' style={linkStyle} />)}
+    {links.map(({ name, url, target }) =>
+      <Link
+        href={url}
+        key={name}
+        label={name}
+        target={target}
+        addClass='link-group-links'
+        style={linkStyle}
+      />
+    )}
   </div>
 )
 
@@ -16,6 +25,7 @@ LinkGroup.propTypes = {
   links: PropTypes.arrayOf(PropTypes.shape({
     name: PropTypes.string.isRequired,
     url: PropTypes.string.isRequired,
+    target: PropTypes.string,
   })).isRequired,
   linkStyle: PropTypes.shape({}),
 }
diff --git a/src/resources/args.js b/src/resources/args.js
index daa8fa20..d2620967 100644
--- a/src/resources/args.js
+++ b/src/resources/args.js
@@ -75,6 +75,11 @@ export const links = [
     name: 'Services',
     url: '/services',
   },
+  {
+    name: 'External tab link',
+    url: 'http://www.google.com',
+    target: '_blank',
+  },
 ]
 
 export const sections = [

From 48a4a3b18b3eb8d1c4fec8270e720dfce4bfc032 Mon Sep 17 00:00:00 2001
From: Summer Cook <summer@scientist.com>
Date: Wed, 15 Nov 2023 10:00:19 -0600
Subject: [PATCH 2/2] change to target self

---
 src/components/Link/Link.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/Link/Link.jsx b/src/components/Link/Link.jsx
index fcfa5003..76e5aeca 100644
--- a/src/components/Link/Link.jsx
+++ b/src/components/Link/Link.jsx
@@ -39,7 +39,7 @@ Link.defaultProps = {
   addClass: '',
   icon: '',
   style: {},
-  target: ''
+  target: '_self'
 }
 
 export default Link