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 one piano keyboard scale #4

Closed

Conversation

guastallaigor
Copy link

Note: I did this only for fun. Accept this only if you really liked this feature, because I had to change some things a bid.

The major changes I did were for the keyboard in the computer could look like a piano keyboard (key positioning). To do that, I needed changed the keyboard keys for the Bongos and Cymbal mainly.

So now the new keys are:
Bongo left: -
Bongo right: =
Cymbal: *
Meow: Space (unchanged)
Piano: ZSXDCVGBHNJM,
Marimba: QW...P (unchanged)

Now "C" is the keyboard letter "Z", and "C#" is the keyboard letter "S", and so on, like a normal piano keyboard.

All the sounds were download at Freesound, so no copyright infringements here.

I also updated the instructions.

…iano to Z...,. Added more keyboard sounds. Moved sounds to a keyboard-olds folder. Missing b note and #'s.
@guastallaigor guastallaigor changed the title Added one keyboard full scale Added one piano keyboard scale Oct 24, 2018
@c4l1c0
Copy link

c4l1c0 commented Oct 26, 2018

why not do the same thing for marimba? q2w3er5t6y7u and so on.
if he merge this i will do the marimba.

@RafaelCamarda
Copy link

Amazing man, thx!

@Externalizable
Copy link
Owner

Sorry about the long wait! I'm doing some early spring cleaning and felt the need to finally address this Pull Request, which has accumulated a lot of merge conflicts over the years, primarily because of 101823e. That's my bad.

This issue outlines a better solution as I don't want to move the bongo keys around anymore: #135

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.

4 participants