From 53d4121778a74ea3b91a5c4b72be6c32313612c6 Mon Sep 17 00:00:00 2001
From: mcoker <mcoker@redhat.com>
Date: Mon, 20 Nov 2023 19:48:41 -0600
Subject: [PATCH] fix(font): use variable font, add opt-in for legacy font

---
 src/patternfly/base/_common.scss             | 12 ++++++++----
 src/patternfly/base/tokens/_tokens-font.scss | 15 ++++++++++++---
 2 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/src/patternfly/base/_common.scss b/src/patternfly/base/_common.scss
index c0bcb63c04..4599fc812a 100644
--- a/src/patternfly/base/_common.scss
+++ b/src/patternfly/base/_common.scss
@@ -18,10 +18,14 @@
 }
 
 // Variable font opt-in
-.#{$pf-prefix}m-vf-font {
-  --#{$pf-global}--FontFamily--text: var(--#{$pf-global}--FontFamily--text--vf);
-  --#{$pf-global}--FontFamily--heading: var(--#{$pf-global}--FontFamily--heading--vf);
-  --#{$pf-global}--FontFamily--monospace: var(--#{$pf-global}--FontFamily--monospace--vf);
+.#{$pf-prefix}m-legacy-font {
+  --pf-t--global--font--family--body: var(--pf-t--global--font--family--body--legacy);
+  --pf-t--global--font--family--heading: var(--pf-t--global--font--family--heading--legacy);
+  --pf-t--global--font--family--mono: var(--pf-t--global--font--family--mono--legacy);
+  --pf-t--global--font--weight--body: var(--pf-t--global--font--weight--body--legacy);
+  --pf-t--global--font--weight--body--bold: var(--pf-t--global--font--weight--body--bold--legacy);
+  --pf-t--global--font--weight--heading: var(--pf-t--global--font--weight--heading--legacy);
+  --pf-t--global--font--weight--heading--bold: var(--pf-t--global--font--weight--heading--bold--legacy);
 }
 
 // RTL helpers
diff --git a/src/patternfly/base/tokens/_tokens-font.scss b/src/patternfly/base/tokens/_tokens-font.scss
index 9dcdb6b41a..85b21e3795 100644
--- a/src/patternfly/base/tokens/_tokens-font.scss
+++ b/src/patternfly/base/tokens/_tokens-font.scss
@@ -5,9 +5,9 @@
 
 :root {
   // Base tokens for fonts
-  --pf-t--global--font--family--100: redhattext;
-  --pf-t--global--font--family--200: redhatdisplay;
-  --pf-t--global--font--family--300: redhatmono;
+  --pf-t--global--font--family--100: redhattextvf, redhattext, helvetica, arial, sans-serif;
+  --pf-t--global--font--family--200: redhatdisplayvf, redhatdisplay, helvetica, arial, sans-serif;
+  --pf-t--global--font--family--300: redhatmonovf, redhatmono, liberationmono, consolas, sfmono-regular, menlo, monaco, courier new, monospace;
   --pf-t--global--font--line-height--100: 1.3;
   --pf-t--global--font--line-height--200: 1.5;
   --pf-t--global--font--weight--body--100: 400;
@@ -45,6 +45,15 @@
   --pf-t--global--font--size--heading--xl: var(--pf-t--global--font--size--heading--500);
   --pf-t--global--font--size--heading--2xl: var(--pf-t--global--font--size--heading--600);
 
+  // Legacy/non-variable font opt-in
+  --pf-t--global--font--family--body--legacy: redhattext, helvetica, arial, sans-serif;
+  --pf-t--global--font--family--heading--legacy: redhatdisplay, helvetica, arial, sans-serif;
+  --pf-t--global--font--family--mono--legacy: redhatmono, liberationmono, consolas, sfmono-regular, menlo, monaco, courier new, monospace;
+  --pf-t--global--font--weight--body--legacy: 400;
+  --pf-t--global--font--weight--body--bold--legacy: 700;
+  --pf-t--global--font--weight--heading--legacy: 400;
+  --pf-t--global--font--weight--heading--bold--legacy: 700;
+
   // Other missing ones
   // text-decoration
   --pf-t--global--text-decoration--100: none;