[Amsterdam] Tighten title line-heights (elastic#4133)

kshitij86 · Nov 29, 2020 · 38d106d · 38d106d
1 parent 52077cf
commit 38d106d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,8 @@
 ## [`master`](https://github.com/elastic/eui/tree/master)
 
-No public interface changes since `29.5.0`.
+**Theme: Amsterdam**
+
+- Tightened `line-height` for some `EuiTitle` sizes ([4133](https://github.com/elastic/eui/pull/4133))
 
 ## [`29.5.0`](https://github.com/elastic/eui/tree/v29.5.0)
 

diff --git a/src/themes/eui-amsterdam/global_styling/variables/_typography.scss b/src/themes/eui-amsterdam/global_styling/variables/_typography.scss
@@ -22,7 +22,7 @@ $euiFontSizeXXL:  floor($euiFontSize * 2.43); // 34px // h1
 $euiTitles: (
   'xxxs': (
     'font-size': $euiFontSizeXS,
-    'line-height': lineHeightFromBaseline(3),
+    'line-height': lineHeightFromBaseline(2),
     'font-weight': $euiFontWeightBold,
   ),
   'xxs': (
@@ -42,12 +42,12 @@ $euiTitles: (
   ),
   'm': (
     'font-size': $euiFontSizeXL,
-    'line-height': lineHeightFromBaseline(5),
+    'line-height': lineHeightFromBaseline(4),
     'font-weight': $euiFontWeightBold,
   ),
   'l': (
     'font-size': $euiFontSizeXXL,
-    'line-height': lineHeightFromBaseline(6),
+    'line-height': lineHeightFromBaseline(5),
     'font-weight': $euiFontWeightBold,
   ),
 );