Follow-on to pull request 237, closurifying some code.

Reponses to code review for https://github.com/danvk/dygraphs/pull/237

Merge branch 'master' of github.com:danvk/dygraphs into closure

Conflicts:
dygraph-options.js

Merge pull request #233 from lucasbergman/master

some small Closure-cleanliness fixes

Merge pull request #240 from timeu/bugfix_upstream

Bugfix: Avoid exception when returning the index of the point and one or more series are hidden

Bugfix: avoid exception when one series is hidden

Merge pull request #239 from timeu/bugfix_upstream

#Issue 333: Pass point index to drawHighlightPointCallback and drawPointCallback

Fixed minor issues

Added test for passing the correct idx to pointDrawCallback and hightlightPointDrawCallback

Code formatting

Added idx parameter to option-reference

#Bug 333: Support for passing point index to drawPointCallback and drawHighlightPointCallback added

Make dygraph-options.js warning-free under Closure Compiler

dygraph-utils.js: Make Dygraph.error and Dygraph.update public

These seem like reasonable utility functions, and they were used outside
of Dygraph (specifically in DygraphOptions). Making these non-private
resolves Closure Compiler warnings inside dygraph-options.js.

dygraph-utils.js: Remove trailing white space

dygraph-tickers.js: Fix variable redeclaration

dygraph-utils.js: isElementContainedBy -> isNodeContainedBy

This fixes a type nit when using the Closure Compiler, since walking
parentNode can give an Element or Document node, not just an Element.

Some closurifying dygraph-canvas and dygraph-layout.

Merge pull request #238 from kberg/master

Add test suite for DygraphOptions object. Fix broken method in it.

Add test suite for DygraphOptions object. Fix broken method in it.

Closurize rgbcolor.js

Closure fix type Dyraph->Dygraph.

Closurize dygraph-options.js. Not done.

Also fix bug: this.labels_ was never used! Booo...

Closure tweak: variable 'd' declared twice within the same method.

Closure fix; force cast from Element to Node.

Merge pull request #234 from wimme/patch-2

Fix in Dygraph.log for IE8

Fix in Dygraph.log for IE8

In IE8, the console.log functions aren't true Javascript functions. They don't support the "apply" or "call" methods.

All Javascript code stopped working when a log message was being generated, therefor this small fix.

Merge pull request #226 from vlsi/wrong_stacked_computation

Fixed computation of stacked series in case some series start in the mid...

Merge pull request #231 from paulfelix/master

Fix for Issue 439: Dragging a range Selector grab handle is not working properly when graph's position on page is offset

Fix for issue #439

Merge pull request #230 from clocksmith/master

Fix broken test file

Fix syntax error in callback.js

Merge pull request #229 from clocksmith/master

Fix the bug intorduced from adding ypad and ypadCompat to no longer return NaN values for the yAxisRange. Test added to guarantee that the underlay callback is called even when there are no series and that the yAxisRange is valid.

Added tests for explicit y axis range in callback.

Fix value of yMax in callback.js test.

Merge branch 'master' of https://github.com/danvk/dygraphs

Add tests for underlayCallback with no series.

Fixed computation of stacked series in case some series start in the middle of the range
The bug is caused by stale value of last_x local across loop in gatherDatasets_.
See https://code.google.com/p/dygraphs/issues/detail?id=438

Merge pull request #225 from vlsi/master

Fix wrong values in legend when using stackedGraph with missing values

Merge pull request #224 from kberg/i294

Use the correct access when computing bounded y-values for constrained panning.

Fix wrong values in legend when using stackedGraph with missing values
Consider all the series when searching previous stacked one and use Dygraph.isValidPoint to validate the point (not just !yval)
See http://jsfiddle.net/6W7EQ/2/

Use the correct access when computing bounded y-values for constrained panning.

Dygraphs was computing the second y-axis using extreme range of the first y-axis. This fixes bug 294, setting panEdgeFraction to non-zero breaks secondary y-axis label computation.

Also add a minor optimization by moving a computation outside the axis loop.

i

i

Merge pull request #222 from kberg/master

Fix bug 436, and include a lint change.

Fix bug 436. Only restore axes that remain after an option update.

Temporarily disable tests which must be run in eastern time

Merge branch 'master' of github.com:danvk/dygraphs

Fix an issue where annotations are set too early in IE8

Fix & regression test for issue 392: connectSeparated and errorBars don't play nicely together

add note about IE8

guard window.getComputedStyle call for IE8

Merge pull request #221 from walkingeyerobot/master

can't change constants

