Issue 1086 - Add acceptable ads checkbox to the settings page

html/static/js/IESettings.js

Index: html/static/js/IESettings.js
===================================================================
--- a/html/static/js/IESettings.js
+++ b/html/static/js/IESettings.js
@@ -8,7 +8,7 @@
   }
   catch (err)
   {
-    //alert("err: " + err);
+//    alert("err: " + err);

[Felix Dahlke, 2014/07/22 14:28:10]

That change seems unrelated :D

   }
 }
 
@@ -25,6 +25,12 @@
   }
 }
 
+function acceptableAdsChange()
+{
+  var aaCheckbox = document.getElementById("acceptableAdsCheckbox");
+  UserSettings().SetAcceptableAdsEnabled(aaCheckbox.checked);  
+}
+
 function initLanguageSettings()
 {
   setElementText("title", "settings-title");
@@ -40,7 +46,16 @@
   setElementText("removeDomains", "settings-exceptions-remove-label");
 
   setElementText("localeWorthSharing", "settings-share-label");
+
+  setElementText("acceptableAdsLabel", "settings-acceptable-ads");
    
+  var aaCheckbox = document.getElementById("acceptableAdsCheckbox");
+  if (aaCheckbox)
+  {
+    aaCheckbox.checked = UserSettings().IsAcceptableAdsEnabled();
+    addListener(aaCheckbox, "change", acceptableAdsChange, false);
+  }
+
   var optionsLanguage = document.getElementById("language");
 
   var languageCount = UserSettings().GetLanguageCount();
@@ -147,4 +162,4 @@
 window.UserSettings = function()
 {
   return window.Settings;
-}
+}

[Felix Dahlke, 2014/07/22 14:28:10]

Was this a missing EOF? If so let's keep it.