Opened on 07/23/2014 at 11:10:00 AM
Closed on 08/20/2015 at 02:12:09 PM
Last modified on 09/02/2015 at 08:18:31 AM
#1109 closed change (fixed)
Support notifications
Reported by: | fhd | Assignee: | sergz |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Adblock-Plus-for-Internet-Explorer | Keywords: | |
Cc: | oleksandr | Blocked By: | #1557, #2810 |
Blocking: | Platform: | Internet Explorer | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by sergz)
Background
#1107 will add support for notifications to libadblockplus.
What to change
Use the new API to show notifications. We have a range of options for how this can look in IE, something similar to how it looks in Firefox would probably be best.
Note for testers
In addition to the notification functionality please pay attention to the uninstalling (it also happens during the update procedure) of ABP for IE. The change in this issue can affect the termination of the engine (background ABP process, AdblockPlusEngine.exe).
To test the notification functionality one can use Fiddler and modify OnBeforeResponse handler by adding
if (oSession.uriContains("notification.json")) { var respBody = '{"notifications": [{"id": "myId", "title": "Update Adblock Plus", "message": "A new version of Adblock Plus is ready to be installed. Do you want to <a>install it now</a>?", "links": ["http://adblockplus.org"]}],"version": "201502031430" }'; oSession.utilSetResponseBody(respBody); }
Take into account that the first download notification request is executed in about one minute if there is nothing about notifications in prefs.json.
While testing the notification please pay attention that
- there is no any prompting dialog to launch AdblockPlusEngine.exe (Now it always runs at Meidum IL, one can ensure about that using some advanced task manager like ProcessExplorer from Windows Sysinternals)
- the link in the notification is opened on mouse-click
- the notification window is correctly displayed on different DPI settings
- it works on the whole range of platforms Windows[XP-10] x IE[8-11].
Attachments (0)
Change History (7)
comment:1 Changed on 02/17/2015 at 11:56:40 PM by oleksandr
- Blocked By 1557 added
comment:2 Changed on 04/20/2015 at 09:08:14 AM by sergz
- Owner set to sergz
comment:3 Changed on 05/11/2015 at 10:07:18 AM by sergz
- Cc oleksandr added
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 07/22/2015 at 01:23:52 PM by sergz
- Blocked By 2810 added
comment:5 Changed on 08/20/2015 at 02:11:45 PM by sergz
- Blocked By 1107 removed
- Tester set to Unknown
Remove #1107 because it's already not blocking and having it opened one cannot close this issue.
comment:6 Changed on 08/20/2015 at 02:12:09 PM by sergz
- Resolution set to fixed
- Status changed from reviewing to closed
Notification requests are now made by ABP for IE since #1557 is fixed now. We now need to implement some UI to display notifications to the user.