Skip to content

Commit 971c6f6

Browse files
committed
make reversed-radial-axis hover logic more readable
1 parent cd05d3d commit 971c6f6

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/traces/barpolar/hover.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,14 @@ module.exports = function hoverPoints(pointData, xval, yval) {
2626
var maxHoverDistance = pointData.maxHoverDistance;
2727
var period = angularAxis._period || 2 * Math.PI;
2828

29+
var rVal = Math.abs(radialAxis.g2p(Math.sqrt(xval * xval + yval * yval)));
30+
var thetaVal = Math.atan2(yval, xval);
31+
2932
// polar.(x|y)axis.p2c doesn't get the reversed radial axis range case right
3033
if(radialAxis.range[0] > radialAxis.range[1]) {
31-
xval *= -1;
32-
yval *= -1;
34+
thetaVal += Math.PI;
3335
}
3436

35-
var rVal = Math.abs(radialAxis.g2p(Math.sqrt(xval * xval + yval * yval)));
36-
var thetaVal = Math.atan2(yval, xval);
37-
3837
var distFn = function(di) {
3938
if(inboxFn(rVal, thetaVal, [di.rp0, di.rp1], [di.thetag0, di.thetag1], vangles)) {
4039
return maxHoverDistance +

0 commit comments

Comments
 (0)