Permalink
Browse files

Fixed #5903, a regression causing the halo to cover bubbles so the bu…

…bbles weren't clickable.

Ref #5818.
  • Loading branch information...
TorsteinHonsi committed Oct 31, 2016
1 parent 57295e8 commit 0134b6e3a9dee0ff55fc0160b9aee6ba81f842d5
Showing with 9 additions and 4 deletions.
  1. +5 −2 js/parts-more/BubbleSeries.js
  2. +4 −2 js/parts/Interaction.js
@@ -242,8 +242,11 @@ seriesType('bubble', 'scatter', {
// Point class
}, {
haloPath: function () {
return Point.prototype.haloPath.call(this, this.shapeArgs.r + this.series.options.states.hover.halo.size);
haloPath: function (size) {
return Point.prototype.haloPath.call(
this,
this.shapeArgs.r + size
);
},
ttBelow: false
});
@@ -566,6 +566,7 @@ extend(Point.prototype, {
halo = series.halo,
haloOptions,
markerAttribs,
hasMarkers = markerOptions && series.markerAttribs,
newSymbol;
state = state || ''; // empty string
@@ -586,7 +587,7 @@ extend(Point.prototype, {
return;
}
if (markerOptions && series.markerAttribs) {
if (hasMarkers) {
markerAttribs = series.markerAttribs(point, state);
}
@@ -678,7 +679,8 @@ extend(Point.prototype, {
if (haloOptions && haloOptions.size) {
if (!halo) {
series.halo = halo = chart.renderer.path()
.add(series.markerGroup || series.group);
// #5818, #5903
.add(hasMarkers ? series.markerGroup : series.group);
}
H.stop(halo);
halo[move ? 'animate' : 'attr']({

0 comments on commit 0134b6e

Please sign in to comment.