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

button sounds #236

Merged
merged 4 commits into from
Sep 19, 2022
Merged

button sounds #236

merged 4 commits into from
Sep 19, 2022

Conversation

DRuppFv
Copy link
Contributor

@DRuppFv DRuppFv commented Aug 21, 2022

Note: I removed a lot of code that was not done

Just need help with playing the audio, the audio files are in the PR

@DRuppFv
Copy link
Contributor Author

DRuppFv commented Sep 1, 2022

I believe it is done.
I just need someone to test it (my computer is not making sounds) and tell me if the sound is too loud or low.

@64kramsystem
Copy link
Member

They sound ok to me. I would (personal taste) increase the volume a bit, but it's fine as now.

When I click on "Start Game", I don't hear anything; I don't know if this is intended or not (also on Quit, but that's more likely to be intended).

@64kramsystem 64kramsystem self-requested a review September 2, 2022 19:52
Copy link
Member

@64kramsystem 64kramsystem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested; notes reported in the PR discussion.

@DRuppFv
Copy link
Contributor Author

DRuppFv commented Sep 2, 2022

They sound ok to me. I would (personal taste) increase the volume a bit, but it's fine as now.

When I click on "Start Game", I don't hear anything; I don't know if this is intended or not (also on Quit, but that's more likely to be intended).

The quit thing can be easily fixed. Just don't close the game immediately. About the start button, it should just play a
different sound, and i don't know the reason. For now, i'll try to fix my computer before updating this PR

@DRuppFv
Copy link
Contributor Author

DRuppFv commented Sep 2, 2022

@64kramsystem
I fixed my computer. The "start game" plays the sound! It's just a different one, if you don't like it, i can change.

About the quit game button... To make this work, it would be necessary to OR put audio code in the "exit" button code OR put "exit" button code in the audio code, and i don't think it would be that cool.

removing random println

button clicking sounds update

cargo fmt & cargo clippy

cargo fmt

new audio at start game button
@DRuppFv DRuppFv marked this pull request as ready for review September 2, 2022 22:21
@64kramsystem
Copy link
Member

I fixed my computer. The "start game" plays the sound! It's just a different one, if you don't like it, i can change.

Ah! I've noticed it. I'm neutral to whatever is used, however, it's hard to notice.

@DRuppFv
Copy link
Contributor Author

DRuppFv commented Sep 3, 2022

I fixed my computer. The "start game" plays the sound! It's just a different one, if you don't like it, i can change.

Ah! I've noticed it. I'm neutral to whatever is used, however, it's hard to notice.

It's a "normal" sound of "Start Game" button, you can see something like this in a lot of games like Punchy, but you are right. It would be better if we had a loading screen.

And i just put a new one btw

@zicklag zicklag linked an issue Sep 7, 2022 that may be closed by this pull request
Copy link
Collaborator

@odecay odecay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like this is almost ready, could you clean up the println?

src/audio.rs Outdated Show resolved Hide resolved
@odecay
Copy link
Collaborator

odecay commented Sep 19, 2022

bors merge

@bors
Copy link
Contributor

bors bot commented Sep 19, 2022

Build succeeded:

@bors bors bot merged commit c8ee96f into fishfolk:master Sep 19, 2022
@DRuppFv DRuppFv deleted the button_sound branch September 27, 2022 15:03
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.

Menu Sound Effects
3 participants