Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.

Conversation

lgriffee
Copy link
Contributor

@lgriffee lgriffee commented Apr 1, 2024

WHY are these changes introduced?

Part of https://github.com/Shopify/polaris-internal/issues/1529

WHAT is this pull request doing?

Adds back the Text heading2xl variant since it will be kept in the next major version for mobile purposes.

@lgriffee lgriffee self-assigned this Apr 1, 2024
@lgriffee lgriffee added the #gsd:40131 Polaris Responsive Text Styles label Apr 1, 2024
@lgriffee lgriffee changed the title [Text] Bring back heading2xl for mobile use-cases [Text] Bring back heading2xl and associated tokens for mobile use-cases Apr 1, 2024
@lgriffee
Copy link
Contributor Author

lgriffee commented Apr 1, 2024

Chromatic changes LGTM! There are a few other changes sneaking in from other updates on the v13 branch but all look good!

@lgriffee lgriffee marked this pull request as ready for review April 1, 2024 22:33
lgriffee added a commit that referenced this pull request Apr 2, 2024
…ion (#11813)

### WHY are these changes introduced?

Part of https://github.com/Shopify/polaris-internal/issues/1529

### WHAT is this pull request doing?
This PR updates the migration for deprecated `font` custom properties
based on the undeprecation of heading2xl.

#### v14-styles-replace-custom-property-font
| Deprecated CSS Custom Property | Replacement Value |
| -- | -- | 
| ~`--p-font-size-750`~  | ~`--p-font-size-600`~ | 
| `--p-font-size-800`  | ~`--p-font-size-600`~<br>`--p-font-size-750` | 
| `--p-font-size-900` | ~`--p-font-size-600`~<br>`--p-font-size-750` | 
| `--p-font-size-1000` | ~`--p-font-size-600`~<br>`--p-font-size-750` | 
| ~`--p-font-letter-spacing-denser`~ | ~`--p-font-letter-spacing-dense`~
|
| `--p-font-letter-spacing-densest` |
~`--p-font-letter-spacing-dense`~<br>`--p-font-letter-spacing-denser` |
| ~`--p-font-line-height-1000`~ | ~`--p-font-line-height-800`~ | 
| `--p-font-line-height-1200` |
~`--p-font-line-height-800`~<br>`--p-font-line-height-1000` |
|`--p-text-heading-3xl-font-family`|
~`--p-text-heading-xl-font-family`~<br>`--p-text-heading-2xl-font-family`|
|`--p-text-heading-3xl-font-size`|
~`--p-text-heading-xl-font-size`~<br>`--p-text-heading-2xl-font-size`|
| `--p-text-heading-3xl-font-weight`|
~`--p-text-heading-xl-font-weight`~<br>`--p-text-heading-2xl-font-weight`|
|`--p-text-heading-3xl-font-letter-spacing`|
~`--p-text-heading-xl-font-letter-spacing`~<br>`--p-text-heading-2xl-font-letter-spacing`|
| `--p-text-heading-3xl-font-line-height`|
~`--p-text-heading-xl-font-line-height`~<br>`--p-text-heading-2xl-font-line-height`|
|~`--p-text-heading-2xl-font-family`~|
~`--p-text-heading-xl-font-family`~|
| ~`--p-text-heading-2xl-font-size`~| ~`--p-text-heading-xl-font-size`~|
| ~`--p-text-heading-2xl-font-weight`~|
~`--p-text-heading-xl-font-weight`~|
| ~`--p-text-heading-2xl-font-letter-spacing`~|
~`--p-text-heading-xl-font-letter-spacing`~|
|~`--p-text-heading-2xl-font-line-height`~|
~`--p-text-heading-xl-font-line-height`~|

>[!NOTE]
>Major version upgrade guidance is updated [on a separate PR on the v13
branch](#11815).
@lgriffee lgriffee merged commit b19986c into v13.0.0 Apr 2, 2024
@lgriffee lgriffee deleted the text-deprecation-updates branch April 2, 2024 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
#gsd:40131 Polaris Responsive Text Styles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants