fix svgTextUtils for FF: use child element instead of SVG to getSize()

plotly · Apr 17, 2019 · fe842c4 · fe842c4
1 parent 172c3e6
commit fe842c4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/lib/svg_text_utils.js b/src/lib/svg_text_utils.js
@@ -114,10 +114,11 @@ exports.convertToTspans = function(_context, gd, _callback) {
                 .style({overflow: 'visible', 'pointer-events': 'none'});
 
                 var fill = _context.node().style.fill || 'black';
-                newSvg.select('g').attr({fill: fill, stroke: fill});
+                var g = newSvg.select('g');
+                g.attr({fill: fill, stroke: fill});
 
-                var newSvgW = getSize(newSvg, 'width');
-                var newSvgH = getSize(newSvg, 'height');
+                var newSvgW = getSize(g, 'width');
+                var newSvgH = getSize(g, 'height');
                 var newX = +_context.attr('x') - newSvgW *
                     {start: 0, middle: 0.5, end: 1}[_context.attr('text-anchor') || 'start'];
                 // font baseline is about 1/4 fontSize below centerline