FrostCo / AdvancedProfanityFilter

A browser extension to filter profanity from webpages
GNU General Public License v3.0
163 stars 26 forks source link

Peacock Muting Not Working #452

Closed richardfrost closed 6 months ago

richardfrost commented 1 year ago

:bug: Description

Muting is no longer working on Peacock.

:pencil: Additional context

Reported by @WolfD1g1tal

richardfrost commented 1 year ago

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 commented 1 year ago

You sir are a steely eyed missile man. She went green soon as I reloaded and set cc on. I will holler back in a day or two after I’ve watched some different things on different sections of the site. I am so grateful you got back so quick. Thank You!

richardfrost commented 1 year ago

@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 commented 1 year ago

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

ahkelly commented 1 year ago

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

richardfrost commented 1 year ago

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

richardfrost commented 1 year ago

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.

ahkelly commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

@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 commented 6 months ago

This has been included in version 3.4.0.