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

ublock unhides hidden elements when it's updated #3185

Closed
SoniEx2 opened this issue Oct 31, 2017 · 7 comments
Closed

ublock unhides hidden elements when it's updated #3185

SoniEx2 opened this issue Oct 31, 2017 · 7 comments

Comments

@SoniEx2
Copy link

SoniEx2 commented Oct 31, 2017

Describe the issue

ublock unhides hidden elements when it's updated

One or more specific URLs where the issue occurs

tumblr.com/dashboard

Screenshot in which the issue can be seen

[pending ublock update]

Steps for anyone to reproduce the issue

  1. put something like www.tumblr.com##.post[data-is_recommended="1"] in your custom filters
  2. go on a page with such elements
  3. wait for ublock to update

Your settings

set:

  • hide placeholders
  • show blocked count
  • use context menu
  • advanced mode
  • no prefetch
  • no auditing

unset:

  • everything else?

  • OS/version: arch linux x86_64
  • Browser/version: firefox 56.0.1
  • uBlock Origin version: 1.14.17b9
Your filter lists

all ublock
easylist
adblock warning removal
easyprivacy
malware domain list
malware domains
fanboy's annoyance
fanboy's social
peter lowe's ad/tracking list
aaklist (which is apparently "custom"?)

Your custom filters (if any)
! 2017-04-19, 12:18:23 PM https://uk.pinterest.com/autisticmind/drawings/
uk.pinterest.com##div:nth-of-type(3) > div:nth-of-type(3)

! 2017-05-03, 5:43:42 PM https://www.tumblr.com/dashboard
www.tumblr.com##.post[data-is_recommended="1"]

||belarr.com/bakercat/
||belarr.com/bakercat/$document

! 2017-08-16, 12:27:28 AM https://www.tumblr.com/dashboard
www.tumblr.com##.chat-status-banner

! 2017-08-16, 12:27:45 AM https://www.tumblr.com/dashboard
www.tumblr.com##.status-indicator

! 9/24/2017, 12:30:08 PM https://www.tumblr.com/dashboard
! www.tumblr.com##.post:has(.recommendation-reason-link)
!

! speedtest stuff
@@/hello^$xmlhttprequest,domain=speedtest.net
@@/ping^$xmlhttprequest,domain=speedtest.net
@@/download^$xmlhttprequest,domain=speedtest.net
@@/upload^$xmlhttprequest,domain=speedtest.net

edit: I believe this can be fixed with startup hooks: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/runtime/onInstalled (but I'm not an extension developer)

@gorhill
Copy link
Owner

gorhill commented Oct 31, 2017

Browser behavior, probably by design.

@SoniEx2
Copy link
Author

SoniEx2 commented Nov 16, 2017

Could this be reopened?

@gorhill
Copy link
Owner

gorhill commented Nov 16, 2017

You can comment on it without having to re-open. Anyway, I will try something on my side, I just realized I broke the manual injection of scripts at launch with a recent change.

@gorhill gorhill reopened this Nov 16, 2017
@gorhill
Copy link
Owner

gorhill commented Nov 16, 2017

What do you mean by "startup hook"?

@gorhill
Copy link
Owner

gorhill commented Nov 16, 2017

Un-hiding cannot be fixed, this is a by-design browser behavior. However I fixed the broken manual injection of content scripts, so the cosmetic filters should be re-applied as used to be the case.

@SoniEx2
Copy link
Author

SoniEx2 commented Nov 16, 2017

You can reapply the hiding when the startup hook is called, I think. https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/runtime/onInstalled

@gorhill
Copy link
Owner

gorhill commented Nov 16, 2017

I know about these listeners -- uBO of course knows when its being launched, this is not the issue. The issue is to have the content scripts injected only when all the filter lists have been loaded so that the required cosmetic filters can be fetched. Anyways, fixed as best as can be with the last commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants