-
Notifications
You must be signed in to change notification settings - Fork 59
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
Chart events not exposed - Feature request #37
Comments
Absolutely agree. This is planned. For whatever reason I neglected to create an Issue. Thank you! |
I would love to receive a Pull Request adding this functionality 😄. |
I will give it a shot. I looked for the D3 instance but could not find it. Where is it buried? Chris On 10/15/2015 03:03 PM, Glavin Wiechert wrote:
|
I think the applicable code would be https://github.com/Glavin001/ember-c3/blob/master/addon/components/c3-chart.js#L118-L120 |
How about this for expected usage: {{c3-chart data=model chart-created=(action "chartCreated")} And your Controller or Route would have an action named {
actions: {
chartCreated: function(chart) {
this.set('myChartProperty', chart);
}
}
} This would also work without any internal changes to Ember-C3: {{c3-chart data=model chart-created=(mut chart)} Where |
See issue #77 |
As a C3 user/developer it would be nice to have the chart actions linked to the ember action chain. This would provide interaction with other components on the view eg:
A group selection on the chart could trigger a refresh of a table showing the data details.
This would involve catching the 'once' method callback in D3 and emitting it as an Ember action.
Thanks
Chris (expatriate Canadian in San Diego)
The text was updated successfully, but these errors were encountered: