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 MusicUI #977

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added MusicUI #977

wants to merge 2 commits into from

Conversation

MichaelK-F
Copy link

Write a short description about your art piece, such as what your artwork represents and your experience drawing it!
This is a simple UI for a music app. The bars on the top are supposed to be a visualiser and the height of them randomly changes. The progress indicator is also placed at a random point. The experience of using the blot editor was overall enjoyable. It took me a while to understand how the polylines work, although being able to drag my mouse over the numbers to change them definitely helped.

  • I have read the steps to getting a blot
  • I have read all the submission requirements and follow them
  • 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
    • is drawable through the editor(does not use outside tools)
    • 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

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:

Lastly, link a picture of your art piece here:
snapshot0

Copy link

vercel bot commented Oct 30, 2024

@MichaelK-F 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 Oct 30, 2024

art/MusicUI-MMKF/index.js looks like art! preview it in the editor

@EnterpriseGoose
Copy link
Contributor

Hello! Sorry it's been so long. The main reviewer has been swamped lately, so I wanted to give you a few hints so you have the best chance of getting a blot.

I really like where you're going with this! It looks really cool. To get a blot, though, you'll need to make it a bit more complex. Generally, people spend ~15-20 hours on a project for it to be "blot-worthy", and I spent closer to 25-30.

Some suggestions I have are:

  • Could you add more buttons to the UI? Like skip buttons, etc.
  • Could you add a little image and name preview above the current UI with a randomly generated "currently playing song"? That would be a great way to add a bunch of complexity to your project!
  • It would be cool if you could use a for loop to generate the bars so that you could control how many bars there are

I'm really excited to see where this goes!

@MichaelK-F
Copy link
Author

I'm planning on working on this over the next few days now that exams are over, so please don't close the pr. Thanks!

Copy link

vercel bot commented Dec 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
blot ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 5, 2024 5:26am

@MichaelK-F
Copy link
Author

@EnterpriseGoose I made the changes you recommended, what do you think? Is there anything else I should change?

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.

2 participants