Opened on 05/20/2015 at 06:17:51 PM

Closed on 09/05/2019 at 05:29:03 PM

#2565 closed change (duplicate)

Add "share filters" option to the "Add Filter(s)?" dialog box in ABP for Chrome

Reported by: nicole Assignee:
Priority: P3 Milestone:
Module: User-Interface Keywords:
Cc: sven, greiner, saroyanm Blocked By:
Blocking: #2535, #2566 Platform: Chrome
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by greiner)


Currently, when users add manual filters we don't track this. In order to proactively improve the quality of filter lists, when a user manually adds a filter we offer the user an option to share with Eyeo, and the manual filter add is captured and stored.

Meta Issue: #2535

What to change

  • Add checkbox to the "Add Filter(s)?" dialog box to allow users to share the filter information with ABP.
  • When the checkbox is checked, clicking "Add" should capture and transmit the following information in addition to adding the filter:
    • Filter text of newly created filters separated by newline character
    • Active filter subscriptions
    • Webpage URL (should be anonymized by removing parameter values)
    • Extension, browser and platform info
  • Fail silently if an error occurs but make sure to log the error nevertheless.

The data should be transmitted to the existing issue reports backend (report_submiturl preference: where %GUID% is generated randomly). Method is POST, with headers Content-Type: text/xml and X-Adblock-Plus: 1. The data should look like this:

<report type="addfilter">
<adblock-plus name="adblockpluschrome" version="1.9" locale="en-US"/>
<application name="chrome" vendor="Google Inc." version="43.0" userAgent="Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.124 Safari/537.36"/>
<platform name="chromium" version="43.0"/>
<subscription id="" disabledFilters="0" version="201506101551" lastDownloadAttempt="-77705" lastDownloadSuccess="-77705" softExpiration="203473" hardExpiration="613495" downloadStatus="synchronize_ok"/>
<subscription id="" disabledFilters="0" version="201506100930" lastDownloadAttempt="-101133" lastDownloadSuccess="-101133" softExpiration="-5815" hardExpiration="71667" downloadStatus="synchronize_ok"/>
<window url="*&param2=*" referrer=""/>

Attachments (2)

addfilter.png (13.7 KB) - added by nicole on 05/20/2015 at 06:23:10 PM.
Dialog Box
add filter dialogue 1.jpg (45.6 KB) - added by sven on 05/22/2015 at 09:58:45 AM.

Download all attachments as: .zip

Change History (17)

Changed on 05/20/2015 at 06:23:10 PM by nicole

Dialog Box

comment:1 Changed on 05/20/2015 at 06:24:38 PM by nicole

  • Blocked By 2535 added

comment:2 Changed on 05/20/2015 at 06:26:40 PM by nicole

  • Blocked By 2535 removed

comment:3 Changed on 05/21/2015 at 10:47:50 AM by greiner

  • Blocking 2535 added

comment:4 Changed on 05/21/2015 at 12:43:23 PM by greiner

  • Cc sven greiner added
  • Component changed from Unknown to User-Interface
  • Description modified (diff)

What's the reason for including which browser a filter was created on? Note that we're automatically getting that information anyway but depending on your use-case, other environment information might be more useful.

@sven What do you think about the suggested UI? I'd imagine there might be issues with other languages.

Changed on 05/22/2015 at 09:58:45 AM by sven

comment:5 Changed on 05/22/2015 at 10:00:52 AM by sven

@greiner, thats right.

I created a quick mock-up which should solve the problem.

(The buttons don't need to be blue, but i saw that they are in a grey which could be seen as inactive. We will tackle this problem in another issue)

Last edited on 05/22/2015 at 10:01:57 AM by sven

comment:6 Changed on 06/01/2015 at 08:45:02 AM by saroyanm

  • Cc saroyanm added

comment:7 Changed on 06/02/2015 at 07:51:22 AM by philll

  • Summary changed from Additions to the "Add Filter(s)?" dialog box to Add "share filters" option to the "Add Filter(s)?" dialog box in ABP for Chrome

comment:8 Changed on 06/11/2015 at 01:58:49 PM by trev

  • Description modified (diff)
  • Platform changed from Unknown to Chrome

I added some more info to the description - this data is supposed to be sent to the existing reports backend, consequently the format is fixed. The example data is mostly identical to what the issue reporter in Firefox is sending (note that all timestamps related to subscriptions are being sent as relative values). The difference is that some data is missing, also the name attribute on the <adblock-plus> tag is new.

comment:9 Changed on 06/22/2015 at 04:07:50 PM by greiner

  • Priority changed from Unknown to P3

I got a couple more questions:

  • How is the data supposed to look if multiple filters are added at the same time? Should we include multiple <comment> tags?
  • What should happen if we fail to send the data? Should we show an error message, try to send the data at a later point again or not send it at all?
  • Users can also add custom filters via the options page. Is the plan to create a separate ticket for adding that checkbox to the new options page as well or was this intentionally left out?

comment:10 Changed on 06/22/2015 at 04:19:50 PM by nicole

Multiple filters should be split into separate records.

If the data send fails, I think we should just not send it at all. Since this is optional information, we don't want to bother the user if the send fails, unless we could have the send try again somehow in the background (not impacting user).

That's a good idea for the new options page, should I create a new issue and link to this as the meta issue? Or should the new options page be the meta?

comment:11 Changed on 06/30/2015 at 07:14:14 PM by trev

  • Tester set to Unknown

Not so sure, I'd rather generate a single report for multiple filters - these belong together. I'd simply create a multi-line comment then.

I agree that a failure to send the data isn't important here.

Adding filters on the options page is a different thing - we don't have any context information whatsoever, meaning that the report won't be of much value. IMHO not worth doing.

comment:12 Changed on 07/01/2015 at 03:01:47 PM by greiner

  • Description modified (diff)
  • Ready set

Modified the description accordingly. I think a multi-line comment makes sense, agreed, to keep the payload size low.

@nicole Could you please find out whether there's value in having this in the options page or whether the lack of context makes those filters useless for us? Feel free to create a separate issue if it turns out to be sufficient.

comment:13 Changed on 07/20/2017 at 09:57:45 AM by wspee

  • Blocking 2566 added

comment:14 Changed on 12/21/2017 at 11:29:28 AM by fhd

  • Cc trev removed

comment:15 Changed on 09/05/2019 at 05:29:03 PM by greiner

  • Resolution set to duplicate
  • Status changed from new to closed

Closing this ticket in favor of ui#36.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Note: See TracTickets for help on using tickets.