From 0b59a721f7e6e7eb49342c232ab6a7f05baab860 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ja=CC=81n=20Mikla=CC=81s=CC=8C?= <neosinner@gmail.com>
Date: Tue, 20 Apr 2021 11:18:27 +0200
Subject: [PATCH] Remove large dimension from <Button> component

[MAILPOET-3545]
---
 assets/css/src/generic/_buttons.scss          |  5 ---
 .../js/src/common/button/_stories/button.tsx  | 25 --------------
 .../common/button/_stories/button_icons.tsx   | 33 -------------------
 assets/js/src/common/button/button.tsx        |  2 +-
 4 files changed, 1 insertion(+), 64 deletions(-)

diff --git a/assets/css/src/generic/_buttons.scss b/assets/css/src/generic/_buttons.scss
index 6168c40f4b..2a53142ef9 100644
--- a/assets/css/src/generic/_buttons.scss
+++ b/assets/css/src/generic/_buttons.scss
@@ -74,11 +74,6 @@
   }
 }
 
-.mailpoet-button-large {
-  min-height: 48px;
-  padding: 13px 24px 11px;
-}
-
 .mailpoet-button-light {
   background: $color-secondary-light;
   color: $color-secondary;
diff --git a/assets/js/src/common/button/_stories/button.tsx b/assets/js/src/common/button/_stories/button.tsx
index d5a1468e35..69e2967cc8 100644
--- a/assets/js/src/common/button/_stories/button.tsx
+++ b/assets/js/src/common/button/_stories/button.tsx
@@ -57,31 +57,6 @@ export const WithoutIcons = () => (
     </p>
     <br />
 
-    <Heading level={3}>Large buttons</Heading>
-    <p>
-      <Button
-        onClick={action('light large')}
-        dimension="large"
-        variant="light"
-      >
-        Light button
-      </Button>
-      <Button
-        onClick={action('regular large')}
-        dimension="large"
-      >
-        Regular button
-      </Button>
-      <Button
-        onClick={action('link large')}
-        dimension="large"
-        variant="link"
-      >
-        Link button
-      </Button>
-    </p>
-    <br />
-
     <Heading level={3}>Disabled buttons</Heading>
     <p>
       <Button
diff --git a/assets/js/src/common/button/_stories/button_icons.tsx b/assets/js/src/common/button/_stories/button_icons.tsx
index bdedbd8f82..26b48394ca 100644
--- a/assets/js/src/common/button/_stories/button_icons.tsx
+++ b/assets/js/src/common/button/_stories/button_icons.tsx
@@ -74,39 +74,6 @@ export const WithIcons = () => (
     </p>
     <br />
 
-    <Heading level={3}>Large buttons</Heading>
-    <p>
-      <Button
-        onClick={action('only icon large')}
-        dimension="large"
-        variant="light"
-        iconStart={icon}
-      />
-      <Button
-        onClick={action('icon start large')}
-        dimension="large"
-        iconStart={icon}
-      >
-        Icon start
-      </Button>
-      <Button
-        onClick={action('both icons large')}
-        dimension="large"
-        variant="light"
-        iconStart={icon}
-        iconEnd={icon}
-      >
-        Both icons
-      </Button>
-      <Button
-        onClick={action('only icon link large')}
-        dimension="large"
-        variant="link"
-        iconStart={icon}
-      />
-    </p>
-    <br />
-
     <Heading level={3}>Disabled buttons</Heading>
     <p>
       <Button
diff --git a/assets/js/src/common/button/button.tsx b/assets/js/src/common/button/button.tsx
index 24d2264629..b8ef4ffe00 100644
--- a/assets/js/src/common/button/button.tsx
+++ b/assets/js/src/common/button/button.tsx
@@ -3,7 +3,7 @@ import classnames from 'classnames';
 
 interface Props extends React.ButtonHTMLAttributes<HTMLButtonElement> {
   children?: React.ReactNode;
-  dimension?: 'extra-small' | 'small' | 'large';
+  dimension?: 'extra-small' | 'small';
   variant?: 'light' | 'link';
   withSpinner?: boolean; // also disables href and onClick (via pointer-events in CSS)
   isDisabled?: boolean; // also disables href and onClick (via pointer-events in CSS)