Fix connectSeparatedPoints, add test
authorKlaus Weidner <klausw@google.com>
Fri, 2 Mar 2012 21:09:47 +0000 (13:09 -0800)
committerKlaus Weidner <klausw@google.com>
Fri, 2 Mar 2012 21:09:47 +0000 (13:09 -0800)
commit04c104d795f1c1857c6e499417ca61684cdcfce3
tree2e7b71eb228dd024de8471e3c6708e750f10cc01
parent90e92cf5ac0af418a771722b81bd7fbce942e5a1
Fix connectSeparatedPoints, add test

The original connectSeparatedPoints code renumbered points within series, with the effect that 'row' as used in setSelection didn't work as expected - it selects unrelated points.

This change keeps the point numbering intact, and instead moves the logic to skip missing points to the canvas renderer and setSelection().

A new auto_test confirms this behavior.
auto_tests/tests/callback.js
dygraph-canvas.js
dygraph-layout.js
dygraph-utils.js
dygraph.js