From 25f973f92b75e551f8960c8c156bd14ed23639c3 Mon Sep 17 00:00:00 2001
From: Mario Souto <13791385+omariosouto@users.noreply.github.com>
Date: Fri, 24 Dec 2021 13:15:39 -0300
Subject: [PATCH] feat: add link support to button component

---
 lib/components/box/box-base.tsx          |  4 ++++
 lib/components/button/button.stories.mdx | 22 ++++++++++++++++++++++
 lib/components/button/button.tsx         | 20 ++++++++++++++++++--
 3 files changed, 44 insertions(+), 2 deletions(-)

diff --git a/lib/components/box/box-base.tsx b/lib/components/box/box-base.tsx
index 116d6a5..de526f9 100644
--- a/lib/components/box/box-base.tsx
+++ b/lib/components/box/box-base.tsx
@@ -10,12 +10,14 @@ interface BoxProps {
   id?: string;
   name?: string;
   value?: string;
+  target?: string;
   placeholder?: string;
   children?: React.ReactNode;
   className?: string;
   styleSheet?: StyleSheet;
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   ref?: Ref<any>;
+  onMouseDown?: unknown;
   onClick?: unknown;
 }
 
@@ -128,8 +130,10 @@ BoxBase.defaultProps = {
   value: undefined,
   placeholder: undefined,
   className: '',
+  target: '',
   name: '',
   id: '',
   ref: undefined,
   onClick: undefined,
+  onMouseDown: undefined,
 };
diff --git a/lib/components/button/button.stories.mdx b/lib/components/button/button.stories.mdx
index 203e5d9..6c7bfc6 100644
--- a/lib/components/button/button.stories.mdx
+++ b/lib/components/button/button.stories.mdx
@@ -61,3 +61,25 @@ Button is the component that you use, for get user taps and clicks.
     {(args) => <Button {...args} />}
   </Story>
 </Canvas>
+
+## With link external
+
+<Canvas>
+  <Story
+    name="Button Component with link external"
+    args={{ ...Button.defaultProps, label: 'Link to https://google.com', href: 'https://google.com' }}
+  >
+    {(args) => <Button {...args} />}
+  </Story>
+</Canvas>
+
+## With link internal
+
+<Canvas>
+  <Story
+    name="Button Component with link internal"
+    args={{ ...Button.defaultProps, label: 'Link to /', href: '/' }}
+  >
+    {(args) => <Button {...args} />}
+  </Story>
+</Canvas>
diff --git a/lib/components/button/button.tsx b/lib/components/button/button.tsx
index 0992c66..f159ea1 100644
--- a/lib/components/button/button.tsx
+++ b/lib/components/button/button.tsx
@@ -1,5 +1,6 @@
 /* eslint-disable react/button-has-type */
 import React from 'react';
+import { useRouter } from 'next/router';
 import { StyleSheet } from '@lib/core/stylesheet/stylesheet';
 import { BoxBase } from '@lib/components/box/box-base';
 import { TypographyVariants } from '@lib/core/typography/typography';
@@ -104,6 +105,7 @@ const buttonSizes = {
 
 interface ButtonProps {
   iconName?: string;
+  href?: string;
   fullWidth?: boolean;
   variant?: 'primary' | 'secondary' | 'tertiary';
   buttonColors?: ButtonColorValues;
@@ -133,6 +135,10 @@ export function Button({
   iconName,
   ...props
 }: ButtonProps): JSX.Element {
+  const isLink = Boolean(props.href);
+  const Tag = isLink ? 'a' : 'button';
+  const openInNewTab = props.href && props.href.startsWith('http');
+  const router = useRouter();
   const { onClickRipple, rippleStyle } = useRipples();
   const borderRadius = rounded[props.rounded];
   const { textVariant, ...buttonSize } = buttonSizes[props.size];
@@ -175,7 +181,8 @@ export function Button({
 
   return (
     <BoxBase
-      as="button"
+      as={Tag}
+      target={openInNewTab ? '_blank' : undefined}
       styleSheet={{
         ...buttonSize,
         ...buttonStyle,
@@ -211,9 +218,17 @@ export function Button({
           height: '48px',
         }),
       }}
+      onMouseDown={onClickRipple}
       onClick={(event: React.MouseEvent<HTMLButtonElement, MouseEvent>) => {
         onClick(event);
-        onClickRipple(event);
+        if (isLink && props.href.startsWith('/')) {
+          event.preventDefault();
+          router?.push(props.href);
+          if (process.env.STORYBOOK) {
+            // eslint-disable-next-line no-alert
+            alert('Internal routing! Only supports Next.JS');
+          }
+        }
       }}
       {...props}
     >
@@ -241,6 +256,7 @@ export function Button({
 Button.defaultProps = {
   iconName: undefined,
   fullWidth: false,
+  href: '',
   variant: 'primary',
   colorVariant: 'primary',
   rounded: 'sm',