Fix bug in text width measurement

Text width should be measured after context is set.
stevewirts · Aug 21, 2015 · 6c91360 · 6c91360
1 parent 16a079e
commit 6c91360
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/javascript/components/defaultcellrenderer.js b/src/javascript/components/defaultcellrenderer.js
@@ -17,8 +17,8 @@ var paint = function(gc, config) {
 
     var cellPadding = config.cellPadding || 0;
     var halignOffset = 0;
-    var textWidth = config.getTextWidth(gc, value);
-    var fontMetrics = config.getTextHeight(font);
+    var textWidth;
+    var fontMetrics;
 
     if (gc.font !== config.font) {
         gc.font = config.font;
@@ -30,6 +30,9 @@ var paint = function(gc, config) {
         gc.textBaseline = 'middle';
     }
 
+    textWidth = config.getTextWidth(gc, value);
+    fontMetrics = config.getTextHeight(font);
+
     if (halign === 'right') {
         //textWidth = config.getTextWidth(gc, config.value);
         halignOffset = width - cellPadding - textWidth;