From d779a4241557cfd076ee04db892e4ebec674952c Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sun, 19 Jul 2026 16:59:55 +0000 Subject: [PATCH 1/3] style: remove positive letter spacing from Inter headings Co-authored-by: Sam Croswell --- .../src/global/styles/_letter-spacing.scss | 27 +++++++++++++++++++ libs/core/src/global/styles/app.scss | 1 + 2 files changed, 28 insertions(+) create mode 100644 libs/core/src/global/styles/_letter-spacing.scss diff --git a/libs/core/src/global/styles/_letter-spacing.scss b/libs/core/src/global/styles/_letter-spacing.scss new file mode 100644 index 000000000..930f7bf69 --- /dev/null +++ b/libs/core/src/global/styles/_letter-spacing.scss @@ -0,0 +1,27 @@ +//// +/// Letter spacing token overrides (Pine-internal until upstream consolidation) +/// +/// Removes the small positive tracking that `@kajabi-ui/styles` applies to +/// Inter headings, which reads as too spread out. Body copy is normalized to +/// `0`, and `h1`/`h2` adopt a tight tracking matching Tailwind's +/// `tracking-tight` (`-0.025em`); the remaining heading roles sit at `0`. +/// +/// Namespace contract: these CSS custom properties use the canonical +/// `--pine-letter-spacing*` namespace owned by `@kajabi-ui/styles`. Pine +/// redefines the values here (this file is `@use`d after the upstream import +/// in `app.scss`, so these win) without renaming the tokens. When upstream +/// publishes matching values, remove this file in the same PR that bumps the +/// `@kajabi-ui/styles` version. Component SCSS does not change. +/// +/// @group pine +//// + +:root { + --pine-letter-spacing: 0; // body/default (cleans up the invalid upstream value) + --pine-letter-spacing-heading-1: -0.025em; // Tailwind tracking-tight + --pine-letter-spacing-heading-2: -0.025em; // Tailwind tracking-tight + --pine-letter-spacing-heading-3: 0; + --pine-letter-spacing-heading-4: 0; + --pine-letter-spacing-heading-5: 0; + --pine-letter-spacing-heading-6: 0; +} diff --git a/libs/core/src/global/styles/app.scss b/libs/core/src/global/styles/app.scss index 7b3e4b2ff..c2e159418 100644 --- a/libs/core/src/global/styles/app.scss +++ b/libs/core/src/global/styles/app.scss @@ -1,4 +1,5 @@ @use '~@kajabi-ui/styles/dist/pine/pine'; +@use 'letter-spacing'; @use 'fonts'; @use 'motion'; @use '../../components/pds-combobox/pds-combobox-dropdown-panel.scss' as *; From edc5b07e45ca0c09207b00ee55f308c8428f9a84 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sun, 19 Jul 2026 17:27:55 +0000 Subject: [PATCH 2/3] docs(pds-text): clarify letter-spacing override comments --- libs/core/src/global/styles/_letter-spacing.scss | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libs/core/src/global/styles/_letter-spacing.scss b/libs/core/src/global/styles/_letter-spacing.scss index 930f7bf69..a89c91395 100644 --- a/libs/core/src/global/styles/_letter-spacing.scss +++ b/libs/core/src/global/styles/_letter-spacing.scss @@ -6,12 +6,19 @@ /// `0`, and `h1`/`h2` adopt a tight tracking matching Tailwind's /// `tracking-tight` (`-0.025em`); the remaining heading roles sit at `0`. /// +/// Note: `--pine-letter-spacing-heading-6` is also consumed as label tracking +/// by `pds-chip`, `pds-combobox`, and `pds-radio-group`, so zeroing it here +/// removes their `0.16px` label tracking too (intentional). Removing this file +/// would therefore restore positive tracking on those labels, not just headings. +/// /// Namespace contract: these CSS custom properties use the canonical /// `--pine-letter-spacing*` namespace owned by `@kajabi-ui/styles`. Pine /// redefines the values here (this file is `@use`d after the upstream import -/// in `app.scss`, so these win) without renaming the tokens. When upstream -/// publishes matching values, remove this file in the same PR that bumps the -/// `@kajabi-ui/styles` version. Component SCSS does not change. +/// in `app.scss`, so these win) without renaming the tokens. Remove this file +/// in the same PR that bumps `@kajabi-ui/styles` to a version that ships these +/// exact values (`--pine-letter-spacing: 0`, `--pine-letter-spacing-heading-1` +/// and `-2` at `-0.025em`, and `--pine-letter-spacing-heading-3` through `-6` +/// at `0`). Component SCSS does not change. /// /// @group pine //// From e06b89bd8e3208d6b0bf225cd0cdfcb5a9b684f1 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sun, 19 Jul 2026 18:06:46 +0000 Subject: [PATCH 3/3] chore: re-trigger CI