My graphs broke from 2.0.2 to 2.1.0. Follow semver?

[olyckne]

I had ^2.0.2 in my package.json, so when running a clean npm install I got the new 2.1.0
and my chart went from working

to not working (I've set black on the points so we can see them)

The fill doesn't seem to work at all. Any ideas why?

This line in the release notes is telling me you don't follow semver which I really think you should.
I used those in my code so that broke, but I've fixed that so it doesn't seem related to the chart looking weird.

You can now share data between charts. Note that this will require changes if dataset.metadata or dataset.metaDataset were being used. There is now a helper function to retrieve the correct meta data for a given dataset

[olyckne]

And right when I posted I figured it out.

I had to change tension from 0 to 0.0

elements: {
    line: {
        tension: 0.0,
    },
}

Doesn't look exactly as before though, and this small one doesn't look the same at all
2.0.2:

2.1.0:

[etimberg]

hi @olyckne

I'm sorry that we broke your chart(s). We may have, wrongly, assumed that not many people were using the meta objects yet. We are committed to keeping the config fully backwards compatible and we don't forsee needing any changes in stuff like this again.

Regarding your tension issue, I have just merged #2473 which fixes them and am preparing for v2.1.1 release that will include those fixes.

[olyckne]

I guess that is a fair assumptions to make though. Sometimes you're wrong :)
Glad I noticed it before we did a new release, and it was a pretty easy fix.

Any idea why the chart suddently isn't taking up the whole canvas?
It makes my really small chart look a lot different.
Guess it might be something with the axes that I've set to display: false, but they're still taking up space?

2.0.2:

2.1.0:

[etimberg]

@olyckne looks like that is getting set at https://github.com/chartjs/Chart.js/blob/master/src/core/core.scale.js#L281

We can either add a check for display: false or you could set the following in the scale config

{
  gridLines: {
    tickMarkLength: 0
  }
}

[olyckne]

Thanks!
tickMarkLength: 0 works, but I think taking display: false into account makes sense.

[etimberg]

@olyckne v2.1.1 released with the tension fix included. I will work on the display: false sizing shortly