Skip to content

chore: update line height in ScrollBasedVelocityDemo and TypingAnimation components (#882)#906

Merged
Yeom-JinHo merged 1 commit intomagicuidesign:mainfrom
1chooo:chore/tailwindcss-name-standardizing
Mar 2, 2026
Merged

chore: update line height in ScrollBasedVelocityDemo and TypingAnimation components (#882)#906
Yeom-JinHo merged 1 commit intomagicuidesign:mainfrom
1chooo:chore/tailwindcss-name-standardizing

Conversation

@1chooo
Copy link
Contributor

@1chooo 1chooo commented Mar 2, 2026

This pull request standardizes the usage of Tailwind CSS line-height classes for large headings and animated text components. The main change is replacing the custom leading-[5rem] class with the default leading-20 class across all relevant files and registry entries, ensuring consistency and better maintainability in the UI.

UI Consistency Improvements

  • Updated all instances of md:leading-[5rem] and leading-[5rem] to use the default Tailwind class md:leading-20 and leading-20 in ScrollVelocityContainer and TypingAnimation components (apps/www/registry/example/scroll-based-velocity-demo.tsx, apps/www/registry/magicui/typing-animation.tsx, [1] [2] [3] [4].
  • Synced registry JSON entries to use the updated line-height classes for both the scroll velocity demo and typing animation UI components (apps/www/public/r/scroll-based-velocity-demo.json, apps/www/public/r/typing-animation.json, [1] [2].

Thanks @Yeom-JinHo for your kind review!

@vercel
Copy link

vercel bot commented Mar 2, 2026

@1chooo is attempting to deploy a commit to the product-studio Team on Vercel.

A member of the Team first needs to authorize it.

…ion components (magicuidesign#882)

- closed: magicuidesign#882

Co-authored-by: Lin Hugo <hugo970217@gmail.com>
@Yeom-JinHo Yeom-JinHo force-pushed the chore/tailwindcss-name-standardizing branch from a88d192 to 2d56fd0 Compare March 2, 2026 22:39
Copy link
Collaborator

@Yeom-JinHo Yeom-JinHo left a comment

Choose a reason for hiding this comment

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

@1chooo
Thank you for your contribution !

There is a merge conflict, so i will rebase first and then merge.

@Yeom-JinHo Yeom-JinHo merged commit f69a187 into magicuidesign:main Mar 2, 2026
1 check failed
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.

Tailwind warning for leading-[5rem] in TypingAnimation component

2 participants