fix hoverdistance and spikedistance when it set to 0

Author: apalchys

src/components/fx/hover.js

@@ -332,8 +332,8 @@ function _hover(gd, evt, subplot, noHoverEvent) {
         // within one trace mode can sometimes be overridden
         mode = hovermode;
 
-        var hoverdistance = fullLayout.hoverdistance ? fullLayout.hoverdistance === 0 ? Infinity : fullLayout.hoverdistance : constants.MAXDIST;
-        var spikedistance = fullLayout.spikedistance ? fullLayout.spikedistance === 0 ? Infinity : fullLayout.spikedistance : hoverdistance;
+        var hoverdistance = fullLayout.hoverdistance === 0 ? Infinity : fullLayout.hoverdistance;
+        var spikedistance = fullLayout.spikedistance === 0 ? Infinity : fullLayout.spikedistance;
 
         // container for new point, also used to pass info into module.hoverPoints
         pointData = {

src/components/fx/layout_attributes.js

@@ -40,7 +40,7 @@ module.exports = {
     },
     hoverdistance: {
         valType: 'integer',
-        min: -1,
+        min: 0,
         dflt: 20,
         role: 'style',
         editType: 'none',
@@ -51,13 +51,13 @@ module.exports = {
     },
     spikedistance: {
         valType: 'integer',
-        min: -1,
+        min: 0,
         dflt: 20,
         role: 'style',
         editType: 'none',
         description: [
             'Sets the default distance (in points) to look for data to draw',
-            'spikelines to (zero means no cutoff). By default inherits from hoverdistance'
+            'spikelines to (zero means no cutoff).'
         ].join(' ')
     },
     hoverlabel: {