From dc5637e9432448508b9fa70cdffd5e38a5be9996 Mon Sep 17 00:00:00 2001
From: Michael Glass <me@mike.is>
Date: Tue, 21 Aug 2018 16:43:55 -0700
Subject: [PATCH] move lineHeight out of config

---
 src/Nri/Ui/Button/V3.elm | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/Nri/Ui/Button/V3.elm b/src/Nri/Ui/Button/V3.elm
index b7b5d280..b6d132e5 100644
--- a/src/Nri/Ui/Button/V3.elm
+++ b/src/Nri/Ui/Button/V3.elm
@@ -707,7 +707,6 @@ sizeStyle size width elementType =
                 Small ->
                     { fontSize = 15
                     , height = 36
-                    , lineHeight = 15
                     , sidePadding = 16
                     , imageHeight = 15
                     , shadowHeight = 2
@@ -717,7 +716,6 @@ sizeStyle size width elementType =
                 Medium ->
                     { fontSize = 17
                     , height = 45
-                    , lineHeight = 19
                     , sidePadding = 16
                     , imageHeight = 15
                     , shadowHeight = 3
@@ -727,7 +725,6 @@ sizeStyle size width elementType =
                 Large ->
                     { fontSize = 20
                     , height = 56
-                    , lineHeight = 22
                     , sidePadding = 16
                     , imageHeight = 20
                     , shadowHeight = 4
@@ -752,7 +749,15 @@ sizeStyle size width elementType =
                     config.height
 
                 Button ->
-                    config.lineHeight
+                    case size of
+                        Small ->
+                            15
+
+                        Medium ->
+                            19
+
+                        Large ->
+                            22
     in
     [ Css.fontSize (Css.px config.fontSize)
     , Css.borderRadius (Css.px 8)