-
-
Notifications
You must be signed in to change notification settings - Fork 579
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
Call sound should also be disabled #209
Comments
What's your actual use case of this? Do you want to disable all the sounds or allow to have notification sounds but mute ringing only? |
Disable all sounds, especially because the ringtone they chose is obnoxious. |
My current idea is to use |
@IssueHunt has funded $50.00 to this issue.
|
@piomar123 Wouldn't that disable all audio from the app? Including videos that you might want to play from a conversation. It would be cumbersome to toggle the option in order to watch a video with sound. |
@whitecrownclown Yes, it disables all audio. I wrote that in my comment up above - it should automatically detect playing videos and doing calls if this way of muting audio is chosen. |
I think that doesn’t work if you’re watching a video and someone calls. It’s best to locate and deactivate the sound (block the HTTP or break the Web Audio API used) |
The question is what's easier - detecting incoming call and muting vs. blocking HTTP requests |
My pull request, as @fregante said, it will block the HTTP request for the call sound (there is a content-md5 header, in that way it find the sound), I also added an option to enable/disable the feature. Due HTTP cache, for example, if the ringtone plays and now you disable it, the sound is already cached so you will need to restart the app. |
@sindresorhus has rewarded $45.00 to @lukks. See it on IssueHunt
|
The Sound is disabled in the settings, but I still hear the sound when I receive a call.
Calls also seem to be bypass the Desktop Notification setting.
Edit: This is not a Caprine bug but a Messenger "feature". Perhaps Caprine can fix it.
IssueHunt Summary
Backers (Total: $50.00)
Submitted pull Requests
Tips
IssueHunt has been backed by the following sponsors. Become a sponsor
The text was updated successfully, but these errors were encountered: