Unified Diff: lib/elemHide.js
Issue 29342837:
Issue 4054 - Remove filters from filtersByDomain (Closed)
Patch Set: Corrected mistake in code ordering
Created May 20, 2016, 12:48 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/elemHide.js |
| diff --git a/lib/elemHide.js b/lib/elemHide.js |
| index 6e2f1b56acbb6e67e79f779503b469f9d8829903..1013f90da5b1a0c5039400e1ab6cbf22b0e797d4 100644 |
| --- a/lib/elemHide.js |
| +++ b/lib/elemHide.js |
| @@ -201,6 +201,17 @@ var ElemHide = exports.ElemHide = |
| delete filterByKey[key]; |
| delete keyByFilter[filter.text]; |
| ElemHide.isDirty = true; |
| + |
| + if (usingFiltersByDomain) |
| + { |
| + let domains = Object.keys(filter.domains || defaultDomains); |
| + for (let domain of domains) |
|
Wladimir Palant
2016/05/20 13:12:34
So how about making this code consistent to the on
kzar
2016/05/20 13:14:25
Done.
|
| + { |
| + let filters = filtersByDomain[domain]; |
| + if (filters) |
| + delete filters[filter.text]; |
| + } |
| + } |
| } |
| }, |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
