Opened 4 years ago

Closed 4 years ago

#3885 closed change (fixed)

Disable the checkbox to toggle the Acceptable Ads list on the new options page

Reported by: sebastian Assignee: sebastian
Priority: P3 Milestone:
Module: User-Interface Keywords:
Cc: greiner, athornburgh Blocked By:
Blocking: #2372 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29339287

Description (last modified by sebastian)

Background

With the old options page, as well as with all of our other products, we don't let the user disable the Acceptable Ads list, but only remove it. That way we can rely on the number of requests downloading exceptionrules.txt, in order to estimate how many people use Acceptable Currently, with the new options page, however it would be possible to just disable the Acceptable Ads list in Advanced tab.

What to change

Disable the checkbox to toggle the Acceptable Ads list in the Advanced tab.

Change History (6)

comment:1 Changed 4 years ago by sebastian

  • Description modified (diff)
  • Summary changed from Don't allow disabling, but removing, the Acceptable Ads list on the new options page to Disable the checkbox to toggle the Acceptable Ads list on the new options page

comment:2 Changed 4 years ago by sebastian

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed 4 years ago by greiner

  • Cc athornburgh added
  • Priority changed from Unknown to P3
  • Ready set

This should be good enough for a temporary fix.

We should, however, keep investigating ways on how to treat the Acceptable Ads filter list in the Advanced tab to fulfill all of our requirements:

  • The Advanced tab should contain all installed filter lists.
  • It should be clear that the filter list can be removed.
  • The user should not be able to enable/disable the filter list (although we could expose a preference for that as we do in Firefox).

@athornburgh Any ideas in that regard?

comment:4 Changed 4 years ago by greiner

  • Blocking 2372 added

comment:5 Changed 4 years ago by abpbot

A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockplusui/rev/315d4d5afa2e

comment:6 Changed 4 years ago by sebastian

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.