-
Notifications
You must be signed in to change notification settings - Fork 784
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
streamsilk.com: popups #25510
Comments
The code responsible for the popup is in the first lines in This works here:
|
I think we can limit to anchor elements only streamsilk.com##+js(set, HTMLAnchorElement.prototype.click, noopFunc) |
both of them work, but limiting to anchor elements is good.also site is self broken i guess, timer never ends. also is issue can be fixed with such idea |
Yeah, that's what I was thinking too. Currently I can't think of other solutions based on current methods that can defuse element @gorhill Sorry, could you take a look? |
another examples we have are hydrax players |
gorhill/uBlock@95b0ce5. This should work:
|
|
The only way I can think of is to output the element at the console. Maybe this could be done when verbose mode is enabled? Another way is to add Problem with either approach is that the page detect dev tools being opened. The issue with the verbose mode approach is that the element is removed immediately from the DOM upon return. |
You will never see elements themselves in the logger, elements or element references are not serializable, they can't be transported across documents. We can output an element to the browser dev console, and from there the browser tools will let you find it in the DOM. In any case, the element is removed from the DOM immediately by the page code, so it's not very useful. Triggering a breakpoint in the code when the override occurs is a better approach, because it will let you examine the element while it still is in the DOM. |
Anyway, how do I trigger the filter on that site? I don't get a report that the method was overridden. |
Support `debug` as valid `disposition` value: to trigger a `debugger` statement, but only if uBO's logger is opened. Related feedback: uBlockOrigin/uAssets#25510 (comment)
Next build you can try |
|
With the new |
|
Forgot about |
Can't you make it specific to certain elements or urls ? like we have for |
Why invest time on this unless there are actual cases to solve? |
This could resolve this case without having to resort to a new scriptlet, but it's your choice though. |
|
i added |
this was part of code responsible for popups
|
I just reproduced successfully. Did you enable My steps in a new browser profile:
|
ok some firefox hang problem on my end, i could see debugger statement on opera👍 |
Can |
I could add a special case for the url parameter:
Which would trigger a |
Yes, that should work. I was thinking of using it on |
Support triggering a `debugger` statement when `window.open()` is called. Related feedback: uBlockOrigin/uAssets#25510 (comment)
|
no popup on my side even disabling |
They still keep that script: function openNewTab(url) {
var a = document.createElement("a");
a.href = url;
a.target = "_blank";
a.style.display = "none";
document.body.appendChild(a);
a.click();
document.body.removeChild(a);
} |
@ghajini Does this work on your side? streamsilk.com##+js(nostif, adDiv) |
it does not work for me |
Oh sorry, I checked wrong page. |
does having stack trace in set-constant will help? |
Maybe. But currently the only way I can think of is streamsilk.com##+js(rpnt, script, window.dataLayer =, 'window.HTMLAnchorElement.prototype.click=new Proxy(window.HTMLAnchorElement.prototype.click,{apply:(target,thisArg,args)=>{console.log(thisArg);if(thisArg&&!thisArg.href.includes("streamsilk.com"))return;return Reflect.apply(target,thisArg,args)}});window.dataLayer =', sedCount, 1) |
popups only on mobile ? |
ok, popups on the first link in the issue, not on the last link |
opera/ firefox mobile |
that works for me |
Prerequisites
URL address of the web page
https://streamsilk.com/d/66fef1e5f0d37
Category
popups
Description
clicking 480/720 generates new popup window which is closed due to matching EL filter
Other extensions used
none
Screenshot(s)
Screenshot(s)
Configuration
The text was updated successfully, but these errors were encountered: