Skip to content
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

Plot bands rendered on top of all series #8392

Closed
v-kabanov opened this issue May 29, 2018 · 2 comments
Closed

Plot bands rendered on top of all series #8392

v-kabanov opened this issue May 29, 2018 · 2 comments
Assignees
Labels

Comments

@v-kabanov
Copy link

@v-kabanov v-kabanov commented May 29, 2018

Expected behaviour

Plot bands by default used to remain under data series, especially if their zIndex was lower.

Actual behaviour

Bands are plotted over data series regardless of zIndex.

Live demo with steps to reproduce

https://jsfiddle.net/6zc79tcr/8/
Click the "Update bands" button repeatedly. See also https://stackoverflow.com/questions/50502086/highcharts-plot-bands-rendered-on-top-of-all-series .

Product version

Highstock 6.1.0

Affected browser(s)

Chrome v66, Firefox v60

@KacperMadej
Copy link
Contributor

@KacperMadej KacperMadej commented May 29, 2018

Hi @v-kabanov

Thank you for reporting about the problem.

Here's a simpler demo: https://jsfiddle.net/BlackLabel/6zc79tcr/9/
After update (on a button click) axis elements are in front - also grid lines.

It happens when accessibility and exporting modules are loaded.

Internal note: addPlotBand and removePlotBand seems to be affected too.
Problematic: http://jsfiddle.net/BlackLabel/mudqbp3m/
Standard (working): http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/members/axis-addplotband/

@TorsteinHonsi
Copy link
Collaborator

@TorsteinHonsi TorsteinHonsi commented May 31, 2018

The issue only happens when the export SVG items are added to the chart.exportSVGElements array. If we comment out the following line, it works as expected:

chart.exportSVGElements.push(button, symbol);

The exportSVGElements array is later only used in the accessibility module, so the issue must lie there @oysteinmoseng .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.