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

Add a sound when receiving chat messages #903

Open
EmmetSchuler opened this issue Aug 5, 2024 · 3 comments · May be fixed by #1466
Open

Add a sound when receiving chat messages #903

EmmetSchuler opened this issue Aug 5, 2024 · 3 comments · May be fixed by #1466
Labels
enhancement Improve existing feature good first issue Good for newcomers

Comments

@EmmetSchuler
Copy link

EmmetSchuler commented Aug 5, 2024

I often miss chat messages during or after games, because the small indication icon can be easily missed.

I think a small 'ding' sound would indicate a message nicely, without posing too much of a distraction during games.

It would make sense to disable these in Zen mode (when chat is already disabled)
and have the option to disable them in preferences too.

@tom-anders tom-anders added enhancement Improve existing feature good first issue Good for newcomers labels Dec 20, 2024
@tom-anders
Copy link
Collaborator

Does the Web version have a sound we can reuse here?

@EmmetSchuler
Copy link
Author

EmmetSchuler commented Dec 21, 2024

As far as I know there isn't a sound when receiving chat messages in game, but there is the lichess general notification sound you could possibly use (For example when a lichess streamer you follow goes live, etc).

@tom-anders
Copy link
Collaborator

@veloce any objections about this feature? Otherwise I'd add the "good first issue" tag

@vincendep vincendep linked a pull request Feb 20, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing feature good first issue Good for newcomers
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

2 participants