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

Update text styles #27318

Merged
merged 3 commits into from
Jan 23, 2025
Merged

Update text styles #27318

merged 3 commits into from
Jan 23, 2025

Conversation

salazarm
Copy link
Contributor

Summary & Motivation

Adds new text styles from #21450 which was reverted in #21517.

I'm adding the new styles, and updating a couple of the existing ones (Title and Heading are the main changes, they're bigger now). Assuming this is fine since we did this in the original PR.

How I Tested These Changes

briefly went around the app and everything looked fine. The only concrete changes are going to be title/headline are bigger now which seems like the intention in the original PR:

Screenshot 2025-01-23 at 1 14 38 PM Screenshot 2025-01-23 at 1 14 29 PM

Copy link

Deploy preview for dagit-storybook ready!

✅ Preview
https://dagit-storybook-1gk29nk6v-elementl.vercel.app
https://salazarm-update-text-styles.components-storybook.dagster-docs.io

Built with commit 69f8ca9.
This pull request is being automatically deployed with vercel-action

Copy link
Collaborator

@bengotow bengotow left a comment

Choose a reason for hiding this comment

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

LGTM!

@salazarm salazarm merged commit 768ef9c into master Jan 23, 2025
6 checks passed
@salazarm salazarm deleted the salazarm/update-text-styles branch January 23, 2025 19:01
@braunjj
Copy link
Contributor

braunjj commented Jan 23, 2025

thank you for doing this

salazarm added a commit that referenced this pull request Jan 29, 2025
## Summary & Motivation

Adds new text styles from
#21450 which was reverted in
#21517.

I'm adding the new styles, and updating a couple of the existing ones
(Title and Heading are the main changes, they're bigger now). Assuming
this is fine since we did this in the original PR.

## How I Tested These Changes

briefly went around the app and everything looked fine. The only
concrete changes are going to be title/headline are bigger now which
seems like the intention in the original PR:

<img width="281" alt="Screenshot 2025-01-23 at 1 14 38 PM"
src="https://github.com/user-attachments/assets/e039f856-a26d-45d1-91a1-bf0a10998199"
/>
<img width="366" alt="Screenshot 2025-01-23 at 1 14 29 PM"
src="https://github.com/user-attachments/assets/d3374088-7756-4098-afa2-6e78549c4b96"
/>
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