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

Peacock Muting Not Working #452

Closed
richardfrost opened this issue Nov 2, 2022 · 11 comments
Closed

Peacock Muting Not Working #452

richardfrost opened this issue Nov 2, 2022 · 11 comments
Assignees
Labels
Audio Bug Something isn't working

Comments

@richardfrost
Copy link
Collaborator

🐛 Description

Muting is no longer working on Peacock.

📝 Additional context

Reported by @WolfD1g1tal

@richardfrost richardfrost added Bug Something isn't working Audio labels Nov 2, 2022
@richardfrost richardfrost self-assigned this Nov 2, 2022
@richardfrost
Copy link
Collaborator Author

It looks like Peacock has updated their site captions. I think I have worked out the updates and will get it included in the next version of APF, but here is the new config if you'd like to test it out.

{
  "www.peacocktv.com": [
    {
      "displaySelector": "div.video-player__subtitles",
      "mode": "elementChild",
      "parentSelector": "div.video-player__subtitles > span.video-player__subtitles__cue span.video-player__subtitles__line",
      "simpleUnmute": true,
      "tagName": "#text"
    },
    {
      "className": "video-player__subtitles",
      "displaySelector": "div.video-player__subtitles",
      "mode": "element",
      "subtitleSelector": "span.video-player__subtitles__cue > span.video-player__subtitles__line",
      "tagName": "DIV"
    }
  ]
}

To use this config, paste it into the Custom Sites area (Options page > Audio tab) and click the blue "SAVE" button underneath it. Then, make sure to refresh the tab for the video site (if you have it open still) and it will load this new configuration.

For best results, make sure to remove any custom config used for testing like this once its been included in the official update.

@WolfD1g1tal
Copy link

WolfD1g1tal commented Nov 2, 2022 via email

@richardfrost
Copy link
Collaborator Author

@WolfD1g1tal lol, thanks! I'm glad I was able to hopefully fix it soon. In the meantime, I've updated the Supported Sites Status Page to reflect that its not currently working. Let me know how your experience goes and if you give it the go-ahead I'll include it in a new update soon. Thanks again for reporting the issue!

richardfrost added a commit that referenced this issue Nov 11, 2022
@richardfrost
Copy link
Collaborator Author

This has been included in version 2.39.0. Feel free to comment here if you have any feedback for this update.

@ahkelly
Copy link

ahkelly commented Jul 13, 2023

Hi Richard, I am having the muting issue again on Peacock - sorry. The subtitle update is working just not the mute.

@richardfrost
Copy link
Collaborator Author

@ahkelly Thanks for reporting the issue. I'll take a look and see if I can get it working again soon.

@richardfrost
Copy link
Collaborator Author

Sorry it has taken me so long to get back to you @ahkelly, could you give this config a try:

{
  "www.peacocktv.com": [
    {
      "displaySelector": "div.video-player__subtitles",
      "mode": "watcher",
      "subtitleSelector": "div.video-player__subtitles > span.video-player__subtitles__cue span.video-player__subtitles__line"
    }
  ]
}

To use this config, paste it into the Custom Sites area (Options page > Audio tab) and click the blue "SAVE" button underneath it. Then, make sure to refresh the tab for the video site (if you have it open still) and it will load this new configuration.

I tested it on a few videos with success, but would love a second opinion before releasing it in an update.

@richardfrost richardfrost reopened this Jul 26, 2023
@ahkelly
Copy link

ahkelly commented Aug 8, 2023

Not sure if you made some more updates since I posted this but I finally checked it again today and the muting appears to be working again now without adding the custom site you note above.

@ahkelly
Copy link

ahkelly commented Aug 8, 2023

Scratch that, as I continued to watch a program it was sporadically not working. I will add this new custom definition and let you know if that helps so far it has in less than a minute test.

@richardfrost
Copy link
Collaborator Author

@ahkelly I went ahead and included the new config in version 3.2.3. Let me know how it goes though if you get the chance.

@richardfrost
Copy link
Collaborator Author

This has been included in version 3.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Audio Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants