-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drilldown event on chart fires multiple Times on stacked column series #2989
Comments
Duplicate: #2922 ? |
For sure this is not the best solution, but it's a patch that worked for me while I'm waiting for a official fix. From v4.0.3 to v4.0.4, it was fixed that it didn't render duplicated series in the drilldown chart, but the event is still being fired N times where N is the number of series on the main chart that share the same xAxis point. I've just added a "check for unique drilldown IDs", at the tick label click event. -> drilldown.src.js line 542~554 (Highcharts-4.0.4) tickLabel
.addClass('highcharts-drilldown-axis-label')
.css(chart.options.drilldown.activeAxisLabelStyle)
.on('click', function () {
var uniqueDrilldowns = {};
each(tickLabel.ddPoints, function (point) {
if (point.doDrilldown && !uniqueDrilldowns[point.drilldown]) {
uniqueDrilldowns[point.drilldown] = true;
point.doDrilldown(true);
}
});
chart.applyDrilldown();
}); |
The problem continue to be present with every version of drilldown that i could find. Thanks. Note: i've forked from pawelfus #2922 example by adding a second serie. |
Yes, getting this issue too, when there are more than one series, and trying to drilldown from label will fire the drilldown event twice, and then it will caused the |
IIRC i "fixed" it with sth similar to @mestremuten's Patch but this code was just for a prototype which never went into production. |
It is supposed to fire the drilldown event for each point. See http://api.highcharts.com/highcharts/chart.events.drilldown. Would it help if we added a |
@TorsteinHonsi Sounds promising, looks like we can simply use |
For workaround, I used
Such as following:
|
Has there been an update to the drilldown.js to include a drilldownall event? |
Thanks for the workaround! |
Any suggestion on handling the same in drillup? Since point, points not available in drillup event? |
Hi @sansmash We have |
That worked! Thank you. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions! |
Can someone explain to me the rationale behind calling the doDrilldown method on every point when clicking on the Axis labels?
This causes the "drilldown" Event in the Chart to fire multiple times,
which is kind of bad when drilling down asynchronously.
The text was updated successfully, but these errors were encountered: