Integrated URL Fixer into Adblock Plus

chrome/content/ui/filters.js

 /*
  * This Source Code is subject to the terms of the Mozilla Public License
  * version 2.0 (the "License"). You can obtain a copy of the License at
  * http://mozilla.org/MPL/2.0/.
  */
 
 /**
  * Initialization function, called when the window is loaded.
  */
 function init()
 {
   if (window.arguments && window.arguments.length)
   {
     let filter = window.arguments[0].wrappedJSObject;
     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);

 }
 
 /**
  * Called whenever the currently selected tab changes.
  */
 function onTabChange(/**Element*/ tabbox)
 {
   updateSelectedSubscription();
 
   Utils.runAsync(function()
@@ skipping 48 matching lines @@
  * Updates filter list when selected subscription changes.
  */
 function updateSelectedSubscription()
 {
   let panel = E("tabs").selectedPanel;
   if (!panel)
     return;
 
   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);
   FilterActions.updateCommands();
 }
 
 /**
  * Template processing functions.
  * @class
  */
@@ skipping 104 matching lines @@
    * data object.
    */
   getNodeForData: function(/**Node*/ parent, /**String*/ property, /**Object*/ data) /**Node*/
   {
     for (let child = parent.firstChild; child; child = child.nextSibling)
       if ("_data" in child && property in child._data && child._data[property] == data)
         return child;
     return null;
   }
 };