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

Spotify Muting (part 2) #5

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

Conversation

Goldiedtv
Copy link

(I made a new pull request since I dug around and found a method to actually mute the application in Mixer, not just change the volume)

Added a feature to mute/unmute Spotify in the Windows Volume Mixer. I wasn't familiar on how this software worked, so I copied the same method as in the Pause/Unpause hotkey function, and just switched it to mute the Spotify Application.

It gets the PIDs of "Spotify" named processes and uses CSCore to find the session with the Spotify's PID in a specific Audio Device Source. After that it just mutes/unmutes the Spotify application in the Mixer. I also added a Dropdown menu where the user selects the right Audio Device where Spotify is playing, if it happens to be something else than the Default Device.

I tested it for some time and it seems to work fine. I added it only to SA since that's what I'm playing currently and could reliably test.

Also there was a typo when it saved the preferences to the config file and it didn't save the "Other" selection:
else if (radioButtonVolume.Checked) Should have been (?):
else if (radioButtonOther.Checked)

Goldiedtv added 3 commits May 17, 2023 19:56
…ng the volume of the app to 0 and unmuting by setting it to 100 (or whatever specified by user as the max). Tested only on SA, didn't add the functionality to III or VC. Yet.

Also fixed a typo where the playerSet config didn't get saved as "Other"
… this there were no need for the Maximum Volume setting and users can set the volume whatever and this won't screw with it.
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.

1 participant