Opened on 06/20/2014 at 01:15:29 PM
Closed on 07/30/2014 at 11:02:21 AM
Last modified on 05/20/2015 at 02:22:39 PM
#690 closed change (fixed)
Issue reporter: don't consider special subscriptions when complaining about high subscription count
Reported by: | trev | Assignee: | saroyanm |
---|---|---|---|
Priority: | P3 | Milestone: | Adblock-Plus-2.6.5-for-Firefox |
Module: | Adblock-Plus-for-Firefox | Keywords: | |
Cc: | Ede_123 | Blocked By: | |
Blocking: | Platform: | Firefox | |
Ready: | yes | Confidential: | no |
Tester: | Verified working: | no | |
Review URL(s): |
Description (last modified by trev)
Background
Issue reporter currently disallows reports listing more than five filter subscription to discourage reports where people subscribed to everything on our list and to give people the feedback that this is a good idea. However, this check was implemented before acceptable ads and anti-adblock were introduced, these "special" subscriptions also count into the number.
What to change
Don't count acceptable ads and anti-adblock for the purpose of this warning.
Testing notes
This is about Firefox desktop only:
- In the Adblock Plus filter preferences, make sure that you have four filter subscriptions in addition to "Adblock Warning Removal List", add more if necessary. Also make sure that "Adblock Warning Removal List" and "Allow some non-intrusive advertising" are checked.
- Go to some webpage, click the ABP icon and select "Report issue on this page." Choose "Adblock Plus doesn't block an advertisement" as issue type and proceed - you should get to the screenshot page.
- Now add one more filter subscription in Adblock Plus filter preferences so that you have five now, not counting "Adblock Warning Removal List."
- Repeat step 2, this time you should get a message telling you to remove some filter lists.
Attachments (0)
Change History (8)
comment:1 Changed on 06/24/2014 at 08:12:43 AM by philll
- Cc Ede_123 added
comment:2 Changed on 07/01/2014 at 12:55:32 PM by trev
- Platform set to Unknown
Five tiny lists still have five different maintainers. This limitation is about figuring out who is responsible, not about sifting through the list of filters.
comment:3 Changed on 07/28/2014 at 08:04:14 AM by saroyanm
- Owner set to saroyanm
comment:4 Changed on 07/28/2014 at 08:04:26 AM by saroyanm
- Status changed from new to reviewing
comment:6 Changed on 07/30/2014 at 11:02:21 AM by saroyanm
- Resolution set to fixed
- Status changed from reviewing to closed
comment:7 Changed on 07/30/2014 at 12:40:06 PM by trev
- Description modified (diff)
- Milestone set to Adblock-Plus-for-Firefox-next
- Platform changed from Unknown to Firefox/Firefox Mobile
comment:8 Changed on 05/20/2015 at 02:22:39 PM by philll
- Platform changed from Firefox/Firefox Mobile to Firefox
Made Firefox and Firefox mobile available as seperate platforms.
by Ede_123:
(I think my last comment was eaten by the SPAM filter...?)
While changing this behaviour you should also consider