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

Conversation

chazdean
Copy link
Contributor

@chazdean chazdean commented Jul 12, 2022

#### WARNING ⚠️
https://polaris.shopify.com/components/text is not currently active

Do Not Merge!

WHY are these changes introduced?

Fixes #6537

WHAT is this pull request doing?

Add deprecation warnings to all of the following existing Text components:

  • TextStyle
  • DisplayText
  • Heading
  • Subheading
  • Caption
  • VisuallyHidden
Text component console.log
Screen Shot 2022-07-12 at 10 12 53 AM Screen Shot 2022-07-12 at 10 12 23 AM

@github-actions
Copy link
Contributor

github-actions bot commented Jul 12, 2022

size-limit report 📦

Path Size
polaris-react-cjs 209.22 KB (0%)
polaris-react-esm 135.63 KB (+0.01% 🔺)
polaris-react-esnext 191.19 KB (+0.01% 🔺)
polaris-react-css 41.72 KB (0%)

Copy link
Contributor

@sarahill sarahill left a comment

Choose a reason for hiding this comment

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

Looks good! 👏 Just some minor spelling suggestions.

Copy link
Contributor

@aveline aveline left a comment

Choose a reason for hiding this comment

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

Looks good overall! Just a few typos to fix up, including several instances of componant. 😁

We should also create a release branch to point it to rather than main. Maybe something like release-text-component. We can always rename it later.

@chazdean
Copy link
Contributor Author

chazdean commented Jul 13, 2022

@aveline

We should also create a release branch to point it to rather than main. Maybe something like release-text-component. We can always rename it later.

Agreed!

@alex-page alex-page force-pushed the main branch 7 times, most recently from a3a40bb to 0281d22 Compare July 14, 2022 05:31
@laurkim laurkim changed the base branch from main to deprecate-type-components July 15, 2022 12:40
Copy link
Contributor

@laurkim laurkim left a comment

Choose a reason for hiding this comment

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

let's gooo! 🚀 💯

I created a new deprecate-type-components branch and changed your pr to merge into that instead of main. I had one suggestion re: changeset content but it's not blocking.

@chazdean chazdean changed the base branch from deprecate-type-components to typography-beta-release August 24, 2022 14:09
@laurkim laurkim force-pushed the typography-beta-release branch 2 times, most recently from cd45e9e to f75fdd8 Compare August 31, 2022 18:56
@chazdean chazdean force-pushed the feat/add-deprecation-warnings-text-components branch from 40c2ecb to fdcc3e3 Compare September 2, 2022 18:31
@laurkim laurkim force-pushed the typography-beta-release branch from 29923e4 to 19e7c3d Compare September 14, 2022 19:25
@laurkim laurkim force-pushed the typography-beta-release branch 3 times, most recently from cfa8688 to 15b7276 Compare October 27, 2022 12:07
@laurkim laurkim removed the #gsd:29805 label Nov 2, 2022
@laurkim laurkim force-pushed the typography-beta-release branch from 15b7276 to a3710e2 Compare November 2, 2022 12:04
@laurkim laurkim force-pushed the feat/add-deprecation-warnings-text-components branch from 4f26dd1 to e360f94 Compare November 2, 2022 12:07
@laurkim laurkim merged commit 8b62db5 into typography-beta-release Nov 2, 2022
@laurkim laurkim deleted the feat/add-deprecation-warnings-text-components branch November 2, 2022 13:13
laurkim added a commit that referenced this pull request Nov 3, 2022
…omponents (#6605)

~~#### WARNING ⚠️~~
~~https://polaris.shopify.com/components/text is not currently active~~

~~Do Not Merge!~~

Fixes #6537 <!-- link to issue if one exists -->

<!--
  Context about the problem that’s being addressed.
-->

Add deprecation warnings to all of the following existing Text
components:
- TextStyle
- DisplayText
- Heading
- Subheading
- Caption
- VisuallyHidden

| Text component | `console.log` |
| --- | --- |
| <img width="870" alt="Screen Shot 2022-07-12 at 10 12 53 AM"
src="https://user-images.githubusercontent.com/59836805/178513139-742442e5-ccdc-4591-b0ad-b5ad0a19cbf0.png">
| <img width="633" alt="Screen Shot 2022-07-12 at 10 12 23 AM"
src="https://user-images.githubusercontent.com/59836805/178513188-95c3db0e-0536-4de8-8396-fc36cf876818.png">
|

Co-authored-by: Lo Kim <lo.kim@shopify.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Typography foundations] Deprecate existing typography components
5 participants