-
Notifications
You must be signed in to change notification settings - Fork 82
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
uBlock Origin breaks "Open image in new tab" when there is a rule to block popups #3436
Comments
https://bato.to/title/81559-skip-beat-official/2492793-ch_299 I can't reproduce firefox_Rqi805gaCe.mp4 |
idk why. i just reproduced it also in chrome. |
Can you open new tab > reproduce the issue and on that page > click on uBO icon > 💬 Report button > scroll to the bottom > click on |
uBlock Origin: 1.60.0
Chromium: 130
filterset (summary):
network: 140587
cosmetic: 48803
scriptlet: 22937
html: 0
listset (total-discarded, last-updated):
default:
user-filters: 0-0, never
ublock-filters: 40518-265, 5m
ublock-badware: 11724-6, 5m
ublock-privacy: 1391-22, 5m
ublock-unbreak: 2493-0, 5m
ublock-quick-fixes: 229-0, 5m
easylist: 80404-586, 5m
easyprivacy: 53214-648, 5m
urlhaus-1: 20480-0, 5m
plowe-0: 3542-0, 5m
filterset (user): [empty]
userSettings: [none]
hiddenSettings: [none]
supportStats:
allReadyAfter: 5034 ms (selfie)
maxAssetCacheWait: 271 ms
cacheBackend: indexedDB
popupPanel:
blocked: 2
network:
bookstore.pics: 1
googletagmanager.com: 1 i guess the "bookstore.pics" is the image |
do you have easylist filters enabled when trying to reproduce it? the rules that block popups on the example sites are part of easylist. |
I'm using default settings. |
This comment was marked as abuse.
This comment was marked as abuse.
This EasyList filter is causing the new tab to close:
Unfortunately uBO doesn't know that the tab was opened using a context menu entry, that information is not available to uBO. Not something which can be fixed. AdGuard and ABP also suffers this issue. The issue does not exist on Firefox because the event listener onCreateNavigationTarget() is not called in Firefox. |
This comment was marked as abuse.
This comment was marked as abuse.
#932 talks about middle-click which i can not test, however it should be the same as ctrl+click or "open in new tab" for links instead of images, which works fine here. Either things have changed in the four years since, or maybe its just in that specific "Cent" chromium derivative. And it was actually asking for an easy way to temporarily allow a popup, not to fix how the $popup filters work (if you say this is impossible). I think that's actually a reasonable idea/alternative. Perhaps something like the browser's built-in popup blocker which iirc showed an indication for the blocked popups in the address bar and you can click it to allow/open a specific popup that was blocked. Or, if uBlock Origin is able to see the modifier key state at the time the popup is opened, perhaps allow them if "Ctrl" is being held, or similar. I'd go as far as to suggest uBlock can probably remove the built-in "Open image in new tab" if its code is such that its impossible to distinguish from automatic popups, and replace it with its own menu item with code that sets a flag or uses a way to open a tab which can be distinguished. Or are extension APIs so limited that even that is impossible now? (I know that was the trend things were going but I'm not aware how bad it's gotten yet) |
This comment was marked as abuse.
This comment was marked as abuse.
Whoever thinks it's possible to fix can submit a PR. I prefer people prove me wrong with a PR than me having to spend a lot of time to explain why suggested ideas don't work. |
@garry-ut99 all those are about opening normal links in new tab, which actually works now. |
i added fix in uBlockOrigin/uAssets@cc49b9f as i see valid content is blocked |
Closing as duplicate of gorhill/uBlock#3282. |
Prerequisites
I tried to reproduce the issue when...
Description
uBlock Origin breaks "Open image in new tab" when there is a rule to block popups.
A manual "Open image in new tab" should not be considered the same as an automatic popup and should not be blocked, the same way as "Open link in new tab" is not.
A specific URL where the issue occurs.
Steps to Reproduce
Expected behavior
The image should open in new tab and stay open.
Actual behavior
In Edge a new tab briefly flashes and immediately disappears.
In Chrome, it doesn't even flash briefly, it just never opens.
uBO version
1.60.0
Browser name and version
Edge 130.0.2849.56
Chrome 130.0.6723.92
Operating system and version
Windows 11
The text was updated successfully, but these errors were encountered: