Opened on 07/31/2017 at 08:35:48 AM
Closed on 08/25/2017 at 09:19:32 AM
Last modified on 09/25/2017 at 02:05:42 PM
#5459 closed change (fixed)
Add support to show a notification based on the number of ads blocked
Reported by: | wspee | Assignee: | wspee |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Core | Keywords: | |
Cc: | trev, sebastian, greiner | Blocked By: | |
Blocking: | #5457 | Platform: | Unknown / Cross platform |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
Description (last modified by wspee)
Background
For our upcoming cross promotion campaign we want to be able to show notifications to users that blocked a specific amount of ads.
What to change
Add support to show a notification only if the total number of ads blocked (Prefs.blocked_total) is greater than and/or less than a certain amount and only if the user didn't opt out of showing the number of ads blocked in the bubble ui (show_statsinpopup), as specified in #5457.
Attachments (0)
Change History (14)
comment:1 Changed on 07/31/2017 at 08:40:11 AM by wspee
- Description modified (diff)
- Summary changed from Add support to show a notification based on the total ads blocked number to Add support to show a notification based on the number of ads blocked
comment:3 Changed on 07/31/2017 at 03:16:26 PM by wspee
- Cc snoack trev added
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 07/31/2017 at 03:30:27 PM by wspee
- Blocking 5457 added
comment:5 Changed on 08/01/2017 at 03:14:57 PM by sebastian
- Cc sebastian added; snoack removed
comment:7 in reply to: ↑ 6 ; follow-up: ↓ 8 Changed on 08/18/2017 at 08:30:11 AM by wspee
Replying to greiner:
Note that we also have the preferences "show_statsinicon" and "show_statsinpopup" which I'd consider to be good indicators for users who are not interested in seeing such a notification.
So what about not showing this notification to users who have opted-out of seeing information about how many ads got blocked? (i.e. either of the two preferences is set to false) That way we could decrease the amount of negative backlash we might get from it and provide people with an easy way to avoid those kinds of notifications.
Hmm although this sounds like a good idea on first glance I'm not so sure about the correlation between users disabling the number of ads block and their interest in notifications. Perhaps they disabled the number in the icon because it's ugly, or they don't like to be distracted by a changing number?
This would further complicate the blocked total notification feature, mainly in things you have to keep in mind when designing or reasoning about notifications, and I'm no not sure if it's worth it?
Also the number of ads in the bubble ui "show_statsinpopup" seems to be not configurable anyways (at least in chrome?).
comment:8 in reply to: ↑ 7 Changed on 08/18/2017 at 04:08:44 PM by greiner
Replying to wspee:
Hmm although this sounds like a good idea on first glance I'm not so sure about the correlation between users disabling the number of ads block and their interest in notifications. Perhaps they disabled the number in the icon because it's ugly, or they don't like to be distracted by a changing number?
This would further complicate the blocked total notification feature, mainly in things you have to keep in mind when designing or reasoning about notifications, and I'm no not sure if it's worth it?
These are valid points. The underlying question appears to be: Do we prefer to show the notification to people who might not want to see it or to not show the notification to people who might want to see it?
Either option is fine with me as long as it's the one in favor of the user. I just wanted to bring up that this option exists since it wasn't mentioned before.
Also the number of ads in the bubble ui "show_statsinpopup" seems to be not configurable anyways (at least in chrome?).
I'm not sure whether this addresses your point but users can toggle the value for "show_statsinpopup" by clicking the "Ads blocked" menu item in the icon popup and they can toggle the value for "show_statsinicon" by clicking the "Show number in icon" checkbox in the icon popup.
comment:9 Changed on 08/21/2017 at 04:47:12 PM by jeen
Good points @greiner. And at this point, I would say to not send notifications to users who have opted out of "show_statsinpopup" - as this is a clear indicator for not wanting to see any information related to ad count. I would still allow notifications for users even if they have opted out of "show_statsinicon" - as @wspee pointed out they could have disabled this option because of distraction.
comment:10 Changed on 08/23/2017 at 10:42:11 AM by wspee
- Description modified (diff)
Updated description based on the above discussion.
comment:11 Changed on 08/25/2017 at 09:10:26 AM by abpbot
A commit referencing this issue has landed:
Issue 5459 - Add support to show a notification based on the number of ads blocked
comment:12 Changed on 08/25/2017 at 09:19:32 AM by wspee
- Resolution set to fixed
- Status changed from reviewing to closed
comment:13 Changed on 08/29/2017 at 06:47:30 AM by trev
- Priority changed from Unknown to P2
- Ready set
comment:14 Changed on 09/25/2017 at 02:05:42 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Done. This works in Chrome. Tested with variable values in notification and extension blocked total. Respects the stats_showinpopup preference. Unable to test in Opera at the moment due to #5759. Will test backend changes separately/when they land.
ABP 1.13.3.1838
Chrome 49 / 61 / Windows 7
Note that we also have the preferences "show_statsinicon" and "show_statsinpopup" which I'd consider to be good indicators for users who are not interested in seeing such a notification.
So what about not showing this notification to users who have opted-out of seeing information about how many ads got blocked? (i.e. either of the two preferences is set to false) That way we could decrease the amount of negative backlash we might get from it and provide people with an easy way to avoid those kinds of notifications.