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

Commands are sent to all open YouTube players when configured not to #845

Open
ConorJS opened this issue Jan 17, 2022 · 13 comments
Open

Commands are sent to all open YouTube players when configured not to #845

ConorJS opened this issue Jan 17, 2022 · 13 comments

Comments

@ConorJS
Copy link

ConorJS commented Jan 17, 2022

Regardless of whether 'General' -> 'General Settings' -> 'Send commands to most recent player tab only.' is set or not, all YouTube players across all Chrome windows pause/unpause when I hit the play/pause hotkey. This has worked for years, and randomly broke a few days ago.

Chrome version: 97.0.4692.71 (Official Build) (64-bit)

It's possible I was running 96.0.4664 until I restarted Chrome, and then started noticing the issue. I hadn't restarted Chrome for a while until recently, so it could be the newer version causing this.

@Timums
Copy link

Timums commented Jan 22, 2022

I have the exact same problem. I'm on Opera 83.0.4254.19 (64bit) and it stopped working the same day as ConorJS. Likewise, I also updated my version before the problem occurred. I can also replicate the problem in Chrome 97.0.4692.99 (Offizieller Build) (64-Bit).
Any help would be appreciated.

@ZornTaov
Copy link
Contributor

After a bit of digging, it looks like tabStates[tab.id].state.isPlaying at https://github.com/berrberr/streamkeys/blob/master/code/js/background.js#L91 always returns true for youtube videos. Digging DEEPER it's because of https://github.com/berrberr/streamkeys/blob/master/code/js/controllers/YoutubeController.js#L14 as area-label never changes when you play/pause a video. In my testing changing it to check title seems to have fixed this issue, as it updates with the change of player state. #851 is my proposed fix.

@Staindk
Copy link

Staindk commented Feb 17, 2022

This issue has been annoying the hell out of me. I depend on Streamkeys to help me handle my debilitating tab hoarding problem and it's been failing since this problem cropped up 😄

Hopefully a maintainer merges this in and they push out a new version to the Chrome store - until then, does anyone know if there is an easy way we could fix this ourselves? Not sure how simple it is to build your own altered version of the extension and install it to Chrome etc 🙈

@lollilol
Copy link

when will this be merged/fixed lol

@BaliBalo
Copy link
Contributor

BaliBalo commented Mar 20, 2022

@Staindk I'm a bit late but it's not too too hard if you have git and node installed, by running these in a terminal:

git clone git@github.com:berrberr/streamkeys.git
cd streamkeys
git fetch origin pull/851/head:youtube-fix
git checkout youtube-fix
npm install
npm run grunt:dev

then on the chrome extensions page (chrome://extensions/) you can use "Load unpacked" with the folder streamkeys/build/unpacked-dev (don't forget to also disable the other streamkeys extension while you have this temporary one active)

I would love to see it merged and published to the actual extension though

@Staindk
Copy link

Staindk commented Jun 15, 2022

@BaliBalo And I'm even later - but just wanted to mention your instructions worked perfectly and I'm back to play/pausing youtube properly ^_^

Thanks!

@stale
Copy link

stale bot commented Oct 30, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 30, 2022
@Staindk
Copy link

Staindk commented Oct 30, 2022

Bump - AFAIK this is still a problem and should not be closed.

@stale stale bot removed the wontfix label Oct 30, 2022
@NimmLor
Copy link

NimmLor commented Nov 7, 2022

@berrberr

@stale
Copy link

stale bot commented Apr 3, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Apr 3, 2023
@runer112
Copy link

runer112 commented Apr 4, 2023

It looks like the fix for this may have been merged in #851? If so, I would be grateful if a release with this fix could be pushed to the Chrome Web Store.

@stale stale bot removed the wontfix label Apr 4, 2023
@MonzterDev
Copy link

Would be great if this was pushed :D

@jdnixx
Copy link

jdnixx commented Jun 27, 2023

This is really annoying as I can't even build the project as @BaliBalo detailed anymore, due to some npm dependency being broken or something. Really unfortunate cuz this extension has been essential for me

Btw the error, after running npm run grunt:dev is:

>> Error: Can't walk dependency graph: Cannot find module 'URIjs' from '/home/jdnixx/Downloads/streamkeys/node_modules/urlutils/index.js'
>>     required by /home/jdnixx/Downloads/streamkeys/node_modules/urlutils/index.js
Warning: Error running grunt-browserify. Use --force to continue.

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

No branches or pull requests

10 participants