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

Speaker button in tabs should mute or play audio for current tab #2025

Closed
bsclifton opened this issue Nov 7, 2018 · 29 comments · Fixed by brave/brave-core#2861
Closed

Speaker button in tabs should mute or play audio for current tab #2025

bsclifton opened this issue Nov 7, 2018 · 29 comments · Fixed by brave/brave-core#2861

Comments

@bsclifton
Copy link
Member

bsclifton commented Nov 7, 2018

Description

When a site is playing audio, the tab that the audio is playing in will show a small speaker icon. You should be able to click this to mute the tab. Unfortunately, that is not working on DEV and BETA channels (BETA worked with 0.56.7, RELEASE works fine). If you right click and choose Mute from the context menu, it does work

Test plan / Steps to Reproduce

  1. Visit a site that plays audio (ex: http://beirutnights.com/)
  2. Ensure audio is playing and that speaker icon shows for the tab
  3. Try clicking the speaker icon and notice that nothing happens

Actual result:

Speaker button does absolutely nothing

Expected result:

Speaker button should toggle (showing a speaker with a cross through it)

Reproduces how often:

100%

Brave version (brave://version info)

Brave 0.57.4 Chromium: 71.0.3578.31 (Official Build) dev (64-bit)
Revision c88fdf2a4ce19a713615ca4fbde7a0d0b5fe2363-refs/branch-heads/3578@{#427}
OS Mac OS X
Brave 0.57.6 Chromium: 71.0.3578.31 (Official Build) beta(64-bit)
Revision c88fdf2a4ce19a713615ca4fbde7a0d0b5fe2363-refs/branch-heads/3578@{#427}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    ONLY reproduces on DEV and BETA. RELEASE work fine
@bsclifton bsclifton added priority/P3 The next thing for us to work on. It'll ride the trains. audio labels Nov 7, 2018
@simonhong
Copy link
Member

This feature was removed from upstream.
https://chromium-review.googlesource.com/c/chromium/src/+/1241813

@bsclifton
Copy link
Member Author

🙀

@stevenchu
Copy link

I dunno why I can't reply to this topic on community.brave.com so will reply here.

This is still happening, and I did not get an audio icon either.

I just had ROBOTIZED NARRATOR VOICES blast at 100% volume in my apartment with no way to shut it off.

No mute tab, no mute site. No clicking on the spam CNET video to block or disable it.

brave has no mute button for annoying sites

@srirambv
Copy link
Contributor

srirambv commented Nov 10, 2018

@stevenchu You are on the older muon browser which is deprecated. Request you to update to the new Brave browser.

@bbondy bbondy added this to the 1.x Backlog milestone Nov 13, 2018
@cndouglas
Copy link
Contributor

+1 from #2145.

@ghost
Copy link

ghost commented Dec 7, 2018

It now in release. I really need this functionality. I can't revert the update?

@eljuno
Copy link

eljuno commented Dec 8, 2018

+1 from Community https://community.brave.com/t/mute-tab-button/39275?u=eljuno

MacOS, Brave 0.57.18

@WorldLanguages
Copy link

+1 from me

@rebron rebron added the design A design change, especially one which needs input from the design team label Dec 11, 2018
@eljuno
Copy link

eljuno commented Dec 13, 2018

@eljuno
Copy link

eljuno commented Dec 27, 2018

@rebron rebron removed the design A design change, especially one which needs input from the design team label Jan 30, 2019
@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@simonhong
Copy link
Member

@CrisBD The fix(brave/brave-core#2861) is in review :)

@webjive
Copy link

webjive commented Jul 12, 2019

Can confirm. Just installed Brave and the Mute/Unmute on the tab doesn't work. Had to install an extension to control this for now.

@bsclifton
Copy link
Member Author

Code merged - our next Nightly build will have this fixed 😄👍

@srirambv
Copy link
Contributor

srirambv commented Aug 29, 2019

Verification passed on

Brave 0.69.119 Chromium: 76.0.3809.132 (Official Build) beta (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Linux

Verification passed on

Brave 0.69.119 Chromium: 76.0.3809.132 (Official Build) beta (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Windows 7 Service Pack 1 (Build 7601.24494)
  • Verified click on speaker icon mutes the audio from a regular tab
  • Verified click on muted speaker icon unmutes the audio from a regular tab
  • Verified click on speaker icon mutes the audio from a pinned tab
  • Verified click on muted speaker icon unmutes the audio from a pinned tab

Verified passed with

Brave 0.69.123 Chromium: 76.0.3809.132 (Official Build) beta (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Mac OS X
  • Verified click on speaker icon mutes the audio from a regular, private, and Tor tab
  • Verified click on muted speaker icon unmutes the audio from a regular, private and Tor tab
  • Verified click on speaker icon mutes the audio from a pinned tab
  • Verified click on muted speaker icon unmutes the audio from a pinned tab

@rebron rebron changed the title Clicking the speaker icon on a tab playing audio does nothing speaker button in tabs should mute or play audio for current tab Oct 2, 2019
@rebron rebron changed the title speaker button in tabs should mute or play audio for current tab Speaker button in tabs should mute or play audio for current tab Oct 2, 2019
@NotAmaan
Copy link

NotAmaan commented Oct 8, 2019

Can this be disabled with a flag or something? I usually have some YouTube music stream playing while I work and tend to Pin that tab. Now I can never switch to the pinned tab playing audio without muting it once or twice.

@webjive
Copy link

webjive commented Oct 8, 2019

There is a new Chrome flag (can't remember the site) out that when enabled puts a play/stop button for sound on any running background tab. Click it and it shows the tab name and stops playback.

@simonhong
Copy link
Member

@NotAmaan @webjive hmm, we don't have a flag for disable it now.
@rebron how should we handle this? My idea is don't show speaker button if tab width is small or pinned.

@webjive
Copy link

webjive commented Oct 8, 2019

The Chrome flag worked for me.
Screen Shot 2019-10-08 at 6 41 44 AM

@bsclifton
Copy link
Member Author

Good call on hiding for pinned tab - I created a new issue to track this:
#6358

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment