Opened on 03/23/2016 at 10:59:56 PM
Closed on 03/24/2016 at 03:22:50 PM
Last modified on 05/26/2016 at 12:34:54 PM
#3869 closed change (fixed)
Migrate the UI to the new FilterNotifier API
Reported by: | sebastian | Assignee: | sebastian |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | User-Interface | Keywords: | |
Cc: | greiner | Blocked By: | |
Blocking: | #3868 | Platform: | Unknown / Cross platform |
Ready: | yes | Confidential: | no |
Tester: | Ross | Verified working: | yes |
Review URL(s): |
Description
Background
With #3862, the FilterNotifier API got refactored and is now migrating to an event API similar to node.js. While the old methods are still supported, they are deprecated now, and the new API should also perform better.
What to change
Adapt existing code to use FilterNofier.on(), FilterNofier.off() and FilterNofier.emit().
Attachments (0)
Change History (6)
comment:1 Changed on 03/23/2016 at 11:01:39 PM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 03/24/2016 at 10:59:22 AM by greiner
- Priority changed from Unknown to P3
comment:3 Changed on 03/24/2016 at 11:18:15 AM by sebastian
- Blocking 3868 added
comment:4 Changed on 03/24/2016 at 03:20:24 PM by abpbot
comment:5 Changed on 03/24/2016 at 03:22:50 PM by sebastian
- Resolution set to fixed
- Status changed from reviewing to closed
comment:6 Changed on 05/26/2016 at 12:34:54 PM by Ross
- Tester changed from Unknown to Ross
- Verified working set
Subscriptions and filters are working without any obvious regressions.
ABP 1.11.0.1606
Chrome 31, 45, 50 / Windows 7
Chrome 32, 49 / OS X 10.11
Chrome 49 / Ubuntu 14.04
Safari 6 / OS X 10.8
Safari 9.1 / OS X 10.11
Opera 36 / Ubuntu 14.04
A commit referencing this issue has landed:
https://hg.adblockplus.org/adblockplusui/rev/9b91cdef871c