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 3d Piechart disappear after resize #6437

Closed
harshpuidev opened this Issue Mar 7, 2017 · 8 comments

Comments

Projects
None yet
3 participants
@harshpuidev

harshpuidev commented Mar 7, 2017

Go to the fiddle below. Resize, increase the width of the container and then click on any part of the pie chart to drilldown for some reason pie chart disappear i can only see the labels.
capture

capture2

Live demo with steps to reproduce

http://jsfiddle.net/quickstep25/yhqqo2wv/
issue can be reproduce in chrome Browser

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 13, 2017

Contributor

Hi @harshpuidev

Thank you for reporting this bug.

Internal note:
It doesn't look like a regression - tested up to 4.1.x.

Contributor

pawelfus commented Mar 13, 2017

Hi @harshpuidev

Thank you for reporting this bug.

Internal note:
It doesn't look like a regression - tested up to 4.1.x.

@pawelfus pawelfus added the Bug label Mar 13, 2017

@KiranHundal

This comment has been minimized.

Show comment
Hide comment
@KiranHundal

KiranHundal Mar 13, 2017

Hi @pawelfus

I am seeing the same issue with highcharts on my application as well. Could you please let me know when will you guys look into this. We are going to use highcharts extensively and this for sure is going to be a buggy problem.

KiranHundal commented Mar 13, 2017

Hi @pawelfus

I am seeing the same issue with highcharts on my application as well. Could you please let me know when will you guys look into this. We are going to use highcharts extensively and this for sure is going to be a buggy problem.

@pawelfus pawelfus self-assigned this Mar 14, 2017

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 14, 2017

Contributor

I will look into it.

Contributor

pawelfus commented Mar 14, 2017

I will look into it.

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 14, 2017

Contributor

Internal note:
The problem was with animating after a resize here. The method modified in the commit above caused shapeArgs to be (almost) empty, so drilldown module couldn't animate slices properly. Solution could to to merge options here, but instead of merging before method call, we can merge inside (we use that method only in two places, both should merge).

Contributor

pawelfus commented Mar 14, 2017

Internal note:
The problem was with animating after a resize here. The method modified in the commit above caused shapeArgs to be (almost) empty, so drilldown module couldn't animate slices properly. Solution could to to merge options here, but instead of merging before method call, we can merge inside (we use that method only in two places, both should merge).

@KiranHundal

This comment has been minimized.

Show comment
Hide comment
@KiranHundal

KiranHundal Mar 16, 2017

Thanks a lot for fixing this bug. Could you please tell when will this fix be ready for us to use?

KiranHundal commented Mar 16, 2017

Thanks a lot for fixing this bug. Could you please tell when will this fix be ready for us to use?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 17, 2017

Contributor

Hi @KiranHundal

It will be part of the next release. Next release should be ready within a few weeks.

Contributor

pawelfus commented Mar 17, 2017

Hi @KiranHundal

It will be part of the next release. Next release should be ready within a few weeks.

@harshpuidev

This comment has been minimized.

Show comment
Hide comment
@harshpuidev

harshpuidev Mar 21, 2017

Hi @pawelfus

Thank you for the fix we have the application up and running. Our customers are begging us for the fix. it would helpful if you can tell us when would be the next release(a specify week or a day would be great)?

harshpuidev commented Mar 21, 2017

Hi @pawelfus

Thank you for the fix we have the application up and running. Our customers are begging us for the fix. it would helpful if you can tell us when would be the next release(a specify week or a day would be great)?

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Mar 21, 2017

Contributor

Unfortunately the date is not specified yet. I suggest modifying sources or using this workaround:

    // Workaround for #6437
    Highcharts.wrap(Highcharts.SVGElement.prototype, 'animate', function (proceed, params, options, complete) {
        return proceed.call(this, Highcharts.merge(params), options, complete);
    });

Demo: http://jsfiddle.net/yhqqo2wv/11/

Note: I strongly recommend to update Highcharts (and remove the workaround) once next release will be ready - the workaround above should be added only to the pages where issue appears. My apologies for the inconvenience!

Contributor

pawelfus commented Mar 21, 2017

Unfortunately the date is not specified yet. I suggest modifying sources or using this workaround:

    // Workaround for #6437
    Highcharts.wrap(Highcharts.SVGElement.prototype, 'animate', function (proceed, params, options, complete) {
        return proceed.call(this, Highcharts.merge(params), options, complete);
    });

Demo: http://jsfiddle.net/yhqqo2wv/11/

Note: I strongly recommend to update Highcharts (and remove the workaround) once next release will be ready - the workaround above should be added only to the pages where issue appears. My apologies for the inconvenience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment