From ee8ec26a6b4f3996bd94903b435ecafd30648c2d Mon Sep 17 00:00:00 2001
From: Jez Ng
Date: Sat, 4 Aug 2012 12:23:56 -0700
Subject: Miscellaneous cleanups.

* " -> " because my syntax highlighting gets confused
* Remove inline onclick handlers as part of manifest v2 upgrade
---
 background_scripts/commands.coffee | 2 +-
 options.html                       | 6 +++---
 options.js                         | 3 +++
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/background_scripts/commands.coffee b/background_scripts/commands.coffee
index 1180ef3f..0fa56e1a 100644
--- a/background_scripts/commands.coffee
+++ b/background_scripts/commands.coffee
@@ -13,7 +13,7 @@ Commands =
   #    command passed to it. This is used to implement e.g. "closing of 3 tabs".
   addCommand: (command, description, options) ->
     if command of @availableCommands
-      console.log(command, "is already defined! Check commands.js for duplicates.")
+      console.log(command, "is already defined! Check commands.coffee for duplicates.")
       return
 
     options ||= {}
diff --git a/options.html b/options.html
index 52dbdef2..730c9bff 100644
--- a/options.html
+++ b/options.html
@@ -126,7 +126,7 @@
                 map j scrollDown<br/>
                 unmap j<br/>
                 unmapAll<br/>
-                " this is a comment<br/>
+                &quot; this is a comment<br/>
                 # this is also a comment<br/>
               </div>
               <a href="#" id="showCommands">Show available commands.</a>
@@ -214,8 +214,8 @@
     </table>
 
     <div id="buttonsPanel">
-      <button id="restoreSettings" onclick="restoreToDefaults()">Restore to Defaults</button>
-      <button id="saveOptions" disabled="true" onclick="saveOptions()">Save Options</button>
+      <button id="restoreSettings">Restore to Defaults</button>
+      <button id="saveOptions" disabled="true">Save Options</button>
     </div>
 
     <br/>
diff --git a/options.js b/options.js
index ff5427a8..bc624882 100644
--- a/options.js
+++ b/options.js
@@ -29,6 +29,9 @@ document.addEventListener("DOMContentLoaded", function() {
     showHelpDialog(
       chrome.extension.getBackgroundPage().helpDialogHtml(true, true, "Command Listing"), frameId);
   }, false);
+
+  document.getElementById("restoreSettings").addEventListener("click", restoreToDefaults);
+  document.getElementById("saveOptions").addEventListener("click", saveOptions);
 });
 
 function onOptionKeyup(event) {
-- 
cgit v1.2.3