Take config.plotGlPixelRatio into account for scattergl trace

Author: jonmmease

src/plot_api/plot_api.js

@@ -223,8 +223,10 @@ function _doPlot(gd, data, layout, config) {
 
         if(fullLayout._glcanvas) {
             fullLayout._glcanvas
-                .attr('width', fullLayout.width)
-                .attr('height', fullLayout.height);
+                .attr('width', fullLayout.width * gd._context.plotGlPixelRatio)
+                .attr('height', fullLayout.height * gd._context.plotGlPixelRatio)
+                .style('width', fullLayout.width + 'px')
+                .style('height', fullLayout.height + 'px');
 
             var regl = fullLayout._glcanvas.data()[0].regl;
             if(regl) {

src/plots/cartesian/index.js

@@ -596,8 +596,8 @@ exports.toSVG = function(gd) {
             preserveAspectRatio: 'none',
             x: 0,
             y: 0,
-            width: canvas.width,
-            height: canvas.height
+            width: canvas.style.width,
+            height: canvas.style.height
         });
     }
 

src/traces/scattergl/plot.js

@@ -14,15 +14,23 @@ var linkTraces = require('../scatter/link_traces');
 
 var styleTextSelection = require('./edit_style').styleTextSelection;
 
-function getViewport(fullLayout, xaxis, yaxis) {
+
+function getViewport(fullLayout, xaxis, yaxis, plotGlPixelRatio) {
     var gs = fullLayout._size;
-    var width = fullLayout.width;
-    var height = fullLayout.height;
+    var width = fullLayout.width * plotGlPixelRatio;
+    var height = fullLayout.height * plotGlPixelRatio;
+
+    var l = gs.l * plotGlPixelRatio;
+    var b = gs.b * plotGlPixelRatio;
+    var r = gs.r * plotGlPixelRatio;
+    var t = gs.t * plotGlPixelRatio;
+    var w = gs.w * plotGlPixelRatio;
+    var h = gs.h * plotGlPixelRatio;
     return [
-        gs.l + xaxis.domain[0] * gs.w,
-        gs.b + yaxis.domain[0] * gs.h,
-        (width - gs.r) - (1 - xaxis.domain[1]) * gs.w,
-        (height - gs.t) - (1 - yaxis.domain[1]) * gs.h
+        l + xaxis.domain[0] * w,
+        b + yaxis.domain[0] * h,
+        (width - r) - (1 - xaxis.domain[1]) * w,
+        (height - t) - (1 - yaxis.domain[1]) * h
     ];
 }
 
@@ -59,7 +67,7 @@ module.exports = function plot(gd, subplot, cdata) {
             scene.line2d = createLine(regl);
         }
         if(scene.scatter2d === true) {
-            scene.scatter2d = createScatter(regl, { constPointSize: true });
+            scene.scatter2d = createScatter(regl);
         }
         if(scene.fill2d === true) {
             scene.fill2d = createLine(regl);
@@ -330,7 +338,7 @@ module.exports = function plot(gd, subplot, cdata) {
 
     // provide viewport and range
     var vpRange0 = {
-        viewport: getViewport(fullLayout, xaxis, yaxis),
+        viewport: getViewport(fullLayout, xaxis, yaxis, gd._context.plotGlPixelRatio),
         // TODO do we need those fallbacks?
         range: [
             (xaxis._rl || xaxis.range)[0],