From 28e15cb66cef7e90b42cf11b0ac3ec6c0164a18d Mon Sep 17 00:00:00 2001
From: Doug Kearns <dougkearns@gmail.com>
Date: Mon, 17 Sep 2007 05:37:29 +0000
Subject: [PATCH] enable vimperator.events.toString() to return keysyms for
 mouse events

---
 chrome/content/vimperator/events.js      | 76 +++++++++++++++++-------
 chrome/content/vimperator/ui.js          |  7 +--
 chrome/content/vimperator/vimperator.xul |  4 +-
 3 files changed, 58 insertions(+), 29 deletions(-)

diff --git a/chrome/content/vimperator/events.js b/chrome/content/vimperator/events.js
index 52615429..b5ac3af5 100644
--- a/chrome/content/vimperator/events.js
+++ b/chrome/content/vimperator/events.js
@@ -317,6 +317,7 @@ function Events() //{{{
 
         var key = null;
         var modifier = "";
+
         if (event.ctrlKey)
             modifier += "C-";
         if (event.altKey)
@@ -324,33 +325,61 @@ function Events() //{{{
         if (event.metaKey)
             modifier += "M-";
 
-        if (event.charCode == 0)
+        if (event.type == "keypress")
         {
-            if (event.shiftKey)
-                modifier += "S-";
-
-            for (var i in keyTable)
+            if (event.charCode == 0)
             {
-                if (keyTable[i][0] == event.keyCode)
+                if (event.shiftKey)
+                    modifier += "S-";
+
+                for (var i in keyTable)
                 {
-                    key = keyTable[i][1][0];
-                    break;
+                    if (keyTable[i][0] == event.keyCode)
+                    {
+                        key = keyTable[i][1][0];
+                        break;
+                    }
                 }
             }
+            // special handling of the Space key
+            else if (event.charCode == 32)
+            {
+                if (event.shiftKey)
+                    modifier += "S-";
+                key = "Space";
+            }
+            // a normal key like a, b, c, 0, etc.
+            else if (event.charCode > 0)
+            {
+                key = String.fromCharCode(event.charCode);
+                if (modifier.length == 0)
+                    return key;
+            }
+
+            if (key == null)
+                return null;
+
         }
-        // special handling of the Space key
-        else if (event.charCode == 32)
+        else if (event.type == "click" || event.type == "dblclick")
         {
             if (event.shiftKey)
                 modifier += "S-";
-            key = "Space";
-        }
-        // a normal key like a, b, c, 0, etc.
-        else if (event.charCode > 0)
-        {
-            key = String.fromCharCode(event.charCode);
-            if (modifier.length == 0)
-                return key;
+            if (event.type == "dblclick")
+                modifier += "2-";
+            // TODO: triple and quadruple click
+
+            switch (event.button)
+            {
+                case 0:
+                    key = "LeftMouse"
+                    break;
+                case 1:
+                    key = "MiddleMouse"
+                    break;
+                case 2:
+                    key = "RightMouse"
+                    break;
+            }
         }
 
         if (key == null)
@@ -358,6 +387,7 @@ function Events() //{{{
 
         // a key like F1 is always enclosed in < and >
         return "<" + modifier + key + ">";
+
     } //}}}
 
     this.isAcceptKey = function(key)
@@ -432,13 +462,13 @@ function Events() //{{{
     {
         if (!vimperator.modes.passNextKey)
         {
-            if(vimperator.modes.passAllKeys)
+            if (vimperator.modes.passAllKeys)
             {
                 vimperator.modes.passAllKeys = false;
                 return;
             }
 
-            switch(vimperator.mode)
+            switch (vimperator.mode)
             {
                 case vimperator.modes.VISUAL:
                     if (vimperator.modes.extended & vimperator.modes.TEXTAREA)
@@ -454,7 +484,7 @@ function Events() //{{{
                     break;
 
                 case vimperator.modes.INSERT:
-                    if((vimperator.modes.extended & vimperator.modes.TEXTAREA) && !vimperator.options["insertmode"])
+                    if ((vimperator.modes.extended & vimperator.modes.TEXTAREA) && !vimperator.options["insertmode"])
                         vimperator.mode = vimperator.modes.TEXTAREA;
                     else
                         vimperator.modes.reset();
@@ -828,13 +858,13 @@ function Events() //{{{
 
         unregister: function()
         {
-            if(!this._branch) return;
+            if (!this._branch) return;
             this._branch.removeObserver("", this);
         },
 
         observe: function(aSubject, aTopic, aData)
         {
-            if(aTopic != "nsPref:changed") return;
+            if (aTopic != "nsPref:changed") return;
             // aSubject is the nsIPrefBranch we're observing (after appropriate QI)
             // aData is the name of the pref that's been changed (relative to aSubject)
             switch (aData)
diff --git a/chrome/content/vimperator/ui.js b/chrome/content/vimperator/ui.js
index f5213507..094688b3 100644
--- a/chrome/content/vimperator/ui.js
+++ b/chrome/content/vimperator/ui.js
@@ -631,10 +631,6 @@ function CommandLine() //{{{
 
         var key = vimperator.events.toString(event);
 
-        // TODO: move to events.toString()
-        if (event.type == "click" && event.button == 0)
-            key = "<LeftMouse>"
-
         switch (key)
         {
             case ":":
@@ -680,6 +676,9 @@ function CommandLine() //{{{
                 break;
 
             case "<LeftMouse>":
+            case "<A-LeftMouse>":
+            case "<C-LeftMouse>":
+            case "<S-LeftMouse>":
                 if (/^(end|more(-help)?)-prompt$/.test(event.target.id))
                     ; // fall through
                 else
diff --git a/chrome/content/vimperator/vimperator.xul b/chrome/content/vimperator/vimperator.xul
index 5a1f8118..1c71830e 100644
--- a/chrome/content/vimperator/vimperator.xul
+++ b/chrome/content/vimperator/vimperator.xul
@@ -94,8 +94,8 @@ the terms of any one of the MPL, the GPL or the LGPL.
                     </listcols>
                 </listbox>
 
-                <iframe id="vimperator-multiline-output" src="about:blank" flex="1" height="10px" hidden="false" collapsed="true" onclick="vimperator.commandline.onMultilineOutputEvent(event)"/>
-                        <!--onkeypress="vimperator.commandline.onMultilineOutputEvent(event)"/-->
+                <iframe id="vimperator-multiline-output" src="about:blank" flex="1" height="10px" hidden="false" collapsed="true"
+                    onclick="vimperator.commandline.onMultilineOutputEvent(event)"/>
 
                 <hbox id="vimperator-commandline" flex="1" hidden="false">
                     <label class="plain" id="vimperator-commandline-prompt"  flex="0" crop="end" value="" collapsed="true"/>