search

fanthos / chartjs-chart-timeline

Google-like timeline chart for Chart.js(currently v2).
BSD 2-Clause "Simplified" License
52 stars 22 forks source link

Bug: barThickness is mandatory, charts break if not supplied #7

Open MBlistein opened 6 years ago

MBlistein commented 6 years ago

Usually the barThickness is an optional value. I would prefer to not use it, since this static value has significant disadvantages when displaying charts of different size screens. However, when removing it in the current code, the entire chart fails.

fanthos commented 6 years ago

I think the barThinkness should be reimplemented as barMaxThinkness.

Currently the chart can be resized by external code. ref: home-assistant Just use the code after creating the chart with setTimeout.