remove period addition hacks now that the filter is gone

Author: archmoj

src/components/fx/hover.js

@@ -14,7 +14,6 @@ var Drawing = require('../drawing');
 var Color = require('../color');
 var dragElement = require('../dragelement');
 var Axes = require('../../plots/cartesian/axes');
-var alignPeriod = require('../../plots/cartesian/align_period');
 var Registry = require('../../registry');
 
 var helpers = require('./helpers');
@@ -662,34 +661,8 @@ function _hover(gd, evt, subplot, noHoverEvent) {
 
         findHoverPoints(customXVal, customYVal);
 
-        // also find start, middle and end point for period
         var axLetter = hovermode.charAt(0);
         if(winningPoint.trace[axLetter + 'period']) {
-            var v = winningPoint[axLetter + 'LabelVal'];
-            var ax = winningPoint[axLetter + 'a'];
-            var T = {};
-            T[axLetter + 'period'] = winningPoint.trace[axLetter + 'period'];
-            T[axLetter + 'period0'] = winningPoint.trace[axLetter + 'period0'];
-
-            T[axLetter + 'periodalignment'] = 'start';
-            var start = alignPeriod(T, ax, axLetter, [v])[0];
-
-            T[axLetter + 'periodalignment'] = 'middle';
-            var middle = alignPeriod(T, ax, axLetter, [v])[0];
-
-            T[axLetter + 'periodalignment'] = 'end';
-            var end = alignPeriod(T, ax, axLetter, [v])[0];
-
-            if(axLetter === 'x') {
-                findHoverPoints(start, customYVal);
-                findHoverPoints(middle, customYVal);
-                findHoverPoints(end, customYVal);
-            } else {
-                findHoverPoints(customXVal, start);
-                findHoverPoints(customXVal, middle);
-                findHoverPoints(customXVal, end);
-            }
-
             var k;
             var seen = {};
             for(k = 0; k < initLen; k++) {

test/jasmine/tests/hover_label_test.js

@@ -4974,6 +4974,7 @@ describe('hovermode: (x|y)unified', function() {
 
                 _hover(gd, { xpx: 100, ypx: 200 });
                 assertLabel({title: 'Jan 1, 2000', items: [
+                    'bar : (Dec, 2)',
                     'scatter : 1.1'
                 ]});
 
@@ -5051,22 +5052,20 @@ describe('hovermode: (x|y)unified', function() {
             _hover(gd, { xpx: 40, ypx: 200 });
             assertLabel({title: 'Jan', items: [
                 'bar : (Jan 1, 2000, 1)',
-                'start : 1',
-                'end : 1'
+                'start : 1'
             ]});
 
             _hover(gd, { xpx: 100, ypx: 200 });
             assertLabel({title: 'Jan 1, 2000', items: [
                 'bar : 1',
-                'start : (Jan, 1)',
-                'end : (Jan, 1)'
+                'start : (Jan, 1)'
             ]});
 
             _hover(gd, { xpx: 360, ypx: 200 });
             assertLabel({title: 'Feb 1, 2000', items: [
                 'bar : 2',
                 'start : (Feb, 2)',
-                'end : (Feb, 2)'
+                'end : (Jan, 1)'
             ]});
 
             _hover(gd, { xpx: 400, ypx: 200 });