Miscellaneous cleanups.

* " -> " because my syntax highlighting gets confused
* Remove inline onclick handlers as part of manifest v2 upgrade

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) {