search

d3 / d3-transition

Animated transitions for D3 selections.
https://d3js.org/d3-transition
ISC License
224 stars 65 forks source link

Rewrite with fewer closures. #31

Closed mbostock closed 8 years ago

mbostock commented 8 years ago

For performance, we should experiment with rewriting schedule.js, styleTween.js and attrTween.js to not use closures. I think it should be possible without changing the public API…

mbostock commented 8 years ago

Possibly still worth pursuing, but punting on this for now. Rewriting everything without closures would be a pretty massive undertaking (starting with d3-timer). There are possibly low-hanging fruit, such as the Schedule class, but I don’t expect it to be a huge performance win, either.