Skip to content

Commit dc704ae

Browse files
committed
move hovermode coerce outside the loop
1 parent 3c47bf9 commit dc704ae

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/plots/cartesian/layout_defaults.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,9 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
207207
}
208208
}
209209

210+
var hovermode = handleHoverModeDefaults(layoutIn, layoutOut, fullData);
211+
var unifiedHover = isUnifiedHover(hovermode);
212+
210213
// first pass creates the containers, determines types, and handles most of the settings
211214
for(i = 0; i < axNames.length; i++) {
212215
axName = axNames[i];
@@ -251,8 +254,6 @@ module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, fullData) {
251254
handleTypeDefaults(axLayoutIn, axLayoutOut, coerce, defaultOptions);
252255
handleAxisDefaults(axLayoutIn, axLayoutOut, coerce, defaultOptions, layoutOut);
253256

254-
var hovermode = handleHoverModeDefaults(layoutIn, layoutOut, fullData);
255-
var unifiedHover = isUnifiedHover(hovermode);
256257
var unifiedSpike = unifiedHover && axLetter === hovermode.charAt(0);
257258
var spikecolor = coerce2('spikecolor', unifiedHover ? axLayoutOut.color : undefined);
258259
var spikethickness = coerce2('spikethickness', unifiedHover ? 1.5 : undefined);

0 commit comments

Comments
 (0)