Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

"Mute other Tabs" limited to active window #8384

Closed
jonathansampson opened this issue Apr 18, 2017 · 2 comments
Closed

"Mute other Tabs" limited to active window #8384

jonathansampson opened this issue Apr 18, 2017 · 2 comments

Comments

@jonathansampson
Copy link
Collaborator

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    The Mute other Tabs option only applies to tabs in the same window. This feature should, perhaps, apply to all tabs, across all windows. Or, there should be an additional option for universal mute.

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Desktop

  • Brave Version (revision SHA):
    632a9bc

  • Steps to reproduce:

    1. Open two Windows
    2. Play a YouTube video in each
    3. In the first window, select Mute other Tabs from the tab context menu
  • Actual result:
    Audio from the second window continues to play.

  • Expected result:
    All tabs, from all windows, will be muted.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

@jonathansampson jonathansampson added this to the 0.15.1 milestone Apr 18, 2017
@alexwykoff alexwykoff modified the milestones: 0.15.1, Backlog Apr 18, 2017
@bbondy
Copy link
Member

bbondy commented Apr 21, 2017

this should dispatch with an app action to the browser process and then handle it in tabs reducer across all windows.

@bbondy bbondy removed this from the Backlog milestone Oct 19, 2017
@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton
Copy link
Member

Closing in favor of brave/brave-browser#883

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants