Add comment

Add test for custom bars with negative values in logScale

LogScale and customBars with negative values

When LogScale and customBars are enabled, it was verifying if the value was negative by using a <= on an array and then inserting a null. This caused exceptions in rollingAverage(), the value can't be null when using customBars.

(This a new patch based on the current code, since pull request #236 was getting old).

Remove the vestigial DygraphLayout.setDateWindow.
(Also, add a clarifying comment.)

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

Fix failing test

Merge pull request #241 from sauter-hq/per-axis-grid

per-axis grid options

Merge remote-tracking branch 'official/master' into per-axis-grid

Conflicts:
tests/two-axes.html

Merge pull request #243 from sauter-hq/fix-for-issue-451

Fix for issue 451

REFACTORING: fixed code style and lint warnings. Readded jsdoc lost in merge.

REFACTORING: Fixed code style nits and extracted the null values check to a method.

Merge commit '9f890c23ad80924d0a30f3a14f8680b7c2d6318e' into fix-for-issue-451

Conflicts:
dygraph-layout.js

only reserve space for y2 axis when it will be drawn

REFACTORING: cleaned up code and style for new grid per axis feature.

Merge commit 'de2545148870a1bdb0957c4c42e80bdb8ce1656d' into per-axis-grid

Fix our lone lint error

Fix syntax errors in docs and update test script to check for them.

Stack Overflow link on home page

Merge pull request #246 from davidmsibley/master

Quarterly granularity ticker fix

Merge pull request #245 from sauter-hq/custom-bar-data-gaps-459

Custom bar data gaps 459

Merge pull request #237 from kberg/closure

Fix closure bugs in Dygraphs

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

BUGFIX: Fixed test not running die to OS differences. Now the edge pixels arn't checked anymore which works fine on linux and windows.

TEST: Added NaN cases to the missingPoints test.

REFACTORING: Changed if / else cases to optimize the performance for the
default cases.

adding comments for LONG_TICK_PLACEMENTS

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

Merge pull request #242 from sauter-hq/draw-xaxis-bugfix

BUGFIX: RangeSelector: xAxisLabelHeight is set to 0 in case the option 'drawXAxis' is set to false.

tying all the large granularity tick decision making together

corrected number of months in a QUARTERLY granularity

TEST: Added automatet tests for customBar and errorBar options and missing data points.

BUGFIX: added missing spaces.

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

BUGFIX: Fixed typo

TEST: testGridLinePattern: Fixed bug not correctly resetting the x-value
in each loop

Merge branch 'per-axis-grid' of https://github.com/sauter-hq/dygraphs into per-axis-grid

REFACTORING: Fixed wrong indents, missing spaces and too much spaces :-)

BUGFIX: Fixed jittering test results by ignoring the alpha value in the color checks.

REFACTORING: fixed jslint violation

REFACTORING: adapted EOL

EXAMPLE: added new example for fixed bug

FEATURE: extended test for the bugfix for issue #451

BUGFIX: bugfix for issue #451 is now also working for bars

TEST: Added examples for the bugfix.

REFACTORING: moved option retrieving out of the for loop for better
performance.

TEST: test to verify the bugfix

BUGFIX: First try of fixing missing data bug for customBars and errorBars

TEST: Added example of range selector without the chart.

TEST: Added automated test proving the bugfix made.

REFACTORING: use the method "getLeftBoundary" instead of a private
variable

BUG FIX: Changed check to undefined with === as proposed by jslint.

REFACTORING: Added exception if no y axis has the independentTicks
option activated.

REFACTORING: Spelling corrections and better documentation.

BUGFIX: Fixed invalid option reference

TEST: Added automated tests for new grid features.

TEST: Added examples of new grid options and features.

FEATURE: Added documentation of new gridLinePattern and adapted other
grid options

FEATURE: Added support for per-axis grid options and added new option:
"gridLinePattern"

TEST: adapted test for bugfix

BUGFIX #451: fixed "connect-separated-points" bug

REFACTORING: corrected the name for test "step_plot_per_series"

Merge pull request #233 from lucasbergman/master

some small Closure-cleanliness fixes

FEATURE: added test for issue #451

BUGFIX: xAxisLabelHeight is set to 0 in case the option 'drawXAxis' is
set to false.

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 branch 'master' of https://github.com/danvk/dygraphs into i382

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 branch 'i382' of https://github.com/kberg/dygraphs into i382

Merge branch 'i382' of https://github.com/kberg/dygraphs into i382

Fix bug 382. Simplify resizing by not recreating everything from scratch. This makes it simpler to deal with i328, since we don't have to worry about unremoved event handlers.

Since we're not building everything from scratch, we have to make sure the clipping area is set correctly. Although there's a resetClip method in the HTML5 spec, it's too new. But, man, if we had it things would be great. So instead there's a conditional restore (no restore on the first operation) and a save, for the subsequent restore.

Almost all tests pass, but the ones that test save/restore balances are unsurprisingly broken, as there's one additional save. This can be addressed with calls to g.destroy prior to the assertion, or make a special assertion that verifies there's one more context save than restore.

This has an additional benefit of being slightly faster. Using /Users/konigsberg/git/kberg-dygraphs/tests/resize.html and the following snippet

var start = new Date().getTime();
for (var idx = 0; idx < 500; idx++) {
    g.resize(100 + idx, 100 + idx);
}
console.log(new Date().getTime() - start);

I frequently saw a difference of about 200ms (2200-range to 2400-range.) How much of that was drawing a mildly complex image? Who knows?

I also did some minor refactoring with regards to events, and event method names to make things easier to track.

There may be a need for a plug-in event for resizing.

Fix bug 328. Simplify resizing by not recreating everything from scratch. This makes it simpler to deal with i328, since we don't have to worry about unremoved event handlers.

Since we're not building everything from scratch, we have to make sure the clipping area is set correctly. Although there's a resetClip method in the HTML5 spec, it's too new. But, man, if we had it things would be great. So instead there's a conditional restore (no restore on the first operation) and a save, for the subsequent restore.

Almost all tests pass, but the ones that test save/restore balances are unsurprisingly broken, as there's one additional save. This can be addressed with calls to g.destroy prior to the assertion, or make a special assertion that verifies there's one more context save than restore.

This has an additional benefit of being slightly faster. Using /Users/konigsberg/git/kberg-dygraphs/tests/resize.html and the following snippet

var start = new Date().getTime();
for (var idx = 0; idx < 500; idx++) {
    g.resize(100 + idx, 100 + idx);
}
console.log(new Date().getTime() - start);

I frequently saw a difference of about 200ms (2200-range to 2400-range.) How much of that was drawing a mildly complex image? Who knows?

I also did some minor refactoring with regards to events, and event method names to make things easier to track.

There may be a need for a plug-in event for resizing.

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.