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

Cherrymxbluesounds #90

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

ahmednofal
Copy link

This uses different sounds for the keyboard. It introduces the -c option to enable using cherrymxblue sounds, the folder of sounds is located in the root directory, and the path is included in the makefile same as the PATH_AUDIO. The entire keylayout uses the same sounds except for enter, space and shift. This pull request uses the -f option to use the 0x31 key for all keys.

…ds but the packaging process has not been modified to include the installation of the sounds in their appropriate location
@AlexanderWillner
Copy link
Contributor

Personally, I really like your suggested sound. However, not sure if most of the code changes are really needed. Isn't make clean ; PATH_AUDIO=$(pwd)/cherrymxbluesounds make && ./buckle -f 31 more flexible, i.e., by this many other sounds could be added?

AlexanderWillner added a commit to AlexanderWillner/bucklespring that referenced this pull request Jan 26, 2021
@Nojipiz
Copy link

Nojipiz commented Mar 13, 2022

I need this chanagessssss, please accept :(

@zevv
Copy link
Owner

zevv commented Mar 13, 2022

Hi; sorry for ignoring this for so long. I think development on bucklespring has kind of ceased, apart from the occasional bugfix.

There is already the command line option -p PATH for loading sounds from a different directory, so you can just put the cherry sounds somewhere on your machine and use them without any need for changing code or recompilation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants