Integrated URL Fixer into Adblock Plus

chrome/content/ui/filters.js

Index: chrome/content/ui/filters.js
===================================================================
--- a/chrome/content/ui/filters.js
+++ b/chrome/content/ui/filters.js
@@ -15,6 +15,8 @@
     if (filter instanceof Filter)
       Utils.runAsync(SubscriptionActions.selectFilter, SubscriptionActions, filter);
   }
+  
+  TypoActions.init();

[Wladimir Palant, 2012/11/09 08:26:15]

Please don't initialize TypoActions explicitly here - they can be initialized in
their own file, like Backup or FilterActions or FilterSearch:

window.addEventListener("load", function()
{
  TypoActions.init();
}, false);

 }
 
 /**
@@ -83,7 +85,12 @@
 
   let list = panel.getElementsByTagName("richlistbox")[0];
   if (!list)
+  {
+    E("filtersContainer").style.visibility = "hidden";
     return;
+  }
+  
+  E("filtersContainer").style.visibility = "visible";

[Wladimir Palant, 2012/11/06 15:48:03]

a) Please use |hidden = true| or at least |collapsed = true| rather than
manipulating styles.
b) You should hide filtersSplitter element as well, that should make sure that
the tabbox can expand to the right.

 
   let data = Templater.getDataForNode(list.selectedItem);
   FilterView.subscription = (data ? data.subscription : null);