makesites / cloudvisio

Data visualizations of cloud data - built on top of D3 #javascript #d3 #html5 #css3
https://npmjs.org/package/cloudvisio
5 stars 1 forks source link

De-couple chart attributes #23

Closed tracend closed 11 years ago

tracend commented 11 years ago

The chart attributes were initially an extension of the general options object.

Although they were put there as an afterthought, they were in their own sub-object from the beginning.

This worked in the few first versions, but as their presence has become progressively more important they "deserve" their own space.

Proposing either introduction of an attributes object, or an _internal _axis object, along the lines of data...

The later implies the same relationship with the method axis which can replace the set method for updating the chart attributes.

tracend commented 11 years ago

Closing as fixed