Fix retina scale when display size is implicit

The retinaScale helper now enforces the display size to the correct values because if no style has been set on the canvas, the render size is used as display size, making the chart bigger (or smaller) when deviceAspectRatio is different of 1.

Fix context state restoration on destroy

In many cases, the canvas render size is changed by the lib, causing the state stack to be discarded, meaning that we can't use save() and restore() to release the context with its initial state (i.e. before creating the chart). Since we don't need (want) to manually save / restore the context initial state, simply make sure to reset it to the default state to give a fresh context back to the user. That also means we don't need to revert the scale when the pixel device ratio is not 1.

Bump version to 2.4.0

X and Y interaction modes now use the intersect option

Fix the event handler when legend is disabled (#3544)

When legend is disabled (i.e. {options: {legend: false}}), me.legend is null. Add the same test on me.tooltip even if the tooltip object is always created in case of {options: {tooltips: false}}.

Fix iframe resize handler when re-attached to DOM (#3527)

When the iframe is attached to the DOM, its content is reloaded (invaliding the resize listener) so make sure to install the handler after the iframe is loaded. Optimize resize events by throttling resize process until the next animation frame. Rewrite the unit test "waitForResize" method, the previous one (timeout) was too weak and most tests was failing on FF.

When an event triggers an update while the bufferedUpdate state is true, we need to do that render with priority over any other renders that take place for animations and tooltips

Bump ESLint to v3.x (gulp and Code Climate) (#3526)

Spelling fixes (#3522)

Including: actually, aspect, bottom, changes, characters, datasets, divisible, downward, guidelines, instance, instances, interrupted, item, javascript, label, lifecycle, lines, plugins, manually, milliseconds, nearest, occurring, position, predefined, recalculate, tooltip, those

fix "area" typo

Fixes issue #3490: Tick documentation is missing autoSkipPadding

Adding a single line to the docs to document autoSkipPadding.

Update installation instructions in README

Update chartColors.js to utils.js and move randomScalingFactor function there

Fix bug in 'y' tooltip mode

Reorganized sample files into sub directories. Added a helper containing colours that should be used by all samples.

I added new samples to explain behaviour and modified all samples to have consistent styling. In updating the samples,
I removed the use of jQuery and instead use standard methods.

For the custom tooltip samples, I updated the styling to show color boxes like the regular tooltips.

Add a way to filter items in the tooltip

Custom tooltip: add data points infos (#3201)

Expose tooltip items from tooltip model and added `x` and `y` properties to `TooltipItemInterface`

Add new properties for the caretX,caretY point of a tooltip. Useful for custom tooltips.
The custom tooltip sample was updated as well to use the new properties.

Fix eslint errors in layout service test

Layout service now supports configurable padding on left, top, right and bottom.
Re-enabled the layout service tests and then properly disabled the tests that
fail on the CI.

Add reset method to chart prototype

Extend eslint to test files (#3473)

* Add eslint to test files

* Fix mockContext for tests

* Make formatting look better for nested objects

Enhance context acquisition on chart creation

Add support for creating a chart from the canvas id and prevent exceptions, at construction time, when the given item doesn't provide a valid CanvasRenderingContext2D or when the getContext API is not accessible (e.g. undefined by add-ons to prevent fingerprinting). New jasmine matcher to verify chart validity.

Make index mode only work with the horizontal distance to an element  (#3471)

Make index mode only work with the horizontal distance to an element if intersect is off

Configurable Tooltip Position Modes (#3453)

Adds new tooltip position option that allows configuring where a tooltip is displayed on the graph in relation to the elements that appear in it

Bar chart performance improvements

New fill modes for lines (#3460)

New fill modes for lines allowing the user to customize where the fill goes to

Update the GitHub issue template

No longer merge arrays during the config merge. Simply replace the property

Replaces Unicode character with HTML entity

Fix/3061 (#3446)

Solve weird animation issues with the tooltip. The optimization in Chart.Element.transition when the animation finishes to set `_view = _model` caused problems during update because we were using `helpers.extend` all over the place.

I changed to code so that we regenerate the model variable rather than continuously extending the old version. I also removed unnecessary tooltip reinitializations from the controller which should improve overall performance during interaction.

Properly merge colors for the label colors in the tooltip. I added a private helper to simplify the code in the tooltip

Fix bubble chart tooltip callback to use correct labels (#3421)

Fix bubble chart tooltip callback to use correct label parsed from scales. Fixes #3029

Refactor tooltip draw function to extract drawBackground method

See issue #3416.

If tick options have min, max and stepSize use them to generate evenly spaced ticks

Added the watchify NPM dependency to satisfy karma-browserify's peerDependency

Display tooltip color boxes for all tooltips. Added a new `displayColors` option to turn them off

Improve Tooltip and Hover Interaction (#3400)

Refactored interaction modes to use lookup functions in Chart.Interaction.modes and added new modes for 'point', 'index', 'nearest', 'x', and 'y'

Better animation when adding or removing data

In order to simulate real-time chart updates (i.e. horizontal animation), it's necessary to distinguish a removed or added value from a simple update. The dataset controller now hooks array methods that alter the data array length to synchronize metadata accordingly. Also remove the duplicate calls of updateBezierControlPoints() for line and radar charts.

Fix config initialization and add unit tests

Inject iframe for responsive charts only

Responsiveness is currently based on the use of an iframe, however this method causes performance issues and could be troublesome when used with ad blockers. So make sure that the user is still able to create a chart without iframe when responsive is false.

Fixes HTML legend string for polar area charts to match doughnut charts. (#3361)

Fixes HTML legend string for polar area charts to match doughnut charts

Better number -> string callback for the radial linear scale (#3281)

Also create a new Chart.Ticks namespace to host common tick generators and formatters.

Merge pull request #3325 from chartjs/fix/3269

Improve multiline labeling for tooltips in doughnut charts

Update default config tests and re-enable

In the doughnut chart, specifically handle multiline strings.

skip non finite data points when determining scale sizes. Fixes #3125

Merge pull request #3356 from simonbrunel/aspect-ratio

Fix aspect ratio and add responsive unit tests

Fix 2 minor documentation issues in the scale documentation. #3341 (#3360)

Compute correct tooltip size when there is no title present (#3324)

Fix the legend drawing when `labels.usePointStyle` is true (#3323)

Merge pull request #3358 from dylan-kerr/fix/limit-title-length

Set maxWidth during title draw to avoid overflow

Merge pull request #3357 from kant/patch-1

Minor fixes (proposal)

Initial core.controller.js unit tests

Adjust expected values in core.title.tests.js

Set maxWidth during title draw to avoid overflow

CanvasRenderingContext2D.fillText() accepts a fourth parameter called maxWidth that sets the maximum width of the drawn text, enforced by scaling the entire line.

This commit uses the title element's layout dimensions to set maxWidth and avoid overflow outside of the canvas.

Minor fixes (proposal)

Fix radar default aspect ratio and samples

Now that the aspect ratio is correctly handled, fix samples for charts with aspect ratio of 1 which was vertically too large. Also fix the default aspect ratio for radar charts which wasn't applied when creating a chart directly using new Chart(ctx, { type: 'radar' }).

Fix initial aspect ratio when not responsive

When responsive is false and no canvas height explicitly set, the aspectRatio option wasn't applied because of the canvas default height. Prevent the retinaScale method to change the canvas display size since this method is called for none responsive charts, but instead make the resize() responsible of these changes. Also, as discussed some time ago, moved most of the core.js logic into core.controller.js. Clean up the destroy process and make sure that initial canvas values are properly saved and restored.

Gulp command switch to run specific test files

Add the --inputs command switch to the unittest and unittestWatch tasks, to be able to run unit tests from the specified files only (e.g. gulp unittest --inputs=test/core.element.tests.js;test/core.helpers.tests.js).

Make charts vertically responsive (#3105) (#3326)

Ensure that the hidden iframe is stretched vertically in order to detect height changes. Remove the classlist check/call since it was incorrectly spelled (should be classList), but also useless since the iframe has just been generated. Also remove the callback check: addResizeListener should never be called w/o a valid callback.

Move reverse option for labels to correct section in docs
Addresses comment in #3102

Bump version to 2.3.0

Ensure the second pie chart sits inline

Bump version to 2.3.0-rc.1

Change bower package name for chart.js (#1562)

Thanks to the Bower team, we have been able to register the chart.js package name, previously pointing on a dead repository, which make the Bower integration more consistent with the npm one. All other alias still work (Chart.js, chartjs, Chart-js, chartjs-Chart.js).

Merge pull request #3315 from zachpanz88/fix-pie-multi-dataset

Fix error where pie/doughnut charts did not render correctly with multiple datasets

Fix bug with pie/doughnut chart legends
Fixes a rendering issue when there are multiple datasets on a pie chart and they do not all contain the same number of data in their datasets
Fixes #3309

Merge pull request #3287 from simonbrunel/tests-cleanup

Tests cleanup

Turn off linebreak-style linting rule

On Windows, this rule conflicts when running ESLint locally on a repository setup with autoCrlf (then all lines are errors).

Merge pull request #3308 from zachpanz88/eslint

ESLint Updates

Implement eslint changes proposed by @simonbrunel in code review

Include changes suggested by @etimberg in code review

Allow variable use before define with warning

Fix style issues in core and scales

Fix more style errors in controllers, elements, and scales

Remove func-call-spacing rule

Add keyword spacing to eslint and update code

First round of eslint fixes

Update new eslint rules

Fix broken links in Maintaining.md

Merge pull request #3261 from desowin/3211

Implement moment diff cache in time scale

Fix check for undefined

Use '!==' instead of '!='. This fixes eqeqeq error reported by gulp-eslint.

Merge pull request #3284 from chartjs/fix/3278

Return correct tooltip data in category scale when double category scale is used

When the category scale is used as the non main scale, return the correct data for the tooltip

Cleanup React extension links (#3282)

Remove the existing link which was for Chart.js v1 and add 2 links for Chart.js v2

Merge pull request #3280 from trantorLiu/master

Update 01-Chart-Configuration.md

Update 01-Chart-Configuration.md

Fix an anchor link.

Automatically expose custom Jasmine matchers

Fix time scale tests using the wrong chart instance

Some tests were using the removed global `chartInstance` variable, so tests values were incorrect since against the wrong chart instance.

Auto-release charts if not persistent (tests)

For convenience, charts are now automatically released after each spec if they are not acquired using `persistent: true`. Also remove the confusing and error prone `chartInstance` global variable and make sure that chart instances are local to each spec.

Fix 3267 Add "onHover" functionality for legend (#3271)

Add "onHover" to the legend options that will hold a user defined function (default is null) and called when a "mousemove" event is registered on top of a label item, with same parameters as the "onClick" option.

Also introduced logic that determines if the type of event passed to the legend "handleEvent" function is one we can handle. Currently allowing "click" and "mousemove" events. If the event is not one of those we stop the function execution (this is for the sake of reusing the legend hitbox calculations).

Merge pull request #3254 from ianks/minDisplayFormat

Suport minUnit for time scale

Merge pull request #3256 from simonbrunel/code-cleanup

ESLint and cleanup

Suport minUnit for time scale

When dealing with time-delineated datasets, often we have data for known
intervals of time. For example, we may have a dataset which represents number
of purchases per day:

```json
{
labels: ['2016-01-01', '2016-01-02', '2016-01-03']
datasets: [
{
data: [12, 87, 42]
}
],
'...': '...'
}
```

In this case, Chart.js will attempt to figure out the best interval to display
the data, and could pick `hours` as the unit. However, in this case, we would
prefer to just use the `days` interval since our data's granularity can not be
represented well with `hours`.

To remedy this, this commit adds the `minUnit` option which allows
users to (optionally) specify what the minimum unit they would like
to use.

Build labelDiffs cache once per update (#3211)

Previously buildLabelDiffs() was called at the end of buildTicks() and
hence labelDiffs cache was calculated twice (in getMinimumBoxSize() and
then in fitBox()).

Build label diff lookup table

This reduces number of calls to momentjs diff() if there are multiple
datasets sharing the same labels (#3211).

Enforce curly braces for single statement block

curly: [error, all] (http://eslint.org/docs/rules/curly)

Enforce consistent tab indentation

indent: [error, tab] (http://eslint.org/docs/rules/indent)

Use gulp-eslint instead of gulp-jshint

Change the linter in gulp tasks to be consistent with Code Climate results which are based on ESLint using .eslintrc options. However, defaults Code Climate rules are too strict, so turn as warnings the 'complexity' and 'max-statements' rules (other errors has been fixed). Note that the Gulp task name has been changed for `gulp lint`.