-
Notifications
You must be signed in to change notification settings - Fork 8
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
Skip Shortcut not working anymore #10
Comments
The skip shortcut allows you to skip pre-roll ads (trailers for other series/movies), intros/recaps, and the "Next up" countdown at the end of episodes. Which these of aren't working for you? Also, which Prime Video site (e.g. |
@lucasdeeiroz Please provide more details or I'll have to close this issue because I can't reproduce it. |
The shortcut doesn't work on any of these situations on primevideo.com. I am installing Linux on another machine and I'll see if I have the same error. |
I just had a friend test this on primevideo.com on his Linux machine and the feature was working for him. Can you check if other features (like blurred thumbnails) are working? Maybe there's something preventing the extension from being enabled. |
I have the same issue in firefox on amazon.co.uk, Mac. Other features are working properly. |
This issue is super hard to debug for me because I can only use the
Here's what these screenshots would look like for me: If you're not sure how to take these screenshots, here's how:
I know this is a little complicated, but there's nothing I can do about this issue without this information. Please let me know if you get stuck taking the screenshots, so I can help you out. Thanks! |
Thanks a lot @listy-bit, this really helps! It seems like Refined Prime Video should have no trouble finding the elements you've screenshotted since they are part of the The second theory almost seems too easy, but just to make sure we're not wasting our time: Prime Video automatically hides the "Skip Intro" and "Skip Recap" buttons after a while and you have to move your mouse to show them again. Are you sure that the buttons are visible in the exact moment you press S? |
Buttons are visible. Interestingly, I'm now seeing Skip Intro working for some episodes of some shows, but not on the others. The other buttons don't work at all. |
I've just added some detailed logging to the skip shortcut feature to help us debug this issue. Because this is just a temporary change, you'll have to install this version manually by downloading it from the build artifacts of this CI run (Chrome version, Firefox version) and then installing it manually ("Load unpacked" in Chrome, "Load temporary add-on" in Firefox). Note that you might have to disable the regular Refined Prime Video extension to avoid issues. With the new version installed, please try using the skip shortcut again and see what is logged in the browser console. Take a look at my commit to see what kind of messages you should see if everything was working correctly and let me know what you find. Thanks again for the help so far! |
I just released a new version with similar logging that has already been approved for Firefox and is currently in review for Chrome. Having the logging in the release build will allow you to always look at the logs whenever you notice the feature not working, so you can let me know. Also let me know if the feature just magically always works now! 😄 |
Oh, this now makes a lot of sense! Amazon is in the process of transitioning to generated class names (like Here are some solutions to this problem I can think of:
I'm definitely going to do some research on the different solutions, but I can't tell you how long that will take because I really want to find a solution that's going to work for a long time and not just break after a week. |
Just here to confirm the same issue. You can likely resolve this by using the :contains selector. Not supported in standard CSS, but I know jquery has an implementation. Not sure about other frameworks / whatever this project uses. |
This is basically one of the possible solutions I outlined above:
As I've already mentioned there, I can't really go with this one because my extension is available on five different Prime Video sites, all of which are again available in multiple languages. The solution I'm favoring at the moment is this one:
I've already started using generated classes for a different feature and generally have a better understanding idea of how they work now, so I'm considering using them for the skip shortcut too. I still have to do some research to make sure that I pick the right classes, but expect this to be fixed soon. |
That's great. Would it be possible to add a setting to always auto skip as well? |
Once this issue is solved, I'll consider it. On that note: I was on a call with a Canadian friend of mine yesterday and we looked at the relevant class names in his browser (on primevideo.com) together. It turns out that he's not getting the new generated class names yet. This might be because Amazon is A/B-testing something, in which case he would get the new classes eventually, but either way he currently can't retrieve them for me. @mjmatthiesen Since you are getting the new classes, could you find them for me? To do that you'd have to do this for every "broken" skip button you find:
When you're done with this process (for one button) you should have:
I know this is a lot, but it's really the only way I can fix this problem (also think of the time it took me to write down all those steps 😅). @mjmatthiesen I mentioned you above because you commented last, but @listy-bit and @lucasdeeiroz, you can help with this too (if you're still getting the cryptic classes). Note that I don't need all of the results at once, I just need all of them at some point, so just start with one of the buttons and post your results for that once you're done. Thanks! |
That's fine, I can totally understand that. It turns out it might not even be necessary to go through the entire process because from your screenshot it seems like Amazon still includes some readable class names in the HTML of their "ATV Web Player SDK": The two classes that can be seen in this screenshot alone already seem like possible replacements for two of the skip button selectors. Now I only really need screenshots of:
@mjmatthiesen This is still a little bit of work, but nowhere near what I was previously talking about. 😅 Could you do this? |
Alright, seems like we have new selectors for:
Now all that's left is the "Skip" button for advertisements. You've already found two classes ( Thanks for all the help so far! |
So that didn't find anything, but I was able to selected the event and open the file. click => function $a() {} onclick => function() { Couldn't find anything that looked like mapping. |
fu4rd6c f1cw2swo ... these don't seem terribly maintainable to me. I suspect the only thing that'd work long term is hunting down the Skip text and faking a click event. |
That's unfortunately not an option due to localization (see my earlier comment).
That's true, but the way these seemingly random class names are generated is, as far as I know, actually totally deterministic. Unless Amazon has overridden this behavior the class names should be generated based on their original name and the path of the original CSS file they come from (see the relevant code in The reason I asked for so much information in this comment is because I want to make sure I pick the right class and am able to document that within the source code (using the original class name as the SCSS variable name is the most readable approach for that in my opinion). @mjmatthiesen said that he wasn't able to find the original class names, maybe you (@jettero) could try again? In the case that it's really impossible to find the mappings, could you find out which CSS files these classes are defined in and post the URLs? |
Addresses some of the problems in #10
An hour ago I released a new version that adds the classes I mentioned here as button selectors, so "Skip intro", "Skip recap", and "Next up" should all work again now (please let me know if they still don't for some reason). It's still in review at the moment for Chrome, so you might have to wait a little bit before you get it Regarding the remaining "Skip ad" button: There's really nothing I can do here because I'm still not getting the new classes and, even in the regions where Amazon has already (sort of) rolled them out, not all users seem to be getting them (see this comment) anyway, so I don't think proxying is worth the hassle (I'm also not sure if I would even be able to watch anything on primevideo.com). Therefore, unless I do eventually get the new classes myself, I'm not going to investigate this any further. Help wanted! 🔧If anyone is already getting the new classes (which means that you're not able to skip ads by pressing S) and has a bit of time to spare, I'd really appreciate it if you could look into this and find a CSS selector that can be used to reliably find the "Skip ad" button. I've talked about my process for figuring out these kinds of things in an earlier comment, but I don't really care how exactly you go about it as long as you can prove that the selector only matches the skip button. Sufficient proof could be:
Note that there's no need to modify the extension's code (or even use the extension at all) to help with this issue. The search feature of your browser's developer tools should be all you need. Thanks everyone for the help with this issue so far, I hope we'll be able fully fix it eventually! |
I'm still having this problem. I'm watching season 2 of the outpost & the shortcut isn't working on the Freevee ads. |
I noticed this feature not working anymore after installing Chrome Beta on Windows. I don't know if that's really the issue, tho. But nothing happens when I press S on keyboard now.
⬇️ Find out about the current status of this issue and how you can help fix it
The text was updated successfully, but these errors were encountered: