-
-
Notifications
You must be signed in to change notification settings - Fork 130
[Feature - CSS] Feature/css animations #149
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
[Feature - CSS] Feature/css animations #149
Conversation
β Deploy Preview for quicksnip ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey thanks for your contribution, before we can move on with your snippets, could you please take a look at the comments I left.
Could you also improve the tags on your snippets, per our guidelines in CONTRIBUTING.md tags shouldn't be just the nam of the language, it would be better if you found an another tag instead of just animation
that could be unique for each animation
from{ | ||
width: 0% | ||
} | ||
f |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo
Hello, thank you for the review, I will fix the typo and add some unique tags later on today, about the blinking animation the cursor inherits the text's color, and because the text's default color is white it will be white ( one can add |
Also remove the |
Description
Includes 5 common CSS animations.
Type of Change
Checklist
Related Issues
Closes #148