Unified Diff: background.js
Issue 29329095:
Issue 2747 - Added empty state for tables in options page (Closed)
Patch Set:
Created Oct. 13, 2015, 2:24 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
locale/en-US/options.json » ('j')
|
options.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: background.js |
| =================================================================== |
| --- a/background.js |
| +++ b/background.js |
| @@ -176,7 +176,9 @@ |
| if (params.filterError) |
| return {errors: ["Invalid filter"]}; |
| return { |
| - filters: text.split("\n").map(modules.filterClasses.Filter.fromText), |
| + filters: text.split("\n") |
| + .filter(function(filter) {return !!filter;}) |
|
saroyanm
2015/11/30 11:24:21
nice.
|
| + .map(modules.filterClasses.Filter.fromText), |
| errors: [] |
| }; |
| } |
« no previous file with comments
|
« no previous file
|
locale/en-US/options.json » ('j')
|
options.html » ('J')
