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

Better alerts when a game is found #2984

Closed
haifron opened this issue Jun 1, 2023 · 13 comments · Fixed by #3216
Closed

Better alerts when a game is found #2984

haifron opened this issue Jun 1, 2023 · 13 comments · Fixed by #3216

Comments

@haifron
Copy link

haifron commented Jun 1, 2023

Is your feature request related to a problem? Please describe.
I often leave the client searching for a game while doing something else & don't notice it has found a game until it is launching & I have seconds to react. This has sometimes been about half an hour after I started searching, and I've forgotten I'm searching so I'm suddenly warping into a game - the extra few seconds of warning would make a big difference.

Describe the solution you'd like
As an option, the FAF client becomes the top window when you find a match - e.g this behaviour appears in dota 2 when you find a game (I believe it's optional there too).

Describe alternatives you've considered
There've also been suggestions on the discord of a sound playing when you find a game which is another good option, though I wouldn't say it's a total replacement.

@BlackYps
Copy link
Collaborator

I like the sound without voice. Nice work! I assume the license is such that we can use it without attribution?

@BlackYps
Copy link
Collaborator

related to #1783

@warrenkc
Copy link

Yes, I give you permission to use it how you want. I got it from here: https://pixabay.com/sound-effects/biodynamic-impact-braam-tonal-dark-176441/

@DotNetSimon
Copy link
Contributor

DotNetSimon commented Jun 27, 2024

@warrenkc Hi, Thanks for this sound. I am trying to include it but it is quite loud and unexpected when the match starts. Do you think we can adjust it ?
I have also tried this alternative from PIxabay: https://pixabay.com/sound-effects/countdown-sound-effect-8-bit-151797/ which works better (I think).

I tried launching games with both sounds and even though I anticipated it, this game launch sound you suggested made my heart jump out of my chest. It's a little difficult to simulate perhaps, but assume the sound plays unexpected, and possibly louder than you thought :)

Maybe we should make another 'ping' or 'pwoop' type of notification sound instead?

@warrenkc
Copy link

warrenkc commented Jun 27, 2024

@DotNetSimon Would you like me to lower the volume of the audio file? Your idea for the countdown sound is good too!

I will try to get back to you with some more samples to try.

I used the loud and unexpected sound because oftentimes I will leave the room and want to be alerted if a match is found. I think I must be searching for matches when very few players are online so it takes a long time to find a match.

@DotNetSimon
Copy link
Contributor

DotNetSimon commented Jun 27, 2024

Yeah this happened to me many times, that's why I am working on it now :D

We could make it more fancy and do a dropdown for the sound to pick, or people can change it with the Theme already too.

@DotNetSimon
Copy link
Contributor

Ok I tried having this sound and I swear, I have had a heart attack when the game launched. I'm replacing it with the 8-bit beepbeepboop and we can iterate from there.
If you want to try, I can send you a copy of the version that has your sound 😬 but I expect you will 💀 also 😅

@warrenkc
Copy link

warrenkc commented Jun 27, 2024

You make me laugh. That is funny. I think it would probably be a good option to have a setting for the path of the audio file. Then if necessary a user can change the setting to point to any audio file.
Thanks for your awesome work!

I setup a python script for my own computer that would play the sound I created. It would play anytime Forged Alliance was launched. It was definitely attention grabbing.

Match found audio - lowered by 8db:
match-found-audio-lowered-8db.zip
Feel free to use it if you would like.

Match found audio - Original volume level.
match-found-audio.zip

@BlackYps
Copy link
Collaborator

Have a look at this: https://www.youtube.com/watch?v=3taTQYor7ZQ it's similar and I don't think it's too scary. If the impact sound effect (this is what I'm gonna call it now) from pixabay is too loud then we can easily adjust this. What I don't like about the countdown is that it is veeeery barebones and sounds unpleasant because it is such a sharp frequency. Also the concept of the countdown doesn't really make too much sense because we can't time it with the game actually launching.

@DotNetSimon
Copy link
Contributor

All fair comments. The countdown sound definitely doesn't fit with the other sound effects either so it's just there for a placeholder. I guess our choice could be to lower the volume, use a different notification sound (like some phone alarm) or keep the one suggested by Warren. Theme overrides allow players to change it however they feel like. (which is easier than adding selection as a feature).

@DotNetSimon
Copy link
Contributor

DotNetSimon commented Jun 27, 2024

Some alternatives that I found with a quick search 🤷
https://pixabay.com/sound-effects/choir-notification-ringtone-tone-e-major-95172/
https://pixabay.com/sound-effects/level-up-2-199574/
https://pixabay.com/sound-effects/success-fanfare-trumpets-6185/
https://pixabay.com/sound-effects/ringtone-bubbly-bubbles-188202/
https://pixabay.com/sound-effects/short-success-sound-glockenspiel-treasure-video-game-6346/
https://pixabay.com/sound-effects/echo-alert-177739/
https://pixabay.com/sound-effects/xylesizer-163606/
https://pixabay.com/sound-effects/bell-123742/

I'd love to have something like this: https://www.wowhead.com/sound=8960/readycheck but I don't think we would have rights for it :(

Honestly, I think about half the people will hate whatever sound we pick :p But they can disable it right, or replace it if they wish.
We just have to make sure we don't make too many people angry when they install the new version and suddenly get their ears blasted out by an alert (alternatively, we set the default to disabled?).

@DotNetSimon
Copy link
Contributor

ok!

@DotNetSimon
Copy link
Contributor

Yeah this works a lot better ❤️

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 a pull request may close this issue.

4 participants