-
-
Notifications
You must be signed in to change notification settings - Fork 130
[Snippets] - add a few useful JS snippets #151
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
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.
Thanks for contributing, Could you please remove the javascript
and utility
tags of all of your snippets ? And take a look at the comment I left before we move on with your snippets
@Mathys-Gasnier I addressed everything you mentioned. Please let me know if there is anything else! |
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.
Everything looks good appart from a small little thing
@jjcantu Thanks for the contribution. These are really good code snippets. Awesome work 🙌 |
Description
This merge request introduces a couple common Javascript snippets. I also added a new category for color-manipulation.
Type of Change
Checklist
Related Issues
Closes #
Additional Context
Screenshots (Optional)
Click to view screenshots