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

RetroTatooine-MahitMarri #834

Closed
wants to merge 11 commits into from
Closed

RetroTatooine-MahitMarri #834

wants to merge 11 commits into from

Conversation

MahitMarri
Copy link

@MahitMarri MahitMarri commented Aug 17, 2024

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).

Screenshot 2024-08-17 at 11 18 07 AM

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

@Ssmidge
Copy link
Member

Ssmidge commented Aug 26, 2024

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!

@MahitMarri
Copy link
Author

MahitMarri commented Aug 26, 2024 via email

Copy link

vercel bot commented Aug 26, 2024

@MahitMarri is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

github-actions bot commented Aug 26, 2024

art/Retro_Tatooine-MahitMarri/index.js looks like art! preview it in the editor

@MahitMarri
Copy link
Author

I fixed the misnamed files

@Ssmidge Ssmidge self-assigned this Aug 27, 2024
@Ssmidge
Copy link
Member

Ssmidge commented Aug 27, 2024

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!

@MahitMarri
Copy link
Author

MahitMarri commented Aug 27, 2024 via email

@MahitMarri
Copy link
Author

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

@Dongathan-Jong
Copy link
Collaborator

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.

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.

3 participants