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

Remove the chat sound when watching a replay #6646

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

Conversation

Garanas
Copy link
Member

@Garanas Garanas commented Feb 7, 2025

Description of the proposed changes

This refers to the sound when you receive a message. The sound plays in replays. I personally find it to be an annoying sound, but this is of course subjective.

Checklist

  • Changes are annotated, including comments where useful
  • Changes are documented in the changelog for the next game version

@Garanas Garanas added the ui: chat window related to chatting with players label Feb 7, 2025
@github-actions github-actions bot marked this pull request as draft February 7, 2025 15:49
@Garanas Garanas marked this pull request as ready for review February 7, 2025 16:00
Copy link
Contributor

@lL1l1 lL1l1 left a comment

Choose a reason for hiding this comment

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

The current implementation of the chat beep doesn't make sense so removing it is okay. There is a chat beep UI mod (ChatBeepLite) that has a more reasonable design (beeps only for ally chat/private chat messages). Maybe that could be a feature to implement with configuration options.

@lL1l1 lL1l1 added this to the Development Iteration II of 2025 milestone Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui: chat window related to chatting with players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants