Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: chrome/content/composer.js

Issue 29357572: Issue 4536 - Make sure the elements stay hidden after adding the filter (Closed) Base URL: https://hg.adblockplus.org/elemhidehelper
Patch Set: Created Oct. 17, 2016, 9:58 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/content/composer.xul » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * This Source Code is subject to the terms of the Mozilla Public License 2 * This Source Code is subject to the terms of the Mozilla Public License
3 * version 2.0 (the "License"). You can obtain a copy of the License at 3 * version 2.0 (the "License"). You can obtain a copy of the License at
4 * http://mozilla.org/MPL/2.0/. 4 * http://mozilla.org/MPL/2.0/.
5 */ 5 */
6 6
7 let {Prefs} = require("prefs"); 7 let {Prefs} = require("prefs");
8 8
9 let domainData; 9 let domainData;
10 let nodeData; 10 let nodeData;
11 let nodeID; 11 let nodeID;
12 let selectedNode = null; 12 let selectedNode = null;
13 var advancedMode = false; 13 var advancedMode = false;
14 let treeView = null; 14 let treeView = null;
15 let ruleAdded = false;
15 let stylesheetData; 16 let stylesheetData;
16 let previewStyle = null;
saroyanm 2016/10/17 14:41:44 I assume it's irrelevant to the review, but redund
Wladimir Palant 2016/10/17 14:51:29 Yes, became redundant due to E10S adjustments and
17 17
18 let abpURL = Cc["@adblockplus.org/abp/public;1"].getService(Ci.nsIURI); 18 let abpURL = Cc["@adblockplus.org/abp/public;1"].getService(Ci.nsIURI);
19 Cu.import(abpURL.spec); 19 Cu.import(abpURL.spec);
20 20
21 /******************* 21 /*******************
22 * TreeView object * 22 * TreeView object *
23 *******************/ 23 *******************/
24 24
25 function TreeView(tree) { 25 function TreeView(tree) {
26 var origView = tree.view; 26 var origView = tree.view;
(...skipping 537 matching lines...) Expand 10 before | Expand all | Expand 10 after
564 .getItemAtIndex(min.value); 564 .getItemAtIndex(min.value);
565 if (!item || !item.nodeData) 565 if (!item || !item.nodeData)
566 return; 566 return;
567 567
568 fillAttributes(item.nodeData); 568 fillAttributes(item.nodeData);
569 } 569 }
570 570
571 function addExpression() 571 function addExpression()
572 { 572 {
573 AdblockPlus.addPatterns([document.getElementById("expression").value]); 573 AdblockPlus.addPatterns([document.getElementById("expression").value]);
574 ruleAdded = true;
575 }
574 576
575 togglePreview(false); 577 window.addEventListener("unload", () =>
576 } 578 {
579 togglePreview(ruleAdded, true);
580 });
OLDNEW
« no previous file with comments | « no previous file | chrome/content/composer.xul » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld