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

Added cursor types #446

Merged
merged 4 commits into from
Oct 3, 2024
Merged

Added cursor types #446

merged 4 commits into from
Oct 3, 2024

Conversation

ZoeADavis
Copy link
Contributor

Issue #427

Added Auto, Default, crosshair, wait, and text cursor types.

Copy link
Member

@aayush105 aayush105 left a comment

Choose a reason for hiding this comment

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

@ZoeADavis can you add the screenshot of the output in the description section so that i can go through it and merge your PR.

@ZoeADavis
Copy link
Contributor Author

@aayush105
output

@aayush105
Copy link
Member

@aayush105 output

Can you just improvise your plain html text to some thing else using CSS or icons?
As the plain text doesn't look cool, so if you can please add the some sort of flex to the various hover cursor buttons, add icons in front of plain text, animated buttons. These are my suggestions so that you can improve your skill.

@ZoeADavis
Copy link
Contributor Author

@aayush105

image_2024-10-02_194737003

updated and added icons, flex, and transition animations when you over over it.

Copy link
Member

@aayush105 aayush105 left a comment

Choose a reason for hiding this comment

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

Great work @ZoeADavis

@aayush105 aayush105 merged commit df73661 into you-dont-need:master Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants