Bug #12731
closed
[crunchstat-summary] zoom/pan should be synchronized for all HTML graphs
Added by Tom Morris about 7 years ago.
Updated about 7 years ago.
- Status changed from New to In Progress
at f24bdaa4b
Need to add synchronizer.js to MANIFEST.in (otherwise it doesn't get packaged)
Is synchronizer.js modified from the original source (other than by adding the attribution)? Maybe mention the git commit in case we ever need to update it.
MIT license requires copyright notice and license text. In case this is considered a "substantial portion" I'd suggest pasting https://github.com/danvk/dygraphs/blob/master/LICENSE.txt into a comment in synchronizer.js.
As for the actual content -- tested & works, LGTM.
at 0e2cc747caebc3e07cbbf0ed6ee18e8141646080
Added author copyright, MIT license, & reference to commit hash. (It is unchanged from original).
Added to MANIFEST.in (and removed webchart.js reference). Tested that ./setup.py build actually works.
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Also available in: Atom
PDF