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

Firefox TestPilot breaks image replacement. #161

Open
bacconnaise opened this issue Dec 20, 2017 · 5 comments
Open

Firefox TestPilot breaks image replacement. #161

bacconnaise opened this issue Dec 20, 2017 · 5 comments
Labels
br-firefox This only affects Firefox

Comments

@bacconnaise
Copy link

I have the latest firefox version and i do not see ads but i dont see cats either.
I have included a screenshot of a site that has ads but i dont see the ads nor the cats. I have also thrown in a screenshot with the CatBlock paused on the page for comparison.

2017-12-20 13_32_59-adblock options

2017-12-20 13_33_45-all resources
2017-12-20 13_33_30-verify email address online - free email verifier - free email address verificat
2017-12-20 13_34_46-verify email address online - free email verifier - free email address verificat

Do you have any other comments? If you can, can you please attach a screenshot of the bug?
=== Filter Lists ===
adblock_custom
last updated: Wed, 20 Dec 2017 18:20:30 GMT
easylist
last updated: Wed, 20 Dec 2017 18:20:34 GMT
malware
last updated: Wed, 20 Dec 2017 18:20:31 GMT
antisocial
last updated: Wed, 20 Dec 2017 18:20:37 GMT

=== Custom Filters ===

=== Settings ===
catblock: true
debug_logging: false
youtube_channel_whitelist: false
whitelist_hulu_ads: false
show_context_menu_items: false
show_advanced_options: true
display_stats: true
display_menu_stats: true
malware-notification: undefined

=== Other Info ===
CatBlock version number: 1.4.9 Developer
UserAgent: Mozilla/5.0 (Windows NT 10.0 Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0

@itskdog
Copy link
Member

itskdog commented Dec 21, 2017

Hi!

Due to how CatBlock works on top of AdBlock's codebase, we are currently unable to replace ads that can be blocked from being dowloaded in the first place, as the browser doesn't inform us of the location of the ad after it has blocked it. These ads are listed in red on the resource list page in the screenshot you sent us.

If it is an ad that cannot be blocked, AdBlock is able to hide it, by making it invisible. CatBlock is currently able to replace those ads only. If you have advanced options enabled (which according to your debug information, is enabled) these hidden ads will show in blue on the resource list page.

Is this problem on this website only? I often test using the Reddit homepage. There is usually a cat or two showing in the sidebar on the right-hand side of the page.

Hope this helps!
Kieran

@bacconnaise
Copy link
Author

bacconnaise commented Dec 21, 2017 via email

@bacconnaise
Copy link
Author

Hi,
I have figured out the issue and I think that, with the new Firefox browser that was launched recently, it would be good to like put a notice somewhere to inform other who might have the same issue.

I have the Firefox Test Pilot add-on installed (https://testpilot.firefox.com/experiments). After disabling it, BAM! cats4ads!

I do now see the cats instead of the ads and this is the best add-on ever. Why would someone not want cats for ads seriously ?!

Just to give you an idea, I have attached the screenshots I get now in Firefox and with all the cats!

But yeah, I think it would be good to drop a note on this if others happen to have the Test Pilot add-on installed that they won't see cats unless they disable it.

2017-12-21 12_23_30-

2017-12-21 12_22_34-all resources

@itskdog itskdog changed the title Do not see ads but do not see cats Firefox TestPilot breaks image replacement. Dec 21, 2017
@itskdog
Copy link
Member

itskdog commented Dec 21, 2017

Hi again!

Thanks for the info!

I'm using Firefox, and have just installed TestPilot, and all seems OK to me. What experiments did you have enabled in TestPilot?

It's best that we can reproduce the behaviour on our end before we can even try fixing it.

Thanks,
Kieran

@itskdog itskdog added the br-firefox This only affects Firefox label Dec 21, 2017
@bacconnaise
Copy link
Author

Hi Kieran,

Make sure to close and re-open Firefox if you just installed the Test Pilot. You should see a humpty-dumpty egg type of icon in the top-right for the Test Pilot add-on when the install is completed.

I only have the Notes and Snooze Tabs and Notes installed through the Test Pilot.

However, after disabling the Test Pilot addon, the Snooze Tabs and Notes are still available without interference for the ads replacement.

I just tested re-enabling the Test Pilot add-on, closed and reopened Firefox, and the issue is back and no ads are replaced with cats.

Hope that helps!

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
br-firefox This only affects Firefox
Projects
None yet
Development

No branches or pull requests

2 participants