Integrated URL Fixer into Adblock Plus

chrome/content/ui/filters.xul

Index: chrome/content/ui/filters.xul
===================================================================
--- a/chrome/content/ui/filters.xul
+++ b/chrome/content/ui/filters.xul
@@ -28,6 +28,7 @@
 <script type="application/x-javascript;version=1.7" src="filters-filteractions.js"/>
 <script type="application/x-javascript;version=1.7" src="filters-backup.js"/>
 <script type="application/x-javascript;version=1.7" src="filters-search.js"/>
+<script type="application/x-javascript;version=1.7" src="typo.js"/>
 
 <keyset id="filtersKeyset">
   <key id="subscription-update-key" key="T" modifiers="accel" command="subscription-update-command"/>
@@ -131,6 +132,7 @@
     <tabs onselect="onTabChange(this.parentNode);">
       <tab label="&subscriptions.tab.label;"/>
       <tab label="&filters.tab.label;"/>
+      <tab label="&typoCorrections.tab.label;"/>
     </tabs>
     <tabpanels flex="1">
       <tabpanel id="subscriptionsTab" orient="vertical" flex="1">
@@ -303,6 +305,22 @@
           </richlistbox>
         </deck>
       </tabpanel>
+      <tabpanel id="typoCorrectionsTab" orient="vertical" flex="1">
+        <checkbox id="typo_enable" label="&typoCorrections.enableLabel;" oncommand="TypoActions.toggleEnable(this.checked)"/>

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

This isn't a "toggle" action if you pass in the value explicitly ;)
How about TypoActions.setEnabled()?

+        <groupbox id="typo_whitelist_container" flex="1">
+          <listbox id="typo_whitelist" class="initialFocus" flex="1" seltype="multiple" onselect="TypoActions.onItemSelected(this);" _removeButton="typo_whitelist_remove-button" _emptyLabel="&typoCorrections.whitelistEmptyLabel;">
+            <listhead>
+              <listheader label="&typoCorrections.whitelistLabel;"/>
+            </listhead>
+            <listcols>
+              <listcol flex="1" />
+            </listcols>
+          </listbox>
+          <hbox>
+            <button flex="1" label="&typoCorrections.removeButtonLabel;" onclick="TypoActions.removeRule(this,'whitelist');" disabled="true" id="typo_whitelist_remove-button" _list="typo_whitelist" />

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

There should be a space after the comma here.

Please use <hbox pack="right"> and remove the flex attribute - this will
position the button on the right side which is more in line with how the Adblock
Plus preferences look like.

+          </hbox>
+        </groupbox>
+      </tabpanel>
     </tabpanels>
   </tabbox>