-
Notifications
You must be signed in to change notification settings - Fork 517
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
RetroTatooine-MahitMarri #834
Conversation
These are pictures of the different outputs of my program
Hey! Your art seems cool, but for it to be reviewed, it has to follow the naming scheme! |
Thanks for letting me know, if I make these changes later today will it get
approved?
…On Mon, Aug 26, 2024 at 11:48 AM Adrian T ***@***.***> wrote:
Hey! Your art seems cool, but for it to be reviewed, it has to follow the
naming scheme!
Your blot JavaScript file is named project.js instead of index.js, and
your snapshots folder isn't fully lowercase, without your files being
specifically named like this, your project won't show up in the gallery
correctly :(
Have a look at the guide here
<https://github.com/hackclub/blot/blob/main/docs/GET_A_BLOT.md#20-add-metadata-to-your-indexjs-file>
!
—
Reply to this email directly, view it on GitHub
<#834 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/A7QBX6SFU2E24TPB4LM24WDZTN2ANAVCNFSM6AAAAABMVVDJGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQHA2DQNRWG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@MahitMarri is attempting to deploy a commit to the Hack Club Team on Vercel. A member of the Team first needs to authorize it. |
|
I fixed the misnamed files |
Thank you! |
Thanks for the tip. I’ll see what I can do
…On Tue, Aug 27, 2024 at 7:06 AM Adrian T ***@***.***> wrote:
Thank you!
The two suns seem to collide with eachother sometimes, and even go outside
of the drawing canvas, the blot can't do that!
I think you can take this even further, you should try to push the blot to
it's limits (without breaking it :D), there's lots of unused space that you
can use to add details!
—
Reply to this email directly, view it on GitHub
<#834 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/A7QBX6XVBWPL4FH4FEW7RCLZTSBWZAVCNFSM6AAAAABMVVDJGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJSGY3DSOBWG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I fixed the problem with the suns overlapping and sometimes leaving the boundary. Ill see what i can do to make the empty space fill up |
Keep working at this! Please create a new PR once you have updated your art. I closing this due to a new management system for the PRs. |
Write a short description about your art piece, such as what your artwork represents and your experience drawing it!
I had no idea what to make at first and I only new how to make straight lines. I made a bunch of those lines in a way so that it made a curve. I did the same for the other side and since im a star wars guy i decided to add two suns. It was supposed to mimic the sky Luke Skywalker looked at in A New Hope. I spent a long time trying to add color but couldn't figure it out in the end, and some other things I spent a while trying to make work was the variables and loops for the placement of the lines (i'm mainly a html/css guy).
I have read the steps to getting a blot
I am submitting art that...
is algorithmically generated (will meaningfully change each time the program is run)
is your own work that is not copied from somewhere else
doesn't call Math.random() (See the documentation on randomness)
is drawable on Blot itself (doesn't have lines overlap more than 5 times)
Doesn't overlap lines more than 5 times
Fits in the working area of 125x125mm
Doesn't rely on colour to look good