🐛[xy-chart] fix top/left tooltip offset

[williaster]

🐛 Bug Fix

This fixes #137 where tooltips have incorrect offsets when using <YAxis orientation="left" /> or <XAxis orientation="top" />.

This bug occurs specifically with <AreaSeries /> and <LineSeries />, and the data point that is highlighted in these instances is approximately margin.left to the right of the point you expect it to be (for <YAxis orientation="left" />) or margin.top below the point you expect it to be (for <XAxis orientation="top" />).

cc @jhofstede

[codecov]

Codecov Report

Merging #140 into master will increase coverage by 0.03%.
The diff coverage is 84.21%.

@@            Coverage Diff             @@
##           master     #140      +/-   ##
==========================================
+ Coverage   79.76%   79.79%   +0.03%     
==========================================
  Files         109      109              
  Lines        2382     2386       +4     
  Branches      547      547              
==========================================
+ Hits         1900     1904       +4     
  Misses        310      310              
  Partials      172      172

Impacted Files	Coverage Δ
packages/xy-chart/src/selection/Brush.jsx	65.67% <ø> (ø)	⬆️
packages/xy-chart/src/series/StackedAreaSeries.jsx	97.29% <ø> (ø)	⬆️
packages/sparkline/src/chart/Sparkline.jsx	82.92% <ø> (ø)	⬆️
packages/sparkline/src/utils/findClosestDatum.js	10% <0%> (ø)	⬆️
packages/xy-chart/src/chart/XYChart.jsx	68.29% <0%> (ø)	⬆️
packages/xy-chart/src/chart/Voronoi.jsx	88% <100%> (ø)	⬆️
packages/sparkline/src/series/LineSeries.jsx	88.23% <100%> (ø)	⬆️
packages/shared/src/enhancer/WithTooltip.jsx	80% <100%> (ø)	⬆️
packages/xy-chart/src/utils/propShapes.js	100% <100%> (ø)	⬆️
packages/xy-chart/src/utils/findClosestDatums.js	93.33% <100%> (+0.47%)	⬆️
... and 4 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58a12bc...61c23eb. Read the comment docs.

[williaster]

build is somehow broken (I think due to babel minify), still debugging.

[williaster]

gonna merge this and fix the build separately.