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

Disable game lobby chat message audio when game is running #658

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

11EJDE11
Copy link
Contributor

@11EJDE11 11EJDE11 commented Feb 4, 2025

As discussed somewhere on Discord.

Copy link

github-actions bot commented Feb 4, 2025

Nightly build for this pull request:

@Metadorius
Copy link
Member

I am not sure that is a proper solution. You're trying to solve the problem of someone's spam coming through and repeatedly making sounds AFAIK, but you're also screwing up legit players chatting in the room when they, for example, didn't launch for some reason.

You should instead deal with the core issue, that is allowing to spam too many messages in a quick succession.

@11EJDE11
Copy link
Contributor Author

11EJDE11 commented Feb 4, 2025

Understood, yeah it is a rather pedestrian fix. I still think something like this could be wanted by some. The lag with IRC can be bad. There's also the case where two users get out of the game early and sit in the lobby talking; all remaining players in game can hear the beeps.

@Metadorius
Copy link
Member

There's also the case where two users get out of the game early and sit in the lobby talking;

Is this a frequent/valid case? Genuine question since all my client exposure is from playing with a group of friends and not in FFA, so such never happened to me.

@11EJDE11
Copy link
Contributor Author

11EJDE11 commented Feb 4, 2025

We normally do 3v3 and 4v4 with a friendly bunch of lads so it's more common for us I would imagine.

Also to your point earlier - If the game fails to launch (anti virus blocking it for instance), or they quit out on the loading screen, the code to enable the audio will still run. I can't think of any cases where it wouldn't be enabled again.

@Metadorius
Copy link
Member

How about a toggle in client options to control whether the chat sound is muted while in-game then?

@11EJDE11
Copy link
Contributor Author

11EJDE11 commented Feb 4, 2025

Sounds good to me. I'll look at into it when I get a mo.

Add UI constants to AudioOptionsPanel
@11EJDE11
Copy link
Contributor Author

11EJDE11 commented Feb 5, 2025

I've added a setting in. Figured while I was in there I should add some constants for control positioning - hope that's OK.

Before/After with these changes:
image

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.

3 participants