Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Massive blocked iframe on some facebook sites #36

Closed
semenko opened this issue Jul 1, 2014 · 8 comments
Closed

Massive blocked iframe on some facebook sites #36

semenko opened this issue Jul 1, 2014 · 8 comments

Comments

@semenko
Copy link

semenko commented Jul 1, 2014

Perhaps this is what's meant by the disclaimer for the anti-facebook list, but a number of sites that use <iframe> to embed facebook content end up with huge blocked icons (the iframe height & width are 1000px ea):

e.g. http://www.monoprice.com/Product?c_id=104&cp_id=10401&cs_id=1040115&p_id=9436&seq=1&format=2

I figured "Hide placeholders of blocked elements" would hide that enormous iframe, but it doesn't.

blocked

@gorhill
Copy link
Contributor

gorhill commented Jul 1, 2014

To me it looks like Fanboy is missing a filter to go with the other Facebook filters. Can you try: ||connect.facebook.net/*/all.js^$third-party?

Looks like I will need a project list to fix Fanboy's annoyance and Fanboy Anti-Facebook until (and if) they add the missing filter.

Other way to address this would be to also listen to attribute changes in the mutation observer, but this is something I am trying to avoid (overhead etc.) So I see the above filter as logically missing in the relevant filter list.

Edit: updated to use a * in lieu of en_US.

@semenko
Copy link
Author

semenko commented Jul 1, 2014

Ah nice, that does the trick.

@gorhill
Copy link
Contributor

gorhill commented Jul 1, 2014

Just for the record, I don't think ABP would have suffered the same problem, because it listens to all load events, while uBlock listens only to the ones from elements which have not have their src property set (that could be a problem in certain cases, but I want to try the lower overhead angle first). Either ways, using above filter is the right thing to do, as otherwise users are still hitting 3rd-party Facebook server.

@gorhill
Copy link
Contributor

gorhill commented Jul 1, 2014

Submitted suggested fix to Fanboy's forum.

@semenko
Copy link
Author

semenko commented Jul 2, 2014

Awesome -- thanks again.

I love the focus on keeping uBlock's overhead incredibly low. I get the sense a lot of people avoid ABP & similar extensions due to CPU overhead on thin-ish clients like Chromeboxes (It's an ongoing problem we face with HTTPS Everywhere).

@gorhill
Copy link
Contributor

gorhill commented Jul 3, 2014

Answer from fanboy:

Would break facebook comments; add the Anti-Facebook sub from https://www.fanboy.co.nz if you don't mind blocking facebook comments.

Enabling Fanboy Anti-Facebook didn't work. Yet I see this filter in the list: ||connect.facebook.net^$third-party,domain=~facebook.com, which means it should be blocked. Investigating.

@gorhill
Copy link
Contributor

gorhill commented Jul 3, 2014

Interesting, this issue allowed to uncover a nasty bug which affected filters using the ~example.com filter option. Once this bug is fixed, the suggestion from Fanboy will work, or else the other workaround of using the custom filter ||connect.facebook.net/*/all.js^$third-party. I will open issue for the uncovered filter bug though, it deserves its own issue.

gorhill added a commit that referenced this issue Jul 20, 2014
@gorhill
Copy link
Contributor

gorhill commented Jul 21, 2014

Fixed in 0.2.3.2.

@gorhill gorhill closed this as completed Jul 21, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants