better cursor positioning fixes #209

* adjust cursor position on first open #209

* added conditions to cursor col

* adjust for : in col condition

Author: cfloume

src/ed.js

@@ -36,6 +36,7 @@ D.Ed=function(ide,opts){ //constructor
                              ed[c]?ed[c](ed.cm):CM.commands[c]?CM.commands[c](ed.cm):0;return!1}
   }
   ed.setTC(!!ed.tc);this.vt=D.vt(this);this.setLN(D.prf.lineNums())
+  ed.firstOpen=true;
 }
 D.Ed.prototype={
   updGutters:function(){
@@ -92,7 +93,7 @@ D.Ed.prototype={
     this.jumps.forEach(function(x){x.n=x.lh.lineNo()}) //to preserve jumps, convert LineHandle-s to line numbers
     cm.setValue(ed.oText=ee.text.join('\n')) //.setValue() invalidates old LineHandle-s
     this.jumps.forEach(function(x){x.lh=cm.getLineHandle(x.n);delete x.n}) //look up new LineHandle-s, forget numbers
-    cm.clearHistory()
+    cm.clearHistory();
     if(D.mac){cm.focus();window.focus()}
     //entityType:             16 NestedArray        512 AplClass
     // 1 DefinedFunction      32 QuadORObject      1024 AplInterface
@@ -109,7 +110,7 @@ D.Ed.prototype={
       ed.dom.getElementsByClassName("tb_RP")[0].style.display="none"
     }
     var line=ee.currentRow,col=ee.currentColumn||0
-    if(line===0&&col===0&&ee.text.length===1)col=ee.text[0].length
+    if(line===0&&col===0&&ee.text.length===1&&/\s?[a-z|@]+$/.test(ee.text[0]))col=ee.text[0].length
     cm.setCursor(line,col);cm.scrollIntoView(null,cm.getScrollInfo().clientHeight/2)
     ed.oStop=(ee.stop||[]).slice(0).sort(function(x,y){return x-y})
     for(var k=0;k<ed.oStop.length;k++)cm.setGutterMarker(ed.oStop[k],'breakpoints',ed.createBPEl())

src/ide.js

@@ -176,6 +176,12 @@ D.IDE=function(){'use strict'
       w.saveScrollPos();
       w.cm.setValue(x.text.join('\n'));
       if (w.tc)w.hl(u.line);
+      if (w.firstOpen!==undefined&&w.firstOpen===true){
+        if (x.text.length===1&&/\s?[a-z|@]+$/.test(x.text[0]))u.ch=w.cm.getLine(u.line).length
+        else if (x.text[0][0]===":")u.ch=0
+        else u.ch=1
+        w.firstOpen=false
+      }
       w.restoreScrollPos();
       w.cm.setCursor(u);
     },