Following up with the issue #10 I raised a couple of days ago I'm submitting this pull request.

These changes allow custom date formatting for both highcharts and google charts.

Hello,

Thank you for this tool! it's amazing! I'm newbie in rails / chartkick.

I'm currently using pie charts, and I'm wondering if there is a way I can get each section of my pie chart to navigate to an url.

I would like to know if it is possible / there is a way to , on clicking on the graph, it open an url link ? So if is there a way let me know. thanks!!!

    <h1>Pie Chart</h1>
    <div id="chart-2" style="height: 300px;"></div>
    <script>
      new Chartkick.PieChart("chart-2", [["Blueberry",44, "http://link1.com"],
                                                             ["Strawberry",23, "http://link2.com"],
                                                             ["Banana",22, "http://link3.com"],
                                                             ["Apple",21, "http://link4.com"],
                                                             ["Grape",13, "http://link5.com"]]);
    </script>

It would be great to add a calendar chart to this awesome library. Docs: https://developers.google.com/chart/interactive/docs/gallery/calendar?hl=en http://www.highcharts.com/demo/heatmap I'll be trying to implement this

Support for hAxisTitle & vAxisTitle options for all charts.

As per https://github.com/ankane/chartkick.js/pull/47#issuecomment-134474115

Do you like the option names? I had some trouble choosing what to use.. xAxis, xAxisLabel, xAxisTitle etc etc...

I initially need to add an Authorization header contains a bearer token. Since my project is already using a configured axios, supporting callback as data source would solve my problem and provide most flexibility meanwhile.

Hi @ankane

This is with regards to https://github.com/ankane/chartkick/issues/445

I've added two options:

• date_format
• time_format

And kept the default values in case they aren't being used.

I'm not sure if the naming is clear like this. Happy to hear your feedback.

Two methods have been added to Chartkick:

// Update one chart
Chartkick.updateChart('chart-id');               // Update chart from the current dataSource
Chartkick.updateChart('chart-id', newRemoteUrl); // Update chart from URL

// Update all charts
Chartkick.updateAllCharts(); // Updates from current `dataSource`
Chartkick.updateAllCharts(function(chart) {
  return chart.dataSource + '?some_param=abc';
}); // Updates all charts and appends ?some_param=abc to each chart's dataSource

This also allows for simple pull-based updates of any chart:

// Updates user-registration-chart every 10 seconds
setInterval(function() {
  Chartkick.updateChart('user-registration-chart');
}, 10000);

Thank you for a great library!

Currently Chartkick.js automatically adds a listener which removes all Chartkick.js charts upon turbo:before-render event. Unfortunately it is causing issues when there is a turbo form with data-turbo-frame and data-turbo-action specified. The latter causes that turbo:before-render is fired, and even though only contents of a single frame were updated, charts on the whole page are destroyed.

Frankly speaking, there is a chance that it is a Turbo bug, but at least from our point of view it was easier to fix that on Chartkick.js side, by disabling the automatic turbo:before-render and adding a code which removes charts in a different way (see the comment).

Anyway, maybe it is worth to check it, or at least provide some easy option of disabling automatic turbo:before-render listener?

I updated chartkick via bower from 2.2.1 to 2.2.2 and now I get the error

Type mismatch. Value 2016-06-26T12:30:00.000Z does not match type number in column index 0

My setup is chartkick in Gemfile set to 2.2.2 and I install the js file via bower, version 2.2.2. This works perfectly with js file 2.2.1.

Possible design (comments welcome)

new Chartkick.LineChart(id, [{name: "Series 1", data: data1}, {name: "Series 2", data: data2, y2: true}])

To set min and max values

new Chartkick.LineChart(id, series, y2: {min: 0, max: 1000})

For ChartJS, data with 0 values were not being plotted into the chart, so there were gaps in places where it shouldn't. That leads to confusion, as the 0 values were confused with null values (a.k.a no data). An use case is for example a chart that measures the heartbeats of an application, 0 values NEED to be plotted as that data exists and is valuable.