Skip to content

No routing new setup with typography - #3568

Draft
Diaan wants to merge 71 commits into
mainfrom
no-routing-new-setup-with-typography
Draft

No routing new setup with typography#3568
Diaan wants to merge 71 commits into
mainfrom
no-routing-new-setup-with-typography

Conversation

@Diaan

@Diaan Diaan commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

This pull request introduces a new typography.css file for theme packages, centralizing and standardizing typography styles across themes. It updates the build and export process to include this new file, adjusts Storybook configuration to better support typography testing and theming, and makes related improvements to the workflow and theme loading logic.

Theme system improvements:

  • Added a new typography.css file to each theme package (e.g., packages/themes/core/typography.css) to define and centralize typography styles for headings, text, labels, captions, and more. This file is now included in the theme package exports and build outputs. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]
  • Updated the theme loading logic in .storybook/themes.ts and related build scripts to load theme.css and global.css only, removing direct loading of typography or mode-specific CSS files. [1] [2]

Storybook enhancements:

  • Added a comprehensive typography.stories.ts Storybook story to demonstrate and test all typography variants, including computed style inspection and font feature controls.
  • Enhanced Storybook global types and decorators to support user group and device simulation, and removed direct font settings from the preview head for better theme control. [1] [2] [3]

Workflow and build process updates:

  • Updated the Chromatic workflow to test all stories, not just changed ones, by setting onlyChanged: false.
  • Added a dark color scheme media query to the core theme’s global.css for improved color mode support.

These changes improve the maintainability and consistency of typography across themes, make it easier to test typography in Storybook, and ensure that theme packages are correctly exporting all necessary style files.

Diaan added 30 commits January 9, 2026 12:30
- Updated `build-themes.js` to include handling for deprecated themes.
- Introduced a new function to create file configurations for deprecated themes.
- Adjusted the theme filtering logic to accommodate deprecated tokens.
- Added a new workspace for `deprecated-tokens` with its own build scripts.
- Created `.tokensstudio.json` for deprecated tokens configuration.
- Updated `package.json` files to include scripts for importing deprecated tokens.
- Modified the main `package.json` to include the new workspace and import script.
…recated-project' of https://github.com/sl-design-system/components into chore/2873-remove-routing-from-token-setup-separate-deprecated-project
…ting-from-token-setup-separate-deprecated-project
… removed redundant code blocks and optimized existing functions.
…ting-from-token-setup-separate-deprecated-project
- Created developing.css with typography settings for the "developing" user group, including font sizes, line heights, letter spacing, and variable spaces.
- Created early.css with typography settings for the "early" user group, mirroring the structure of developing.css but with adjusted values for font sizes and spacing.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 08:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 09:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 09:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

- Adjusted regex for CSS variable names to allow uppercase letters.
- Modified border radius token matching to use the correct prefix and exclude certain variables.
- Enhanced typography mapping to use camelCase properties for better consistency.
Copilot AI review requested due to automatic review settings August 5, 2026 13:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 14:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 14:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 5, 2026 14:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 6, 2026 09:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Copilot AI review requested due to automatic review settings August 6, 2026 11:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants