From 5aea24eb617277404891de4d49679f362969b872 Mon Sep 17 00:00:00 2001
From: Kris Maglione <maglione.k@gmail.com>
Date: Thu, 27 Nov 2008 08:21:07 +0000
Subject: [PATCH] Make substring preview less quirky

---
 content/io.js |  4 ++--
 content/ui.js | 37 ++++++++++++++++++++++++++-----------
 2 files changed, 28 insertions(+), 13 deletions(-)

diff --git a/content/io.js b/content/io.js
index a166c178..a5fcfb6c 100644
--- a/content/io.js
+++ b/content/io.js
@@ -776,7 +776,7 @@ lookup:
         // no need (actually forbidden) to add: js <<EOF ... EOF around those files
         source: function (filename, silent)
         {
-            var wasSourcing = ioManager.sourcing;
+            let wasSourcing = ioManager.sourcing;
             try
             {
                 var file = ioManager.getFile(filename);
@@ -808,7 +808,7 @@ lookup:
                 // handle pure javascript files specially
                 if (/\.js$/.test(filename))
                 {
-                    var loader = Components.classes["@mozilla.org/moz/jssubscript-loader;1"]
+                    let loader = Components.classes["@mozilla.org/moz/jssubscript-loader;1"]
                                            .getService(Components.interfaces.mozIJSSubScriptLoader);
                     try
                     {
diff --git a/content/ui.js b/content/ui.js
index ae5b90dd..4f0a7ff0 100644
--- a/content/ui.js
+++ b/content/ui.js
@@ -306,6 +306,30 @@ function CommandLine() //{{{
         promptWidget.setAttributeNS(NS.uri, "highlight", highlightGroup || commandline.HL_NORMAL);
     }
 
+    function previewSubstring()
+    {
+        if (!options.get("wildoptions").has("auto"))
+            return;
+        // Kludge. Major kludge.
+        let editor = commandWidget.inputField.editor;
+        try
+        {
+            let node = editor.rootElement;
+            editor.deleteNode(node.firstChild.nextSibling);
+        }
+        catch (e) {}
+        let wildmode = options.get("wildmode");
+        let wildType = wildmode.values[Math.min(wildIndex, wildmode.values.length - 1)];
+        if (wildmode.checkHas(wildType, "longest"))
+        {
+            let start = commandWidget.selectionStart;
+            let substring = completionContext.longestAllSubstring.substr(start - completionContext.allItems.start);
+            removeSuffix = substring;
+            editor.insertNode(util.xmlToDom(<span style="color: gray">{substring}</span>, document), editor.rootElement, 1);
+            commandWidget.selectionStart = commandWidget.selectionEnd = start;
+        }
+    }
+
     // sets the command - e.g. 'tabopen', 'open http://example.com/'
     function setCommand(cmd)
     {
@@ -827,17 +851,7 @@ function CommandLine() //{{{
             else if (event.type == "input")
             {
                 liberator.triggerCallback("change", currentExtendedMode, command);
-                // Kludge. Major kludge.
-                let wildmode = options.get("wildmode");
-                let wildType = wildmode.values[Math.min(wildIndex, wildmode.values.length - 1)];
-                if (wildmode.checkHas(wildType, "longest"))
-                {
-                    let start = commandWidget.selectionStart;
-                    let substring = completionContext.longestAllSubstring.substr(start - completionContext.allItems.start);
-                    removeSuffix = substring;
-                    editor.insertNode(util.xmlToDom(<span style="color: gray">{substring}</span>, document), editor.rootElement, 1);
-                    commandWidget.selectionStart = commandWidget.selectionEnd = start;
-                }
+                previewSubstring();
             }
             else if (event.type == "keypress")
             {
@@ -1260,6 +1274,7 @@ function CommandLine() //{{{
             // prefix when wrapping around searches
             // with that, we SOMETIMES have problems with <tab> followed by <s-tab> in :open completions
 
+            previewSubstring();
             let command = this.getCommand();
             if (command.substr(command.length - removeSuffix.length) == removeSuffix)
                 command = command.substr(0, command.length - removeSuffix.length)