Project

General

Profile

Actions

Bug #12731

closed

[crunchstat-summary] zoom/pan should be synchronized for all HTML graphs

Added by Tom Morris over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Tom Morris
Category:
-
Target version:
Story points:
-

Description

Currently the four graphs in the HTML plot are zoomed and panned independently. This should be synchronized, locking the X axes together for all of them.

Example code available here: https://github.com/danvk/dygraphs/blob/master/src/extras/synchronizer.js


Subtasks 1 (0 open1 closed)

Task #12732: Review 12731-synchronized-graphsClosedTom Morris12/04/2017Actions
Actions #1

Updated by Tom Morris over 6 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Tom Clegg over 6 years ago

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.

Actions #3

Updated by Tom Morris over 6 years ago

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.

Actions #4

Updated by Tom Clegg over 6 years ago

LGTM

Actions #5

Updated by Anonymous over 6 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF