Skip to content

sound@cinnamon.org: add a key binding to switch active audio output device - #13978

Open
rostidev wants to merge 1 commit into
linuxmint:masterfrom
rostidev:quick-switch-output-device
Open

sound@cinnamon.org: add a key binding to switch active audio output device#13978
rostidev wants to merge 1 commit into
linuxmint:masterfrom
rostidev:quick-switch-output-device

Conversation

@rostidev

@rostidev rostidev commented Sep 6, 2026

Copy link
Copy Markdown

Demonstration of this new functionality is in the attached video below. Suggest to watch with sound turned on.

20260617_105708_compressed.mp4

Almost the same change is tested by me on Fedora 44 with Cinnamon 6.6.7. Unfortunately the 6.6-maintenance branch doesn't accept new features anymore and @leigh123linux suggested to make this PR against the master brunch.

Regarding the default <Alt><Super>Tab keyboard shortcut, I chose this one as the default for switching the active audio output device by one hand only and by something similar to the application switching, i.e. similar to <Alt>Tab.

P.S. Link to the YouTube video in the demonstration for those who liked it: www.youtube.com/watch?v=ky2rtCpbn7k

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Best-practices scanner

This is a regex-based check for API usage that can pose security, performance or
maintainability issues, or that may already be provided by Cinnamon. Most findings
are advisory and do not automatically disqualify a pull request.

This check is not perfect and will not replace a normal review.


Found 1 potential issue(s):

⚠️ WARNING

⚠️ lang_bind

files/usr/share/cinnamon/applets/sound@cinnamon.org/applet.js:1116

Main.keybindingManager.addXletHotKey(this, "sound-cycle-output", this.cycleOutputKey, Lang.bind(this, this._cycleOutputDevice));

Lang.bind() is deprecated. Use arrow functions (() => {}) or Function.prototype.bind() instead.


Automated pattern check.

@rostidev
rostidev force-pushed the quick-switch-output-device branch from 7f1042c to 5c624e4 Compare September 6, 2026 13:35
@rostidev

rostidev commented Sep 6, 2026

Copy link
Copy Markdown
Author

The above warning about use of the deprecated Lang.bind() is resolved in the amended commit.

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