[Done] Removing "check banner links"

Various discussions related to Adblock Plus development
Wladimir Palant

[Done] Removing "check banner links"

Post by Wladimir Palant »

I would like to know your opinion on this - is there a good reason to keep the "Check banner links" feature and the corresponding option around?

Reasons against it:

1. It has a strong negative effect on performance if enabled - meaning that switching it on is not advisable.
2. It is difficult to explain what this feature does, people often assume that it can be used to "block" links.
3. It causes some hard issues (e.g. bug 13633, bug 16965).
4. There are very few cases where this feature is useful - dict.leo.org is the only popular example that comes to mind (ads on the n-tv site that have been brought up by chewey before can be blocked directly by now). And element hiding should be better to deal with these cases, now that we have Element Hiding Helper.
5. I don't remember anybody complaining when we switched this feature off by default (thus switching it off for everybody who had it on before) - barely anybody noticed.

Any strong arguments for leaving this feature in?
Last edited by Wladimir Palant on Thu Oct 30, 2008 11:14 am, edited 1 time in total.
User avatar
rick752
Posts: 2709
Joined: Fri Jun 09, 2006 7:59 pm
Location: New York USA
Contact:

Post by rick752 »

Removing it is fine with me. I find that it make ABP slower and just makes looking through the blockable items list more confusing.

I've always though that the problems outweighed any benefits of having that feature 'on' .... but that is just MY opinion.
User avatar
Adblock Plus Fan
Posts: 1255
Joined: Sat Feb 24, 2007 11:08 am

Post by Adblock Plus Fan »

Personally I don't use that feature either.

What I REALLY would prefer though was a revision of this feature.

I think $~link should be a default behaviour for all filters, and when one wants to use the feature one can add $link to the filter.

This is the best way that feature can be used in my opinion.
IceDogg
Posts: 909
Joined: Fri Jun 09, 2006 11:22 pm

Post by IceDogg »

I don't use the feature either. Tried it a few times and never seen positive results. But I make very few filters on my own.
Wladimir Palant

Post by Wladimir Palant »

@Fan: It doesn't work that way - either we are looking at the links or we don't. So if we make it dependent on filters, one single filter with $link will switch the option on and make the performance go down. This is very non-obvious, especially since you could get filters from all the various sources. Not to mention that I will still have to support a feature that nobody uses.
User avatar
Adblock Plus Fan
Posts: 1255
Joined: Sat Feb 24, 2007 11:08 am

Post by Adblock Plus Fan »

I guess it's best to just kill it then if it's buggy and all that.

I don't mind as long as people who used it still has an alternative with element hiding.

Does this mean we can have room for another new feature? Hope it'll be a good one :P
User avatar
rick752
Posts: 2709
Joined: Fri Jun 09, 2006 7:59 pm
Location: New York USA
Contact:

Post by rick752 »

@ Wladimir:

If this feature is disabled in the next version of ABP and the '$link' option is removed, would that screw up any filters that contain '$link' or '$~link' ?

In other words, would a filter string that uses that switch become screwed up somehow because of an 'unknown' switch? Or would it just ignore the invalid switch and still use the string?
User avatar
fanboy
Posts: 3446
Joined: Sun Jun 17, 2007 4:45 am
Contact:

Post by fanboy »

I'd probably more inclined to removing it from the menu, and make it an about:config option only... rather than removing it completely.
Wladimir Palant

Post by Wladimir Palant »

@Rick: Unknown switches are ignored - so $~link will behave as before but $link will suddenly match everything. Looking through all subscriptions, there is exactly one such rule (in RU AdList). So I don't think this is a big issue.

@fanboy: Leaving a feature that nobody uses is a great way to end up with a broken feature after the next big change in Adblock Plus core. Unless I remember to test everything both with this hidden pref switched on and off - but I think my time is better spent on more important things.
sheepy
Posts: 147
Joined: Sat Jun 17, 2006 8:44 pm

Post by sheepy »

+1 for removing it. I am using element blocking for the links.
Wladimir Palant

Post by Wladimir Palant »

Apparently, in Firefox 3 the "Check banner links" option causes issues with some Flash objects (object doesn't appear at all). While this is something to be fixed in Firefox, I would prefer to touch the web page as little as possible to avoid exactly this kind of problems (this also means finding a replacement for object tabs).
dgod

Post by dgod »

I would like to keep it there, so I can have more general filters.
gustoflax

Post by gustoflax »

If it were removed could it be ported to a separate extension?

Perhaps it would give RIP a run for its no money ;)
Dr. Evil
Posts: 194
Joined: Fri Sep 08, 2006 3:51 pm

Post by Dr. Evil »

another example where this feature is useful is welt.de:
http://www.welt.de/wirtschaft/article1691199/...

the ads in the right column there are pretty much unblockable just by their url, but (some) clicks are registered via doubleclick.

I have the feeling that the number of such self-hosted ads is increasing (which would be positive, if it weren't flash or images and a human would control whether it's malware or something), so I would vote against removing the feature altogether.

I can't use the feature myself though, because virtually all the users of my subscription have it disabled.
Ares2
Posts: 1275
Joined: Fri Feb 15, 2008 12:47 pm

Post by Ares2 »

I've never used the feature, so I would just hide them with something like

Code: Select all

#*(href*=doubleclick.net)
instead.

Well, they are still downloaded, but I think a site that hosts the ads itself wants to keep the traffic low for them, so that won't be a serious problem.

Another vote to remove it. :)
Locked