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 Linear-Gradient as a micro Project #915

Merged
merged 3 commits into from
Oct 15, 2023

Conversation

05hiteshbansal
Copy link
Contributor

@05hiteshbansal 05hiteshbansal commented Oct 13, 2023

Developer Checklist

  • Followed guidelines mentioned in the readme file.
  • Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
  • Starred ⭐ the Repo (Optional)

Summary

This project is a javascript based Product. It allows you to click a cell and make a random Linear Gradient at random degree with random colours . Also number of colours are also not specific so a user can generate linear-Gradient with any number of colours .

Screenshot

screenshort

@05hiteshbansal 05hiteshbansal changed the title Linear-Gradient Added Linear-Gradient as a micro Project Oct 13, 2023
@05hiteshbansal
Copy link
Contributor Author

@alvalle09 Please review my PR

@05hiteshbansal
Copy link
Contributor Author

@vclayton please review my PR

Copy link
Collaborator

@PBJI PBJI left a comment

Choose a reason for hiding this comment

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

hi @05hiteshbansal, please rename the folder to BackgroundGenerator/05hiteshbansal as this types of projects come under that category which pre-exists on the repo. Thank you.

@PBJI PBJI added the changeRequested The requested changes should be adhered to before further review or approval. label Oct 14, 2023
@05hiteshbansal
Copy link
Contributor Author

changes done @PBJI as requested

@05hiteshbansal 05hiteshbansal requested a review from PBJI October 14, 2023 19:17
@PBJI
Copy link
Collaborator

PBJI commented Oct 15, 2023

its should be 05hiteshbansal not hitesh_banshal in the folder name. @05hiteshbansal

@05hiteshbansal
Copy link
Contributor Author

@PBJI done as stated above

@PBJI PBJI merged commit 8c6820f into thinkswell:master Oct 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changeRequested The requested changes should be adhered to before further review or approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants