Closed Bug 647876 Opened 13 years ago Closed 13 years ago

Omnibus ad-blocking bug for April/May/June/July/August 2011

Categories

(Camino Graveyard :: Annoyance Blocking, defect)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: alqahira, Assigned: alqahira)

References

Details

Attachments

(2 files)

Attached file Testcase from Finn.no
Ad-blocking prevent content from showing up at the norwegian online sale site Finn.no
The attached file is a testcase from the "My findings" page, and when I check "Block web advertising" the content with picture disappear.

Hope the ad-blocking can be fine tuned to work around this. :-)
(In reply to comment #1)

<div class="ad_image"> and <div class="ad_info">
blocked by div[class^="ad_"], /* line 647 - added in Bug 502536 */

I think you can unblock those with:
@-moz-document domain(Finn.no) {div[class^="ad_"] { display: block !important; }
Thanks!
The content disappeared just a couple of weeks ago, but I guess that is due to path changes at their side.

I have added the two lines to my chrome/userContent.css (which I created), but it does not seem to make a difference. Any tricks, or should I just wait for a fix to be checked in?
Blocks: 654965
On browsershot.org

<div class="advert_list">
<ul class="left">
<li>
<a target="_blank" rel="header1" class="adbox" title="PSD to HTML" href="http://www.psd2html.com/order-now.html?b1">
<img width="125" height="125" alt="PSD to HTML" src="/static/images/creative/125x125/p2h.jpg"/>
</a>
</li>
….

and (front page only, I think) 
<a target="_blank" rel="right1" class="adbox" title="PSD to HTML" href="http://www.w3-markup.com/order?b2">
<img width="125" height="125" alt="PSD to HTML" src="/static/images/creative/125x125/w3m.gif"/>
</a>

--> div[class="advert_list"] and a[href][class="adbox"]
On http://articles.sun-sentinel.com/2011-06-08/news/fl-photographers-letter-20110608_1_trespassing-signs-public-property

Tons of ads in various places, all which share

div[class="mod-adcpc]
Summary: Omnibus ad-blocking bug for April 2011 → Omnibus ad-blocking bug for April/May/June 2011
Someone please remind me about this bug shortly after we ship b1, so that we can get nightly baking of new rules in advance of 2.1rc.  We're too close to b1 to do anything with it now.

(In reply to comment #2)
> (In reply to comment #1)
> 
> <div class="ad_image"> and <div class="ad_info">
> blocked by div[class^="ad_"], /* line 647 - added in Bug 502536 */
> 
> I think you can unblock those with:
> @-moz-document domain(Finn.no) {div[class^="ad_"] { display: block
> !important; }

(In reply to comment #3) 
> I have added the two lines to my chrome/userContent.css (which I created),
> but it does not seem to make a difference. Any tricks, or should I just wait
> for a fix to be checked in?

Kai, philippe's rule had a small but significant typo; it needs another } at the end.  I usually write such rules over three lines to prevent myself from making that same typo, because I've done it a bunch myself:

@-moz-document domain(Finn.no) {
div[class^="ad_"] { display: block !important; }
}

I don't foresee us removing the |div[class^="ad_"]| line, so let us know if putting that rule in userContent.css or the new, designed-for-this-exact-purpose $profile/chrome/user_ad_blocking.css still doesn't work.
Noticed this on
http://stiern.com/articles/usability/usability-in-icons/

Using section[id="ads"] to wrap a block around the ads [*]. The rest of the code is typical buysellads stuff, which I'm a bit wary to block on id or class with 'bsa'

<section id="ads" class="adpacks">
<div id="bsap_1260443" class="bsap_1260443 bsap">
<div class="bsa_it one">
<div id="bsa_1210907" class="bsa_it_ad ad1 odd">

[*] I would nitpick on his usage of <section> here, I'd used <aside> … Just mentioning it as we might see more alternates to <div> as a wrapper in the future.
Fixes the items here (with the exception of not removing div[class^="ad_"], per comment 6) as well as the ad part of bug 654965.

Lots of months, but little more than 1 rule/month :P
Assignee: nobody → alqahira
Status: NEW → ASSIGNED
Attachment #560508 - Flags: superreview?(stuart.morgan+bugzilla)
Flags: camino2.0.10?
Summary: Omnibus ad-blocking bug for April/May/June 2011 → Omnibus ad-blocking bug for April/May/June/July/August 2011
for the record, the correct url in comment 4 is browsershots.org.
Comment on attachment 560508 [details] [diff] [review]
Apr/May/Jun/Jul/Aug 2011 omnibus ads

sr=smorgan
Attachment #560508 - Flags: superreview?(stuart.morgan+bugzilla) → superreview+
http://hg.mozilla.org/camino/rev/08681bc48996
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
2.0.x is gone.
Flags: camino2.0.10? → camino2.0.10-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: