Merge pull request #79 from marmelab/zoom_center

[RFR] Fix zoom center position (fixes #70)

.travis.yml

@@ -1,7 +1,5 @@
 sudo: false
 language: node_js
-node_js:
-  - "0.10.32"
 cache:
   directories:
     - node_modules

lib/drawer/drops.js

@@ -10,13 +10,13 @@ export default (svg, scales, configuration) => function dropsSelector(data) {
     dropLines.each(function dropLineDraw(drop) {
         const drops = d3.select(this).selectAll('.drop').data(drop.dates);
 
-        drops.attr('cx', d => scales.x(d) + 200);
+        drops.attr('cx', d => scales.x(d));
 
         const circle = drops.enter()
             .append('circle')
             .classed('drop', true)
             .attr('r', 5)
-            .attr('cx', d => scales.x(d) + 200)
+            .attr('cx', d => scales.x(d))
             .attr('cy', -5)
             .attr('fill', configuration.eventColor);
 

test/karma/drawer/drops.js

@@ -37,6 +37,19 @@ describe('Drops drawer', () => {
         expect(hoverSpy.calls.any()).toBe(true);
     });
 
+    it('should set `cx` attribute to given scale `x`', () => {
+        const data = [{ name: 'foo', dates: [new Date('2014-04-03')] }];
+
+        const chart = d3.chart.eventDrops()
+            .start(new Date('2014-04-02'))
+            .end(new Date('2014-04-04'));
+
+        d3.select(wrapper).datum(data).call(chart);
+
+        const drop = d3.select('.drop');
+        expect(+drop.attr('cx')).toBe(375);
+    });
+
     afterEach(() => {
         document.body.removeChild(wrapper);
     });