From 05fd6df8fe9e42bdfe8b3217f65719c201690d18 Mon Sep 17 00:00:00 2001
From: Fabian Jakobs <fabian.jakobs@web.de>
Date: Wed, 5 May 2010 10:50:38 +0200
Subject: [PATCH] fix ie issue to convert document to screen coordinates

---
 demo/editor.html                |  9 ++++-----
 src/ace/VirtualRenderer.js      |  5 +++--
 src/test/VirtualRendererTest.js | 13 +++++++++++++
 3 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/demo/editor.html b/demo/editor.html
index 35dd4d1e..408f59f1 100644
--- a/demo/editor.html
+++ b/demo/editor.html
@@ -115,11 +115,10 @@
     </div>
     
     <script type="text/editor" id="jstext">function foo(items) {
-    for (var i=0; i<items.length; i++) {
-        alert(items[i] + "juhu";
-    }
-}
-    </script>
+	for (var i=0; i<items.length; i++) {
+		alert(items[i] + "juhu";
+	}
+}</script>
 
     <script type="text/editor" id="csstext">.text-layer {
     font-family: Monaco, "Courier New", monospace;
diff --git a/src/ace/VirtualRenderer.js b/src/ace/VirtualRenderer.js
index e8552a53..48e02331 100644
--- a/src/ace/VirtualRenderer.js
+++ b/src/ace/VirtualRenderer.js
@@ -228,7 +228,7 @@ ace.VirtualRenderer = function(container) {
         var screenColumn = 0;
         var remaining = docColumn;
 
-        var line = this.doc.getLine(row).split(/\t/g);
+        var line = this.doc.getLine(row).split("\t");
         for (var i=0; i<line.length; i++) {
             var len = line[i].length;
             if (remaining > len) {
@@ -240,6 +240,7 @@ ace.VirtualRenderer = function(container) {
                 break;
             }
         }
+
         return screenColumn;
     };
 
@@ -249,7 +250,7 @@ ace.VirtualRenderer = function(container) {
         var docColumn = 0;
         var remaining = screenColumn;
 
-        var line = this.doc.getLine(row).split(/\t/g);
+        var line = this.doc.getLine(row).split("\t");
         for (var i=0; i<line.length; i++) {
             var len = line[i].length;
             if (remaining >= len + tabSize) {
diff --git a/src/test/VirtualRendererTest.js b/src/test/VirtualRendererTest.js
index 13cf3cfa..f77ed742 100644
--- a/src/test/VirtualRendererTest.js
+++ b/src/test/VirtualRendererTest.js
@@ -25,6 +25,19 @@ var VirtualRendererTest = new TestCase("VirtualRendererTest", {
         assertEquals(15, renderer.$documentToScreenColumn(0, 13));
     },
 
+    "test: convert document to scrren coordinates with leading tabs": function() {
+        var el = document.createElement("div");
+        var renderer = new ace.VirtualRenderer(el);
+
+        var doc = new ace.Document("\t\t123");
+        doc.setTabSize(4);
+        renderer.setDocument(doc);
+        assertEquals(0, renderer.$documentToScreenColumn(0, 0));
+        assertEquals(4, renderer.$documentToScreenColumn(0, 1));
+        assertEquals(8, renderer.$documentToScreenColumn(0, 2));
+        assertEquals(9, renderer.$documentToScreenColumn(0, 3));
+    },
+
     "test: convert screen to document coordinates" : function() {
         var el = document.createElement("div");
         var renderer = new ace.VirtualRenderer(el);