fix(Typography): set line height for preset 004 to 28px ratio (#1776)

chanzuckerberg · Oct 6, 2023 · 5b6c400 · 5b6c400
1 parent 31fe80b
commit 5b6c400
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/.storybook/data/tokens.json b/.storybook/data/tokens.json
@@ -123,9 +123,9 @@
   "eds-typography-preset-003-light": "var(--eds-font-weight-light) var(--eds-font-size-22)/1.45454545 var(--eds-font-family-primary)",
   "eds-typography-preset-003-bold": "var(--eds-font-weight-bold) var(--eds-font-size-22)/1.45454545 var(--eds-font-family-primary)",
   "eds-typography-preset-003-mobile": "var(--eds-font-weight-bold) var(--eds-font-size-20)/1.4 var(--eds-font-family-primary)",
-  "eds-typography-preset-004": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)",
-  "eds-typography-preset-004-light": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)",
-  "eds-typography-preset-004-bold": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)",
+  "eds-typography-preset-004": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)",
+  "eds-typography-preset-004-light": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)",
+  "eds-typography-preset-004-bold": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)",
   "eds-typography-preset-005": "var(--eds-font-weight-medium) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary)",
   "eds-typography-preset-005-light": "var(--eds-font-weight-light) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary)",
   "eds-typography-preset-005-bold": "var(--eds-font-weight-bold) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary)",

diff --git a/src/design-tokens/tier-1-definitions/typography.json b/src/design-tokens/tier-1-definitions/typography.json
@@ -112,13 +112,13 @@
         },
         "004": {
           "@": {
-            "value": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)"
+            "value": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)"
           },
           "light": {
-            "value": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)"
+            "value": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)"
           },
           "bold": {
-            "value": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)"
+            "value": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)"
           }
         },
         "005": {

diff --git a/src/tokens-dist/css/variables.css b/src/tokens-dist/css/variables.css
@@ -123,9 +123,9 @@
   --eds-typography-preset-003-light: var(--eds-font-weight-light) var(--eds-font-size-22)/1.45454545 var(--eds-font-family-primary);
   --eds-typography-preset-003-bold: var(--eds-font-weight-bold) var(--eds-font-size-22)/1.45454545 var(--eds-font-family-primary);
   --eds-typography-preset-003-mobile: var(--eds-font-weight-bold) var(--eds-font-size-20)/1.4 var(--eds-font-family-primary); /* This is deprecated and missing from figma */
-  --eds-typography-preset-004: var(--eds-font-weight-medium) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary);
-  --eds-typography-preset-004-light: var(--eds-font-weight-light) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary);
-  --eds-typography-preset-004-bold: var(--eds-font-weight-bold) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary);
+  --eds-typography-preset-004: var(--eds-font-weight-medium) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary);
+  --eds-typography-preset-004-light: var(--eds-font-weight-light) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary);
+  --eds-typography-preset-004-bold: var(--eds-font-weight-bold) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary);
   --eds-typography-preset-005: var(--eds-font-weight-medium) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary);
   --eds-typography-preset-005-light: var(--eds-font-weight-light) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary);
   --eds-typography-preset-005-bold: var(--eds-font-weight-bold) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary);

diff --git a/src/tokens-dist/json/variables-nested.json b/src/tokens-dist/json/variables-nested.json
@@ -181,9 +181,9 @@
           "mobile": "var(--eds-font-weight-bold) var(--eds-font-size-20)/1.4 var(--eds-font-family-primary)"
         },
         "004": {
-          "@": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)",
-          "light": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)",
-          "bold": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.333333333 var(--eds-font-family-primary)"
+          "@": "var(--eds-font-weight-medium) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)",
+          "light": "var(--eds-font-weight-light) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)",
+          "bold": "var(--eds-font-weight-bold) var(--eds-font-size-18)/1.55555556 var(--eds-font-family-primary)"
         },
         "005": {
           "@": "var(--eds-font-weight-medium) var(--eds-font-size-16)/1.5 var(--eds-font-family-primary)",