From 19f1d99a985dbd55b442d2a346f612351bf1fcf6 Mon Sep 17 00:00:00 2001
From: Eric Arnold <ericarnold@google.com>
Date: Fri, 9 May 2014 16:30:35 -0700
Subject: [PATCH] Review fixes: Changed mousedown to click and changed event
 naming to use camelCase.  Removed unnecessary exports

---
 lib/ace/ext/linking.js | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/lib/ace/ext/linking.js b/lib/ace/ext/linking.js
index ccd3061e..cfd333b2 100644
--- a/lib/ace/ext/linking.js
+++ b/lib/ace/ext/linking.js
@@ -36,10 +36,10 @@ require("../config").defineOptions(Editor.prototype, "editor", {
     enableLinking: {
         set: function(val) {
             if (val) {
-                this.on("mousedown", onMouseDown);
+                this.on("click", onClick);
                 this.on("mousemove", onMouseMove);
             } else {
-                this.off("mousedown", onMouseDown);
+                this.off("click", onClick);
                 this.off("mousemove", onMouseMove);
             }
         },
@@ -57,11 +57,11 @@ function onMouseMove(e) {
         var session = editor.session;
         var token = session.getTokenAt(docPos.row, docPos.column);
 
-        editor._emit("link_hover", {position: docPos, token: token});
+        editor._emit("linkHover", {position: docPos, token: token});
     }
 }
 
-function onMouseDown(e) {
+function onClick(e) {
     var ctrl = e.getAccelKey();
     var button = e.getButton();
 
@@ -71,11 +71,8 @@ function onMouseDown(e) {
         var session = editor.session;
         var token = session.getTokenAt(docPos.row, docPos.column);
 
-        editor._emit("link", {position: docPos, token: token});
+        editor._emit("linkClick", {position: docPos, token: token});
     }
 }
 
-exports.onMouseMove = onMouseMove;
-exports.onMouseDown = onMouseDown;
-
 });