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

Added lobby sounds for leave, join and option change #20050

Merged
merged 3 commits into from Aug 4, 2023

Conversation

Mailaender
Copy link
Member

Closes #6246

@Mailaender Mailaender force-pushed the lobby-sounds branch 2 times, most recently from 50498e1 to de90265 Compare June 3, 2022 19:45
Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

Were the sounds in the original games? All mods sound fine except for RA, it just feels like those sounds belong to TD, not RA

We should also play the leave sound for the person who's leaving as we play a sound on join. Currently the leave is only played for other players.

Imo skirmish / missions lobbies should play the sounds as well, the game would feel more responsive

@dragunoff
Copy link
Contributor

I like the idea of a doppler effect for join/leave sounds but I don't like the sound effects that are available in the original games. Especially the join/leave ones - I find them too intrusive. From @Smittytron's selection in #6246 I find the RA sounds most pleasing as they are subtle.

I'd suggest that for now we can keep it simple and use the same sound for leave, join and option change. Or maybe even don't change the sounds at all but just add the plumbing. Mods can then take advantage of it and in the future we may revisit this.

@Mailaender
Copy link
Member Author

Used the same sound for all notifications instead of the selection picked by @Smittytron. Made all lobby beeps optional, including the chat beep missing won't crash the game.

Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

There are no sounds for D2K and TS

We should also play the leave sound for the person who's leaving as we play a sound on join. Currently the leave is only played for other players

still stands

And for beeping sounds we should really use InterruptType: Overlap. It just sounds better. Though I guess for chat we can leave Interrupt so that after a lag spyke you wouldn't get a very loud beep

@Mailaender
Copy link
Member Author

There are no sounds for D2K and TS

No sounds for Tiberian Sun is intentional. I needed a test case and the proposed sounds were all from in-game.

We should also play the leave sound for the person who's leaving as we play a sound on join. Currently the leave is only played for other players

No idea how to solve that. If you leave the lobby, the lobby can't send you a beep.

@PunkPun
Copy link
Member

PunkPun commented Dec 27, 2022

We can just play the sound separately. Not via notifications

@Mailaender Mailaender force-pushed the lobby-sounds branch 2 times, most recently from 4b297d1 to d8d6381 Compare December 27, 2022 12:02
@Mailaender
Copy link
Member Author

Fixed the Dune 2000 sounds and changed the interrupt types. Added the player left sound when you press disconnect.

@penev92
Copy link
Member

penev92 commented Dec 27, 2022

Playing a leave sounds to the person that has already left the lobby sounds weird.

@PunkPun
Copy link
Member

PunkPun commented Dec 27, 2022

Currently it's silent when you join a singleplayer lobby, or when you leave any lobby. If we are adding sounds to entering and leaving then we should be consistent

@abcdefg30
Copy link
Member

I don't think we should play a sound for either oneself entering or leaving the lobby.

@PunkPun
Copy link
Member

PunkPun commented Dec 27, 2022

for me personally the absence of sound is offputting. Besides the fact that it would be inconsistent not to have it, in general I think it's really nice polish to have sound for entering major sequences

@matjaeck
Copy link

The usecase is to notify lobby members that a player-change in the lobby has happened. It is consistent to not play the sound for the person joining or leaving, as that person can not be considered a lobby member yet.

@PunkPun
Copy link
Member

PunkPun commented Jun 12, 2023

Needs a rebase, my request of making the join lobby sound consistently play still stands

@Mailaender
Copy link
Member Author

I am a bit confused as I seem to hear my own join/leave sounds.

@PunkPun
Copy link
Member

PunkPun commented Jun 14, 2023

Interesting, something must have changed. Now sounds for leaving are always played, however there's still no sound when entering skirmish

@Mailaender
Copy link
Member Author

Not for skirmish, but when you choose multiplayer you get the sounds. I am not really sure why, because I thought skirmish is just local multiplayer with yourself.

@PunkPun
Copy link
Member

PunkPun commented Jun 15, 2023

It's because when you enter skirmish "Player has joined the lobby" message is not sent

@PunkPun
Copy link
Member

PunkPun commented Jun 18, 2023

if (Type != ServerType.Local)
SendLocalizedMessage(Joined, Translation.Arguments("player", client.Name));

Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

#6246 (comment)

I'd be fine if this PR just added the tech but didn't change the sounds themselves

OpenRA.Game/Network/UnitOrders.cs Show resolved Hide resolved
@Mailaender
Copy link
Member Author

Removed sounds from the mods.

Copy link
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

Please add back the sounds. I meant keeping all the sounds as they are, that is the chat sound for everything

mods/common/languages/en.ftl Outdated Show resolved Hide resolved
@PunkPun PunkPun merged commit c234b4c into OpenRA:bleed Aug 4, 2023
3 checks passed
@PunkPun
Copy link
Member

PunkPun commented Aug 4, 2023

Changelog

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.

Different sounds for joining/leaving/messages ?
7 participants