Skip to content

Commit d0e1bef

Browse files
authored
Merge pull request #5662 from plotly/bar-period-width-fixup
Fixup and simplify bar hover min and max positions with period
2 parents d8a1654 + 3969715 commit d0e1bef

File tree

1 file changed

+1
-11
lines changed

1 file changed

+1
-11
lines changed

Diff for: src/traces/bar/hover.js

+1-11
Original file line numberDiff line numberDiff line change
@@ -39,17 +39,7 @@ function hoverOnBars(pointData, xval, yval, hovermode) {
3939
function thisBarMaxPos(di) { return thisBarExtPos(di, 1); }
4040

4141
function thisBarExtPos(di, sgn) {
42-
var w = di.w;
43-
var delta = sgn * w;
44-
if(trace[posLetter + 'period']) {
45-
var alignment = trace[posLetter + 'periodalignment'];
46-
if(alignment === 'start') {
47-
delta = (sgn === -1) ? 0 : w;
48-
} else if(alignment === 'end') {
49-
delta = (sgn === -1) ? -w : 0;
50-
}
51-
}
52-
return di[posLetter] + delta / 2;
42+
return di[posLetter] + 0.5 * sgn * di.w;
5343
}
5444

5545
var minPos = isClosest ?

0 commit comments

Comments
 (0)