jaramat Reporter
	Description • 9 years ago

Current keyboard shortcut for muting a tab is Alt+Shift+M. While that may be acceptable on Mac or even Linux, Windows and Windows programs rarely use Alt+Shift as a keyboard combo.

Proposed change: Change it to Ctrl+M. Ctrl+M isn't a system-wide shortcut nor is it used internally in Firefox, so it's available. It's also easier to memorize and easier to hit with one hand (Alt+Shift+M is nearly impossible to hit).

	Dão Gottwald [:dao]
	Comment 1 • 9 years ago

(In reply to jaramat from comment #0)
> Current keyboard shortcut for muting a tab is Alt+Shift+M. While that may be
> acceptable on Mac or even Linux, Windows and Windows programs rarely use
> Alt+Shift as a keyboard combo.

It's similarly odd on Linux as far as I can tell.

	(no longer active) Assignee
	Comment 2 • 9 years ago

If Ctrl+M is free, this is reasonable.  I'll write a patch.

	(no longer active) Assignee
	Comment 3 • 9 years ago

(Note that Cmd+M minimizes the window on OSX, but we can use Ctrl+M there.)

	(no longer active) Assignee
	Comment 4 • 9 years ago

Also adding a test since I realized that I didn't do that before for some reason.

	(no longer active) Assignee
	Comment 5 • 9 years ago

Ping?

	Jared Wein [:jaws] (please needinfo? me)
	Comment 6 • 9 years ago

Comment on attachment 8647196 [details] [diff] [review]
Switch the keyboard shortcut to toggle the mute state of a tab to Ctrl+M

Review of attachment 8647196 [details] [diff] [review]:
-----------------------------------------------------------------

Weird, I already reviewed this but must not have hit Publish. Thanks for the ping.

::: browser/base/content/test/general/browser_audioTabIcon.js
@@ +275,5 @@
> +  }
> +  function* test_on_browser(browser) {
> +    let tab = gBrowser.getTabForBrowser(browser);
> +
> +    // Make sure it's possible to mute before the tab is playing.

This is a neat feature and I could see myself maybe using this, but I don't see a way for people to find out about it. Out of curiosity, is there any way to access this feature outside of the keyboard shortcut?

	(no longer active) Assignee
	Comment 7 • 9 years ago

Not currently, no.  I've thought about it, and maybe we should add a context menu entry for muting a tab?

	(no longer active) Assignee
	Comment 8 • 9 years ago

(Note that users have asked for this too, see bug 1193848.)

	Pulsebot
	Comment 9 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/6930b12ee1d1

	(no longer active) Assignee
	Comment 10 • 9 years ago

Comment on attachment 8647196 [details] [diff] [review]
Switch the keyboard shortcut to toggle the mute state of a tab to Ctrl+M

Approval Request Comment
[Feature/regressing bug #]: Tab audio indicators
[User impact if declined]: The current keyboard shortcut is hard to use.
[Describe test coverage new/current, TreeHerder]: Has test.
[Risks and why]: Not risky at all
[String/UUID change made/needed]: This doesn't have a string change, but it affects a string shown in the UI (the keyboard shortcut string in the tooltip message.)  Therefore I'd like to take it on Aurora ASAP.  Thanks!

	Ryan VanderMeulen [:RyanVM]
	Comment 11 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/6930b12ee1d1

	Sylvestre Ledru [:Sylvestre]
	Comment 12 • 9 years ago

Comment on attachment 8647196 [details] [diff] [review]
Switch the keyboard shortcut to toggle the mute state of a tab to Ctrl+M

Let's take this patch. We want to polish this feature!

	Ryan VanderMeulen [:RyanVM]
	Comment 13 • 9 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/b44a8c68fc60

	Camelia Badau [:cbadau], Desktop QA
	Comment 14 • 9 years ago

Verified fixed using latest Aurora 42.0a2 (buildID: 20150917004025) and latest Nightly 43.0a1 (buildID: 20150917030229).