Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

volume-control: don't mute monitor sources #251

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

GranPC
Copy link
Contributor

@GranPC GranPC commented May 5, 2023

Middle-clicking the microphone indicator causes monitors to also be muted.

This is confusing because if an user is recording their screen while using the indicator to mute and un-mute their microphone, the screen recording will be silent as well.

Ask me how I know. :)

Middle-clicking the microphone indicator causes monitors to also be muted.

This is confusing because if an user is recording their screen while using
the indicator to mute and un-mute their microphone, the screen recording will
be silent as well.
@GranPC
Copy link
Contributor Author

GranPC commented May 5, 2023

Side-effect I just noticed: this also fixes compatibility with cadmus - whereas before muting and unmuting would cause the sound from your microphone to get fed back to the speakers and create a devilishly loud loop.

@GranPC
Copy link
Contributor Author

GranPC commented Jun 1, 2023

@lenemter could you check this one out please? Thanks!

Copy link
Member

@lenemter lenemter left a comment

Choose a reason for hiding this comment

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

Fixes the issue. Good work 🎉

@lenemter lenemter merged commit 1dc8a08 into elementary:master Jun 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants