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

Element Picker does not work for certain sites #889

Closed
5 of 8 tasks
ghost opened this issue Feb 7, 2020 · 7 comments
Closed
5 of 8 tasks

Element Picker does not work for certain sites #889

ghost opened this issue Feb 7, 2020 · 7 comments
Labels
duplicate This issue or pull request already exists

Comments

@ghost
Copy link

ghost commented Feb 7, 2020

Prerequisites

  • I verified that this is not a filter issue
  • This is not a support issue or a question
  • I performed a cursory search of the issue tracker to avoid opening a duplicate issue
    • Your issue may already be reported.
  • I tried to reproduce the issue when...
    • uBlock Origin is the only extension
    • uBlock Origin with default lists/settings
    • using a new, unmodified browser profile
  • I am running the latest version of uBlock Origin
  • I checked the documentation to understand that the issue I report is not a normal behavior

Description

On certain sites the Element Picker does not work. When you click ... | uBlock origin | Element Picker Symbol the uBlock tab is closed but the UI is not injected into the tab of interest. It does work as expected on other sites for example google.com.

I used the remote debugger and the only log event on my phone was:

Scripts may not close windows that were not opened by script.

and it happens for both working and non-working site. It appears to be a different issue than #81 as it does work google.com. Also, I reproduced this with both the latest stable version and the b5 as of last night.

I cannot reproduce this issue with uBlock 1.24.4, Firefox 68.4 on Linux (desktop), i.e. it appears to specific to Mobile. That said, reddit.com is doing a lot mobile specific stuff including aggressively pushing their app which makes the site annoying to browse on mobile.

A specific URL where the issue occurs

https://reddit.com

Steps to Reproduce

  1. Open https://reddit.com
  2. Click ... | uBlock origin | Element Picker Symbol the uBlock tab

Expected behavior:

The uBlock tab should be closed and the tab with the reddit.com should have the element picker ui element injected.

Actual behavior:

The ublock tab is closed but the element picker ui is not injected into the reddit.com tab.

Your environment

  • uBlock Origin version: 1.24.4, 1.24.5b3
  • Browser Name and version: Firefox Browser (Mobile) 68.4.2
  • Operating System and version: Android 9
@uBlock-user
Copy link
Contributor

uBlock-user commented Feb 7, 2020

Enable cosmetic filtering ?

Did you click on the element after clicking on the picker icon ?

@uBlock-user uBlock-user added the unable to reproduce cannot reproduce the issue label Feb 7, 2020
@Yuki2718
Copy link

Yuki2718 commented Feb 8, 2020

I know it is only for confirmed issue so haven't reported but I experienced the same problem a few times on Firefox mobile running on Android 10. I doubt it has sth to do w/ reddit, it rather seemed to be a problem of some tab(s) open at the time. Restart the browser, reload the tabs, close and open uBO dashboard etc. eventually fixed it, please test @allanwind. I still don't figure out conditions to reproduce.

@Yuki2718
Copy link

Yuki2718 commented Feb 8, 2020

Enable cosmetic filtering ?

Did you click on the element after clicking on the picker icon ?

In my case, cosmetic is ofc enabled and if I click sth just a usual event (e.g. if it was a link, go to the site) occurred.

@ghost
Copy link
Author

ghost commented Feb 8, 2020 via email

@Yuki2718
Copy link

Yuki2718 commented Feb 8, 2020

Then this issue should be closed, as I so far have no idea to reproduce what I said.

@uBlock-user
Copy link
Contributor

Duplicate of gorhill/uBlock#3212

@uBlock-user uBlock-user marked this as a duplicate of gorhill/uBlock#3212 Feb 8, 2020
@uBlock-user uBlock-user added duplicate This issue or pull request already exists and removed unable to reproduce cannot reproduce the issue labels Feb 8, 2020
@gorhill
Copy link
Member

gorhill commented Feb 8, 2020

Maybe the flash and element picker icons shouldn't be clickable if
the cosmetic filtering is not enabled?

The zapper must still be usable without cosmetic filtering, so the solution is to make the zapper independent from the picker code. As for the picker, someone may still want to create network-based filters, so in that case the solution is to ensure the picker can still function without the DOM filterer code present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants