Skip to content
This repository has been archived by the owner on Jul 10, 2020. It is now read-only.

Feature/pushdata #32

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

hybr1dz
Copy link

@hybr1dz hybr1dz commented Sep 29, 2016

Feature to push data into chart without making the whole chart to redraw in browser (refreshData redraws the whole chart which looks awkward in a dynamically updating chart).

@gokhanoner
Copy link
Contributor

As the one responsible refreshData method, chart.update is the only solution that chart.js gave us unfortunately, and yes my intention was to renew the data, like putting updated values. But when you call update, whether u updated the whole data or just a label, it'll again re-draw the whole chart, it's what doc. says unfortunately.

@moberwasserlechner
Copy link
Owner

I think both approaches are need.

The chart.js samples show these features:

  • add dataset
  • remove dataset
  • add data point
  • remove data point
  • refresh data

I would like to support these for all supported chart types.

@hybr1dz Unfortunatly your approach only covers number based datasets. So please consider a more generic design.

@sancheztavo
Copy link

Hello, I'm very interested in this feature. If no one is implementing it, I'd give it a try.

@myungsunscott
Copy link

@moberwasserlechner @sancheztavo

I agree. This would be a great feature to have. I can also have a look if no-one else is interested in pushing it through.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants