🔪 hoveron in scattergl

- hoveron was added *just* to make scatter/hover.js
  not break, it's functionality was never implemented in scattergl.

Author: etpinard

src/traces/scatter/hover.js

@@ -24,11 +24,12 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
         ya = pointData.ya,
         xpx = xa.c2p(xval),
         ypx = ya.c2p(yval),
-        pt = [xpx, ypx];
+        pt = [xpx, ypx],
+        hoveron = trace.hoveron || '';
 
     // look for points to hover on first, then take fills only if we
     // didn't find a point
-    if(trace.hoveron.indexOf('points') !== -1) {
+    if(hoveron.indexOf('points') !== -1) {
         var dx = function(di) {
                 // scatter points: d.mrc is the calculated marker radius
                 // adjust the distance so if you're inside the marker it
@@ -84,7 +85,7 @@ module.exports = function hoverPoints(pointData, xval, yval, hovermode) {
     }
 
     // even if hoveron is 'fills', only use it if we have polygons too
-    if(trace.hoveron.indexOf('fills') !== -1 && trace._polygons) {
+    if(hoveron.indexOf('fills') !== -1 && trace._polygons) {
         var polygons = trace._polygons,
             polygonsIn = [],
             inside = false,

src/traces/scattergl/attributes.js

@@ -84,17 +84,5 @@ module.exports = {
     fillcolor: scatterAttrs.fillcolor,
 
     error_y: scatterAttrs.error_y,
-    error_x: scatterAttrs.error_x,
-
-    hoveron: {
-        valType: 'flaglist',
-        flags: ['points', 'fills'],
-        role: 'info',
-        description: [
-            'Do the hover effects highlight individual points (markers or',
-            'line points) or do they highlight filled regions?',
-            'If the fill is *toself* or *tonext* and there are no markers',
-            'or text, then the default is *fills*, otherwise it is *points*.'
-        ].join(' ')
-    }
+    error_x: scatterAttrs.error_x
 };

src/traces/scattergl/defaults.js

@@ -50,17 +50,6 @@ module.exports = function supplyDefaults(traceIn, traceOut, defaultColor, layout
         handleFillColorDefaults(traceIn, traceOut, defaultColor, coerce);
     }
 
-    var dfltHoverOn = [];
-
-    if(subTypes.hasMarkers(traceOut) || subTypes.hasText(traceOut)) {
-        dfltHoverOn.push('points');
-    }
-    if(traceOut.fill === 'tonext' || traceOut.fill === 'toself') {
-        dfltHoverOn.push('fills');
-    }
-
-    coerce('hoveron', dfltHoverOn.join('+') || 'points');
-
     errorBarsSupplyDefaults(traceIn, traceOut, defaultColor, {axis: 'y'});
     errorBarsSupplyDefaults(traceIn, traceOut, defaultColor, {axis: 'x', inherit: 'y'});
 };