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

bug in drilldown #7817

Closed
smj-hosseini opened this Issue Feb 7, 2018 · 9 comments

Comments

Projects
None yet
2 participants
@smj-hosseini

smj-hosseini commented Feb 7, 2018

There seems to be a problem in the drilldown module after destroying and re-initialising the chart. This occurs when a drilldown series in the new chart existed in the previous chart.
highcharts team said this bug was fixed but i test drilldown and still not working

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 7, 2018

Contributor

Hi @smj-hosseini

Could you follow our instruction when reporting the bug? It would be great to know:

  • Highcharts version
  • browser name(s) and version
  • jsfiddle demo with the issue

I know it's a follow up from #3990 - but demo posted there works fine: http://jsfiddle.net/BlackLabel/dxdzdw5q/2/

Contributor

pawelfus commented Feb 7, 2018

Hi @smj-hosseini

Could you follow our instruction when reporting the bug? It would be great to know:

  • Highcharts version
  • browser name(s) and version
  • jsfiddle demo with the issue

I know it's a follow up from #3990 - but demo posted there works fine: http://jsfiddle.net/BlackLabel/dxdzdw5q/2/

@smj-hosseini

This comment has been minimized.

Show comment
Hide comment
@smj-hosseini

smj-hosseini Feb 7, 2018

hi pawelfus,

Highcharts version: Highcharts JS v6.0.6 (2018-02-05)
browser name(s) and version: firefox quantum 58.0.1(64bit) 

i used currently highcharts and drilldown version but this not work
go to this jsfiddle link and follow the below steps to see the bug:
http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/demo/column-drilldown/
first: click on "microsoft internet explorer" bar and go back,
second: click on "safari" bar, and now you see this bug
thanks & good luck

smj-hosseini commented Feb 7, 2018

hi pawelfus,

Highcharts version: Highcharts JS v6.0.6 (2018-02-05)
browser name(s) and version: firefox quantum 58.0.1(64bit) 

i used currently highcharts and drilldown version but this not work
go to this jsfiddle link and follow the below steps to see the bug:
http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/demo/column-drilldown/
first: click on "microsoft internet explorer" bar and go back,
second: click on "safari" bar, and now you see this bug
thanks & good luck

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 7, 2018

Contributor

Hi @smj-hosseini

Thank you for the details! Just tested with the same Firefox version (osx), and this is how it works me:

7y8mnrq0hv

Contributor

pawelfus commented Feb 7, 2018

Hi @smj-hosseini

Thank you for the details! Just tested with the same Firefox version (osx), and this is how it works me:

7y8mnrq0hv

@smj-hosseini

This comment has been minimized.

Show comment
Hide comment
@smj-hosseini

smj-hosseini Feb 7, 2018

thanks men!
my issue is after click on "safari" bar there is a extra and unusable data on "safari" chart. this extra data mark as '4' on x-axis that not in data series for "safari" chart.
how to prevent of showing this extra data?
chart shows 'safari' data correctly when after steps that i said above, you click again on 'safari' bar and now the extra data omitted !!!

smj-hosseini commented Feb 7, 2018

thanks men!
my issue is after click on "safari" bar there is a extra and unusable data on "safari" chart. this extra data mark as '4' on x-axis that not in data series for "safari" chart.
how to prevent of showing this extra data?
chart shows 'safari' data correctly when after steps that i said above, you click again on 'safari' bar and now the extra data omitted !!!

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus

pawelfus Feb 7, 2018

Contributor

Thank you for the explanation, I was paying attention to the bug you reported in #3990 - but it's actually something different. You are right there shouldn't be extra category "4" when drilling down second time.

It's a regression since v6.0.5, v6.0.4 works fine: http://jsfiddle.net/BlackLabel/4ehmx54k/

Internal note:
Regression after 652e11b commit.

Contributor

pawelfus commented Feb 7, 2018

Thank you for the explanation, I was paying attention to the bug you reported in #3990 - but it's actually something different. You are right there shouldn't be extra category "4" when drilling down second time.

It's a regression since v6.0.5, v6.0.4 works fine: http://jsfiddle.net/BlackLabel/4ehmx54k/

Internal note:
Regression after 652e11b commit.

@smj-hosseini

This comment has been minimized.

Show comment
Hide comment
@smj-hosseini

smj-hosseini Feb 8, 2018

Thank you very much pawelfus,
Yes, highcharts v6.0.4 works fine,
thanks again for your help and fast replying

smj-hosseini commented Feb 8, 2018

Thank you very much pawelfus,
Yes, highcharts v6.0.4 works fine,
thanks again for your help and fast replying

@smj-hosseini

This comment has been minimized.

Show comment
Hide comment
@smj-hosseini

smj-hosseini Feb 10, 2018

hello again,
i faced again this problem, i have to ues both highcharts and highstock in my project. after googling and based on highcharts doc i realized that i should import only highstock to my project.
but when i import only highstock, the problem go back!!
i tested with a older highstock version but this trick not work.
jsfiddle link: http://jsfiddle.net/cjjm3xjL/

smj-hosseini commented Feb 10, 2018

hello again,
i faced again this problem, i have to ues both highcharts and highstock in my project. after googling and based on highcharts doc i realized that i should import only highstock to my project.
but when i import only highstock, the problem go back!!
i tested with a older highstock version but this trick not work.
jsfiddle link: http://jsfiddle.net/cjjm3xjL/

@smj-hosseini smj-hosseini changed the title from bug in drilldown to bug in drilldown when highstock used Feb 10, 2018

@smj-hosseini smj-hosseini changed the title from bug in drilldown when highstock used to bug in drilldown Feb 10, 2018

@pawelfus

This comment has been minimized.

Show comment
Hide comment
@pawelfus
Contributor

pawelfus commented Feb 12, 2018

@smj-hosseini

This comment has been minimized.

Show comment
Hide comment
@smj-hosseini

smj-hosseini Feb 12, 2018

yes, it works
thanks

smj-hosseini commented Feb 12, 2018

yes, it works
thanks

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