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

Gurgle sound (quack) #924

Open
erlend-sh opened this issue Mar 13, 2024 · 4 comments
Open

Gurgle sound (quack) #924

erlend-sh opened this issue Mar 13, 2024 · 4 comments
Labels
good first issue Good for newcomers scope:small A small and well-defined task

Comments

@zicklag zicklag added good first issue Good for newcomers scope:small A small and well-defined task labels Apr 4, 2024
@MaxCWhitehead
Copy link
Collaborator

One of the steps here will be adding a new button for triggering the glug sound.

This change is an example of how to add a new button and hooking it up: 2f684e6

@safa-amin
Copy link

Hello! I am writing on behalf of a team of software engineering students from Carnegie Mellon University in Qatar. For our Software Engineering class (17-313), we are tasked with contributing to an open-source project, and we are very excited about the possibility of contributing to Jumpy! We would be grateful if you could let us know whether our team can contribute to your project and, if so, how we can get started. Please feel free to reach out to me at [email protected] or +974 55041045 if you’d like to discuss this further. We would really appreciate if you could get back to us. Thank you :))

@erlend-sh
Copy link
Member Author

erlend-sh commented Dec 3, 2024

Sure! The basic recommendations for getting started are the same as here: #986 (comment)

Looking at how the player jump sound is setup would be a good reference for how audio is triggered.

See if you can find where the jump sound exists in our codebase; change it to a different sound to confirm. If you can’t locate it just ask us.

For the actual feature proposed here, two changes are required:

  • a new button action; see Max’ comment above.
  • that button action should trigger a sound; I suggest using this one since it doesn’t require any further audio editing.

We prefer keeping communications in the open, I.e. here on GitHub or on Discord.

@MaxCWhitehead
Copy link
Collaborator

On the note of adding a button (new key to trigger some action) - as can be seen in the comment @erlend-sh linked, it's a bit involved. The slide button is currently not actually used - referencing that diff, could also re-name this button to glug or gurgle and then add code to trigger sound based on that key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers scope:small A small and well-defined task
Projects
None yet
Development

No branches or pull requests

4 participants