Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(many): add new Typography tokens and update text and heading #1828

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

HerrTopi
Copy link
Contributor

Closes: INSTUI-4403

Copy link

github-actions bot commented Dec 13, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-12-18 12:11 UTC

Copy link
Collaborator

@matyasf matyasf left a comment

Choose a reason for hiding this comment

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

see my comments

packages/ui-heading/src/Heading/README.md Outdated Show resolved Hide resolved
packages/ui-heading/src/Heading/index.tsx Outdated Show resolved Hide resolved
packages/ui-text/src/Text/README.md Show resolved Hide resolved
packages/ui-text/src/Text/props.ts Show resolved Hide resolved
@HerrTopi HerrTopi force-pushed the typo branch 3 times, most recently from e0fcafc to 222ed9f Compare December 16, 2024 19:54
@HerrTopi HerrTopi requested review from matyasf and balzss and removed request for joyenjoyer December 16, 2024 20:17
Copy link
Collaborator

@matyasf matyasf 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, except some doc changes (see my comment)

packages/ui-text/src/Text/README.md Show resolved Hide resolved

## Legacy values

For compatibility reasons we still provide the legacy typography tokens (xxLarge, medium, etc.) so updating InstUI is easier but these tokens shouldn't be used when creating new screens.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please list here what is exactly deprecated. Also can we make some conversation table like

lineHeight 'lineHeightCondensed' --> 'lineHeight125'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is no official mapping, however, I added the deprecated list

@HerrTopi HerrTopi merged commit a09fa0f into master Dec 18, 2024
10 of 11 checks passed
@HerrTopi HerrTopi deleted the typo branch December 18, 2024 12:11
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.

3 participants