beautify completer.js

IPython/frontend/html/notebook/static/js/completer.js

@@ -1,7 +1,6 b''
 // function completer.
 //
 // completer should be a class that take an cell instance
-
 var IPython = (function(IPython ) {
     // that will prevent us from misspelling
     "use strict";
@@ -10,11 +9,14 b' var IPython = (function(IPython ) {'
     var key = IPython.utils.keycodes;
 
     // what is the common start of all completions
+
+
     function sharedStart(B){
-            if(B.length == 1){return B[0];}
+        if (B.length == 1) {
+            return B[0];
+        }
             var A = new Array();
-            for(var i=0; i< B.length; i++)
-            {
+        for (var i = 0; i < B.length; i++) {
                A.push(B[i].str);
             }
             if(A.length > 1 ){
@@ -26,8 +28,11 b' var IPython = (function(IPython ) {'
                 while(s && tem2.indexOf(tem1) == -1){
                     tem1 = tem1.substring(0, --s);
                 }
-                if (tem1 == ""  || tem2.indexOf(tem1) != 0){return null;}
-                return { str  : tem1,
+            if (tem1 == "" || tem2.indexOf(tem1) != 0) {
+                return null;
+            }
+            return {
+                str: tem1,
                          type : "computed",
                          from : B[0].from,
                          to   : B[0].to
@@ -52,8 +57,7 b' var IPython = (function(IPython ) {'
 
 
 
-    Completer.prototype.startCompletion = function()
-    {
+    Completer.prototype.startCompletion = function() {
         // call for a 'first' completion, that will set the editor and do some
         // special behaviour like autopicking if only one completion availlable
         //
@@ -67,18 +71,24 b' var IPython = (function(IPython ) {'
         // Pass true as parameter if you want the commpleter to autopick when
         // only one completion. This function is automatically reinvoked at
         // each keystroke with ff = false
-
         var cur = this.editor.getCursor();
         var line = this.editor.getLine(cur.line);
-        var pre_cursor = this.editor.getRange({line:cur.line,ch:cur.ch-1},cur);
+        var pre_cursor = this.editor.getRange({
+            line: cur.line,
+            ch: cur.ch - 1
+        }, cur);
 
         // we need to check that we are still on a word boundary
         // because while typing the completer is still reinvoking itself
-        if(!/[0-9a-z._]/i.test(pre_cursor)){ this.close(); return;}
+        if (!/[0-9a-z._]/i.test(pre_cursor)) {
+            this.close();
+            return;
+        }
 
         this.autopick = false;
-        if( ff != 'undefined' && ff==true)
-        { this.autopick=true; }
+        if (ff != 'undefined' && ff == true) {
+            this.autopick = true;
+        }
 
         // We want a single cursor position.
         if (this.editor.somethingSelected()) return;
@@ -86,9 +96,14 b' var IPython = (function(IPython ) {'
         // one kernel completion came back, finish_completing will be called with the results
         // we fork here and directly call finish completing if kernel is busy
         if (this.skip_kernel_completion == true){
-            this.finish_completing({'matches' :[], matched_text : ""})
+            this.finish_completing({
+                'matches': [],
+                matched_text: ""
+            })
         } else {
-            var callbacks = {'complete_reply': $.proxy(this.finish_completing,this)};
+            var callbacks = {
+                'complete_reply': $.proxy(this.finish_completing, this)
+            };
             IPython.notebook.kernel.complete(line, cur.ch, callbacks);
         }
     };
@@ -105,13 +120,18 b' var IPython = (function(IPython ) {'
         // append the introspection result, in order, at at the beginning of
         // the table and compute the replacement range from current cursor
         // positon and matched_text length.
-        for(var i= matches.length-1; i>=0 ;--i)
-        {
-            results.unshift(
-                {   str  : matches[i],
+        for (var i = matches.length - 1; i >= 0; --i) {
+            results.unshift({
+                str: matches[i],
                     type : "introspection",
-                    from : {line: cur.line, ch: cur.ch-matched_text.length},
-                    to   : {line: cur.line, ch: cur.ch}
+                from: {
+                    line: cur.line,
+                    ch: cur.ch - matched_text.length
+                },
+                to: {
+                    line: cur.line,
+                    ch: cur.ch
+                }
                 });
         }
 
@@ -122,28 +142,29 b' var IPython = (function(IPython ) {'
         if (!this.raw_result || !this.raw_result.length) return;
 
         // When there is only one completion, use it directly.
-        if (this.autopick == true && this.raw_result.length == 1)
-        {
+        if (this.autopick == true && this.raw_result.length == 1) {
             this.insert(this.raw_result[0]);
             return;
         }
 
-        if (this.raw_result.length == 1)
-        {
+        if (this.raw_result.length == 1) {
             // test if first and only completion totally matches
             // what is typed, in this case dismiss
             var str = this.raw_result[0].str;
-            var pre_cursor = this.editor.getRange({line:cur.line,ch:cur.ch-str.length},cur);
-            if(pre_cursor == str)
-                { this.close(); return ; }
+            var pre_cursor = this.editor.getRange({
+                line: cur.line,
+                ch: cur.ch - str.length
+            }, cur);
+            if (pre_cursor == str) {
+                this.close();
+                return;
+            }
         }
 
         this.complete = $('<div/>').addClass('completions');
         this.complete.attr('id','complete');
 
-        this.sel = $('<select/>')
-                .attr('multiple','true')
-                .attr('size',Math.min(10,this.raw_result.length));
+        this.sel = $('<select/>').attr('multiple', 'true').attr('size', Math.min(10, this.raw_result.length));
         var pos = this.editor.cursorCoords();
 
         // TODO: I propose to remove enough horizontal pixel
@@ -155,15 +176,21 b' var IPython = (function(IPython ) {'
         $('body').append(this.complete);
         //build the container
         var that = this;
-        this.sel.dblclick(function(){that.pick();});
+        this.sel.dblclick(function() {
+            that.pick();
+        });
         this.sel.blur(this.close);
-        this.sel.keydown(function(event){that.keydown(event);});
+        this.sel.keydown(function(event) {
+            that.keydown(event);
+        });
 
         this.build_gui_list(this.raw_result);
 
         this.sel.focus();
         // Opera sometimes ignores focusing a freshly created node
-        if (window.opera) setTimeout(function(){if (!this.done) this.sel.focus();}, 100);
+        if (window.opera) setTimeout(function() {
+            if (!this.done) this.sel.focus();
+        }, 100);
         return true;
     }
 
@@ -174,9 +201,7 b' var IPython = (function(IPython ) {'
     Completer.prototype.build_gui_list = function(completions){
         // Need to clear the all list
         for (var i = 0; i < completions.length; ++i) {
-            var opt = $('<option/>')
-                .text(completions[i].str)
-                .addClass(completions[i].type);
+            var opt = $('<option/>').text(completions[i].str).addClass(completions[i].type);
             this.sel.append(opt);
         }
         this.sel.children().first().attr('selected','true');
@@ -192,7 +217,9 b' var IPython = (function(IPython ) {'
         this.insert(this.raw_result[this.sel[0].selectedIndex]);
         this.close();
         var that = this;
-        setTimeout(function(){that.editor.focus();}, 50);
+        setTimeout(function() {
+            that.editor.focus();
+        }, 50);
         }
 
 
@@ -200,11 +227,19 b' var IPython = (function(IPython ) {'
       var code = event.keyCode;
       var that = this;
       // Enter
-      if (code == key.enter) {CodeMirror.e_stop(event); this.pick();}
+        if (code == key.enter) {
+            CodeMirror.e_stop(event);
+            this.pick();
+        }
       // Escape or backspace
-      else if (code == key.esc ) {CodeMirror.e_stop(event); this.close(); this.editor.focus();}
-      else if (code == key.space || code == key.backspace) {this.close(); this.editor.focus();}
-      else if (code == key.tab){
+        else if (code == key.esc) {
+            CodeMirror.e_stop(event);
+            this.close();
+            this.editor.focus();
+        } else if (code == key.space || code == key.backspace) {
+            this.close();
+            this.editor.focus();
+        } else if (code == key.tab) {
             //all the fastforwarding operation,
             //Check that shared start is not null which can append with prefixed completion
             // like %pylab , pylab have no shred start, and ff will result in py<tab><tab>
@@ -217,17 +252,20 b' var IPython = (function(IPython ) {'
             CodeMirror.e_stop(event);
             this.editor.focus();
             //reinvoke self
-            setTimeout(function(){that.carryOnCompletion();}, 50);
-      }
-      else if (code == key.upArrow || code == key.downArrow) {
+            setTimeout(function() {
+                that.carryOnCompletion();
+            }, 50);
+        } else if (code == key.upArrow || code == key.downArrow) {
         // need to do that to be able to move the arrow
         // when on the first or last line ofo a code cell
         event.stopPropagation();
-      }
-      else if (code != key.upArrow && code != key.downArrow) {
-        this.close(); this.editor.focus();
+        } else if (code != key.upArrow && code != key.downArrow) {
+            this.close();
+            this.editor.focus();
         //we give focus to the editor immediately and call sell in 50 ms
-        setTimeout(function(){that.carryOnCompletion();}, 50);
+            setTimeout(function() {
+                that.carryOnCompletion();
+            }, 50);
       }
     }