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: Text Underline added #203

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

HydenLiu
Copy link
Contributor

@HydenLiu HydenLiu commented Jul 9, 2024

Hi, can I add this component? here's the result

Kapture 2024-07-09 at 21 23 43

I'm looking forward to your feedback and hope this can be a valuable contribution.

Copy link

vercel bot commented Jul 9, 2024

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

A member of the Team first needs to authorize it.

@dillionverma
Copy link
Collaborator

Hey @HydenLiu love what you did here! Would be really valuable addition.

If you can figure out a way to address this glitching when I hover in the middle of animation then i'll merge it in :)

Screen.Recording.2024-08-06.at.1.55.55.PM.mov

@HydenLiu
Copy link
Contributor Author

HydenLiu commented Aug 7, 2024

@dillionverma Thanks for the reply, I fixed this problem, hope I didn't misunderstand you!

@HydenLiu HydenLiu force-pushed the feat/text-underline branch from a48c629 to 81b676e Compare August 19, 2024 02:07
@alamenai
Copy link
Contributor

alamenai commented Sep 7, 2024

It's nice one @dillionverma , @HydenLiu , is there a way to choose between underline or highlighting (background)?

@itsarghyadas
Copy link
Collaborator

Hey @HydenLiu, the glitch hasn’t been fixed - it’s still the same as Dillion shows in the video. When I hover quickly, it creates a very bad and glitchy effect.

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.

4 participants