danvk / oldnyc

Mapping photos of Old New York
Apache License 2.0
288 stars 132 forks source link

Breaks with ad blocker #72

Closed danvk closed 9 years ago

danvk commented 9 years ago

@mgiraldo says that disconnect.me prevents the details page from loading.

The solution here might be to do #32, so that I can drop xdomain and do same-domain XHRs for data.

If I take this route, I'd like to do it ASAP so that I'm not mucking with DNS post-launch (DNS changes terrify me).

danvk commented 9 years ago

@mgiraldo if you feel strongly about this, it would help to get some more specifics, e.g. does removing the facebook comment widget make the issue go away?

mgiraldo commented 9 years ago

it works fine in my local instance following the instructions in the README:

old nyc - photo 708788f-a

i just see it in the hosted version

mgiraldo commented 9 years ago

ReferenceError: FB is not defined in the hosted version

inspector - http___www oldnyc org_ 708760f-a

mgiraldo commented 9 years ago

what if we use Disqus or some other non-FB alternative? i know we discussed this as an option

danvk commented 9 years ago

Is it blocking the FaceBook JavaScript bundle? If I open oldnyc.org in Chrome, go to the devtools and type FB, I get an object back. The error you posted would imply that this isn't happening.

mgiraldo commented 9 years ago

correct

On May 13, 2015, at 1:58 PM, Dan Vanderkam notifications@github.com wrote:

Is it blocking the FaceBook JavaScript bundle? If I open oldnyc.org in Chrome, go to the devtools and type FB, I get an object back. The error you posted would imply that this isn't happening.

— Reply to this email directly or view it on GitHub.

danvk commented 9 years ago

This is a bug with disconnect.me then, not with OldNYC. I imagine if you went to facebook.com, nothing would load at all!

mgiraldo commented 9 years ago

ok... i wanted to see if we could avoid FB altogether

it's ok if you want to make it a requirement

mgiraldo commented 9 years ago

FWIW, it does work locally:

inspector - http___localhost_5000_ 708760f-a

your call anyway

danvk commented 9 years ago

Yeah, I don't know why your adblocker would block the facebook JS in one case but not the other.