Skip to content

fix(design-tokens): replace deprecated Sass slash division with math.div#1127

Merged
danielfsousa merged 3 commits into
mainfrom
fix/sass-slash-div
May 27, 2026
Merged

fix(design-tokens): replace deprecated Sass slash division with math.div#1127
danielfsousa merged 3 commits into
mainfrom
fix/sass-slash-div

Conversation

@danielfsousa
Copy link
Copy Markdown
Contributor

@danielfsousa danielfsousa commented May 25, 2026

Summary

  • Replace deprecated / division operator with math.div() in generated SCSS rem() and em() functions, fixing Dart Sass 2.0 deprecation warnings for all downstream consumers
  • Remove unused yarn.lock — all CI workflows and scripts use npm
  • Update package-lock.json matchbox version

Closes #1126

Test plan

  • Run npm run build in matchbox and verify design-tokens SCSS output uses math.div()
  • Verify downstream projects importing design-tokens no longer see Sass deprecation warnings

Note

Low Risk
Template-only change to generated SCSS math; no runtime, auth, or data-path impact beyond Sass compatibility.

Overview
Updates the design-tokens SCSS generator so emitted rem() and em() helpers use math.div() instead of slash division, and adds @use "sass:math" in that output. This removes Dart Sass 2.0 deprecation warnings for consumers that import the generated styles.

Includes a patch changeset for @sparkpost/design-tokens and a matchbox version bump in package-lock.json (7.2.4 → 7.2.6).

Reviewed by Cursor Bugbot for commit 39aecc8. Bugbot is set up for automated code reviews on this repo. Configure here.

@danielfsousa danielfsousa changed the title fix(design-tokens): replace deprecated Sass slash division with math.div() fix(design-tokens): replace deprecated Sass slash division with math.div May 25, 2026
@danielfsousa danielfsousa self-assigned this May 25, 2026
@danielfsousa danielfsousa merged commit ab001bb into main May 27, 2026
6 of 7 checks passed
@danielfsousa danielfsousa deleted the fix/sass-slash-div branch May 27, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace deprecated sass division with math.div()

2 participants