Fixes #645 (text highlight problems)

nlplab · Feb 16, 2012 · 6232709 · 6232709
1 parent c6e4e85
commit 6232709
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/client/src/visualizer.js b/client/src/visualizer.js
@@ -869,6 +869,7 @@ var Visualizer = (function($, window, undefined) {
                 to: endPos
               };
             } else { // it's markedText [id, start?, char#, offset]
+              if (span[2] < 0) span[2] = 0;
               if (!span[2]) { // start
                 span[3] = text.getStartPositionOfChar(span[2]).x;
               } else {
@@ -879,9 +880,12 @@ var Visualizer = (function($, window, undefined) {
 
         // get the span annotation text sizes
         var spanTexts = {};
+        var noSpans = true;
         $.each(data.spans, function(spanNo, span) {
           spanTexts[span.glyphedLabelText] = true;
+          noSpans = false;
         });
+        if (noSpans) spanTexts.$ = true; // dummy so we can at least get the height
         var spanSizes = getTextMeasurements(spanTexts, {'class': 'span'});
 
         return {
@@ -1275,6 +1279,9 @@ Util.profileStart('chunks');
           // }
           var rightBorderForArcs = hasRightArcs ? arcHorizontalSpacing : (hasInternalArcs ? arcSlant : 0);
 
+          var lastX = current.x;
+          var lastRow = row;
+
           if (chunk.sentence) {
             while (sentenceNumber < chunk.sentence) {
               sentenceNumber++;
@@ -1288,9 +1295,6 @@ Util.profileStart('chunks');
             sentenceToggle = 1 - sentenceToggle;
           }
 
-          var lastX = current.x;
-          var lastRow = row;
-
           if (chunk.sentence ||
               current.x + boxWidth + rightBorderForArcs >= canvasWidth - 2 * Configuration.visual.margin.x) {
             // the chunk does not fit