Multiple charts per page?

[skipchris]

Hi,

I might be making a mistake, in which case sorry for the non-issue, but it currently doesn’t seem possible to use Chart.Bands on a page which contains both a bar and a line graph – the isSupported flag is set ‘globally’ rather than on a per-chart basis.

I’ve worked around this locally by removing the isSupported check, and instead checking for a per-instance bands config option, and only trying to apply the plugin if this param is present.

I’m happy to submit this as a PR, and update the README to match, would you like this, or am I missing something important?

[martov1]

oh man thanks for pointing that out, I have been fighting this problem for 2 hours.

[EdwardLuke]

Hi @skipchris - I don't think you're missing anything important. If you'd still like to submit that PR, feel free (although I realise this message has been a long time coming!)

[skipchris]

😂

OK, I am currently on leave for a month, but I would still like this little clean-up, so I hope to tackle it at some point Feb/March

[EdwardLuke]

Ha, enjoy your leave! Great - I'll keep an eye out 👍

[thormug]

There is a lot of problems when you have multiple charts on the same page with plugin.
Everything seems to be set globally, when you try to render 2 line charts, the plugin does mistakes.

I'm supposed to have the top rendering, but if it load after the other chart, it looks like that.
The belowThresholdColour is screwed.