Since we can't change constants, checking to see if they exist instead of always changing the value.

getComputedStyle -> window.getComputedStyle

Merge pull request #220 from walkingeyerobot/master

adding missing license block to dygraph-options.js

adding missing license block

Ignore zooms entirely outisde the plot area; Fix broken tests.

Clean up some zooming oddities; fixes issues 434 and 280

kberg review comments

Fix & regression test for issue 433: Glitch in two-hourly ticks when crossing a "spring forward" daylight savings switch

One more case in which ticks across DST boundaries are still broken

A few tweaks to make all tests pass on FF 3.5

factor out Dygraph.setDateSameTZ

Fix some more DST issues

Fix and regression test for Issue 147: X-axis date labels break after DST transitions

Return failure code when lint fails.

Fix & regression test for issue 363: Calling setSelection() after an updateOptions may have no effect

fix & regression test for issue 355: Row number Issue

Fix a few lint errors

Move KMB/KMG2 formatting into default formatter function (issue 414)

Fix issue 176, which involved duplicated x-values in stacked graphs

Add another test for DST transitions

Regression test for issue 430: Incorrect rendering when all values are negative and the same

Merge pull request #210 from klausw-g/range-pad-2

Add new options xRangePad and yRangePad

Merge pull request #216 from klausw-g/issue-424-b

Re-do the fix for stacked graph highlighting

Merge pull request #217 from paulfelix/master

Fix for Issue 378: Mouse gradually drifts off of range selection handle

Merge pull request #218 from walkingeyerobot/master

Make dygraph-tickers more independent

rv 34ad56b341, which introduced issue 432

Making dygraph-tickers.js slightly more independent.

code review: rename to .singleSeriesName

Call addEvent on graph instance where needed for auto removal

Merge branch 'master' of github.com:danvk/dygraphs

Fix for Issue 378: Mouse gradually drifts off of range selection handle

add missing semicolon

Merge branch 'master' of github.com:danvk/dygraphs

add Great Lakes Water Level dashboard example

Re-do the fix for stacked graph highlighting

This reverts the change from https://github.com/danvk/dygraphs/pull/211
and instead provides a boolean arg to the plotter so that it can decide
not to draw if needed.

The problem with the approach from #211 is that this prevents error bars
from being highlighted, and those should be redrawn on mouseover.

Fixes issue 424.

Address code review comments

Remove naming of anoymous functions. Unnecessary and slightly wrong.

Merge pull request #213 from kberg/i329

Fix bug 329 which accumulates clipping rectangles on subsequent draws.

Fix bug 329 which accumulates clipping rectangles on subsequent draws.

Merge pull request #211 from klausw-g/issue-424

Don't redraw graph stack in interactive highlight

Merge pull request #212 from kberg/i153

Fix bug 153, findPosX and findPosY don't take borders into account. This...

Fix bug 153, findPosX and findPosY don't take borders into account. This also fixes the zoom operation when the graph is surrounded by something with a border.

remove console.log()

Don't redraw graph stack in interactive highlight

When drawing a single series during interactive highlight,
only use the last plotter.

Fixes issue 424.

auto_test for stacked annotations

Fix the other half of Issue 256 (annotation stacking)

Add new options xRangePad and yRangePad

These options provide a fixed margin (in pixels) around the
user-specified or autoselected axis range to ensure all data points are
fully visible. If these are not set by the user, behavior reverts to
backward compatible mode.

This helps avoid the issue of data points drawn at the graph edges not
being properly visible.  For example, in a 100-pixel high plot area, a Y
coordinate of zero on a graph with yrange [0, ...] will be scaled to
domY=100 which is outside the 0..99 visible area.

If I'm understanding it right, the original padding algorithm works
as follows:

  Add 10% padding for automatic Y ranges, but not for user-supplied
  ranges, and move a close-to-zero edge to zero except if avoidMinZero
  is set. If logscale is set, add a variable amount of padding at the
  top but none at the bottom.

This is not very intuitive, and lines drawn at the edge of a
user-supplied range will still be invisible.

The new algorithm consistently ensures that there are at least
{x,y}RangePad pixels available for drawing outside of the data range,
both for user-specified and autoselected ranges.

Setting a small xRangePad combines nicely with drawAxesAtZero,
it will provide tick marks next to the legend due to the grid extending
slightly past the axis.

See the included gallery demo 'Edge Padding'. The default setting
corresponds to legacy behavior. Recommended settings:
  includeZero: true,
  drawAxesAtZero: true,
  xRangePad: 4,
  yRangePad: 10