search

my-archives / nchart

nChart for node.js inspired by Chart.js.
MIT License
48 stars 10 forks source link

Remove canvas require #5

Closed kirbysayshi closed 10 years ago

kirbysayshi commented 10 years ago

As far as I can tell, canvas as an official dependency is no longer needed since the context is expected to be passed into the graph. This means that this module can be browserified and used in the browser too!

Please let me know if I missed something!

fundon commented 10 years ago

Good catch. browserify is cool. Thanks!