]> git.ipfire.org Git - thirdparty/Chart.js.git/log
thirdparty/Chart.js.git
9 years agoCorrection of spelling mistake 3032/head
Neabfi [Mon, 25 Jul 2016 14:53:28 +0000 (10:53 -0400)] 
Correction of spelling mistake

9 years agoCorrection of spelling mistake
Neabfi [Mon, 25 Jul 2016 14:50:35 +0000 (10:50 -0400)] 
Correction of spelling mistake

9 years agoMerge pull request #3027 from moberwasserlechner/bugfix/3026
Evert Timberg [Sun, 24 Jul 2016 13:02:47 +0000 (09:02 -0400)] 
Merge pull request #3027 from moberwasserlechner/bugfix/3026

Remove non existing property from radar samples #3026

9 years agoRemove non existing property from radar samples #3026 3027/head
Michael Oberwasserlechner [Sun, 24 Jul 2016 08:49:03 +0000 (10:49 +0200)] 
Remove non existing property from radar samples #3026

9 years agoMerge pull request #2947 from chartjs/fix/524 v2.2.0-rc.2
Evert Timberg [Tue, 19 Jul 2016 22:56:09 +0000 (18:56 -0400)] 
Merge pull request #2947 from chartjs/fix/524

Add polar area start angle setting

9 years agoMerge pull request #2998 from chartjs/v2.2.0-rc.2-release
Evert Timberg [Tue, 19 Jul 2016 22:52:03 +0000 (18:52 -0400)] 
Merge pull request #2998 from chartjs/v2.2.0-rc.2-release

Bump version to v2.2.0-rc.2

9 years agoBump version to v2.2.0-rc.2 2998/head
Evert Timberg [Tue, 19 Jul 2016 22:47:08 +0000 (18:47 -0400)] 
Bump version to v2.2.0-rc.2

9 years agoFix missing comma syntax error in js block
Nick Downie [Tue, 19 Jul 2016 21:01:41 +0000 (22:01 +0100)] 
Fix missing comma syntax error in js block

9 years agoMerge pull request #2984 from slinhart/master
Evert Timberg [Tue, 19 Jul 2016 02:25:54 +0000 (22:25 -0400)] 
Merge pull request #2984 from slinhart/master

Issue #1012. Added offsetAngle option for radar charts.

9 years agoMerge pull request #2986 from sibennayak/master
Evert Timberg [Mon, 18 Jul 2016 22:18:58 +0000 (18:18 -0400)] 
Merge pull request #2986 from sibennayak/master

Fix the Pie Chart hover issue (#2983)

9 years agoMerge pull request #2991 from tiagoroldao/issue-2990
Evert Timberg [Mon, 18 Jul 2016 22:18:36 +0000 (18:18 -0400)] 
Merge pull request #2991 from tiagoroldao/issue-2990

Check for existence of data element before accessing property

9 years agoCheck for existence of data element before accessing property 2991/head
Tiago Roldão [Mon, 18 Jul 2016 21:05:44 +0000 (22:05 +0100)] 
Check for existence of data element before accessing property

9 years agochanged option name for radar chart from offsetAngle to startAngle. Added test to... 2984/head
unknown [Mon, 18 Jul 2016 05:11:30 +0000 (23:11 -0600)] 
changed option name for radar chart from offsetAngle to startAngle. Added test to make sure correct angles are computed for all points in the radar chart (with and without startAngle option set).

9 years agoMerge branch 'master' of https://github.com/sibennayak/Chart.js 2986/head
Siben Nayak [Sun, 17 Jul 2016 13:35:21 +0000 (19:05 +0530)] 
Merge branch 'master' of https://github.com/sibennayak/Chart.js

9 years agoFIx the Pie Chart hover issue (#2983)
Siben Nayak [Sun, 17 Jul 2016 13:31:59 +0000 (19:01 +0530)] 
FIx the Pie Chart hover issue (#2983)

9 years agoFIx the Pie Chart hover issue (#2983)
Siben Nayak [Sun, 17 Jul 2016 13:31:59 +0000 (19:01 +0530)] 
FIx the Pie Chart hover issue (#2983)

9 years agoMerge remote-tracking branch 'chartjs-origin/master'
unknown [Sun, 17 Jul 2016 00:59:22 +0000 (18:59 -0600)] 
Merge remote-tracking branch 'chartjs-origin/master'

9 years agoAdd an option for radar chart to have offsetAngle. Issue #1012
unknown [Sun, 17 Jul 2016 00:58:04 +0000 (18:58 -0600)] 
Add an option for radar chart to have offsetAngle. Issue #1012

9 years agoMerge pull request #2925 from Joshuabaker2/master
Evert Timberg [Sat, 16 Jul 2016 18:46:59 +0000 (14:46 -0400)] 
Merge pull request #2925 from Joshuabaker2/master

Ability to manually set bar chart bar width

9 years agoFix missing Chart.js.zip file for Travis release v2.2.0-rc.1
Simon Brunel [Fri, 15 Jul 2016 08:24:56 +0000 (10:24 +0200)] 
Fix missing Chart.js.zip file for Travis release

9 years agoMerge pull request #2971 from moberwasserlechner/patch-2
Evert Timberg [Thu, 14 Jul 2016 13:42:19 +0000 (09:42 -0400)] 
Merge pull request #2971 from moberwasserlechner/patch-2

Docs for Title and Legend position incomplete

9 years agoDocs for Title and Legend position incomplete 2971/head
Michael Oberwasserlechner [Thu, 14 Jul 2016 10:30:34 +0000 (12:30 +0200)] 
Docs for Title and Legend position incomplete

Positioning of title and legend to the left or to the right works.

9 years agoPackage.json for v.2.2.0-rc.1 and fixed travis deploy
Evert Timberg [Thu, 14 Jul 2016 00:00:12 +0000 (20:00 -0400)] 
Package.json for v.2.2.0-rc.1 and fixed travis deploy

9 years agoMerge pull request #2926 from chartjs/fix/2915
Evert Timberg [Wed, 13 Jul 2016 22:22:11 +0000 (18:22 -0400)] 
Merge pull request #2926 from chartjs/fix/2915

Fix bar and line controllers to convert strings to numbers when stacking

9 years agoMerge pull request #2918 from chartjs/fix/2916
Evert Timberg [Wed, 13 Jul 2016 22:21:57 +0000 (18:21 -0400)] 
Merge pull request #2918 from chartjs/fix/2916

Fix time point lines. Improved logic for using the value in the time …

9 years agoMerge pull request #2961 from moberwasserlechner/patch-1
Evert Timberg [Wed, 13 Jul 2016 13:13:36 +0000 (09:13 -0400)] 
Merge pull request #2961 from moberwasserlechner/patch-1

Fix copy typo

9 years agoFix copy typo 2961/head
Michael Oberwasserlechner [Wed, 13 Jul 2016 06:49:42 +0000 (08:49 +0200)] 
Fix copy typo

9 years agoAdded support for manually specifying bar thickness in bar charts 2925/head
Josh Baker [Wed, 6 Jul 2016 20:06:43 +0000 (13:06 -0700)] 
Added support for manually specifying bar thickness in bar charts

9 years agoUpdate link to Colors in documentation for each chart type
zachpanz88 [Tue, 12 Jul 2016 15:33:48 +0000 (11:33 -0400)] 
Update link to Colors in  documentation for each chart type

9 years agoMerge pull request #2883 from Mesonyx/mesonyx/bugfix/null-tooltips
Evert Timberg [Mon, 11 Jul 2016 14:34:20 +0000 (10:34 -0400)] 
Merge pull request #2883 from Mesonyx/mesonyx/bugfix/null-tooltips

Fix for null value label/x-axis multi-tooltip bug

9 years agoBring back capping of bezier curve points and have an option to disable it (#2948)
Evert Timberg [Sat, 9 Jul 2016 21:24:41 +0000 (17:24 -0400)] 
Bring back capping of bezier curve points and have an option to disable it (#2948)

9 years agoNon numeric y (#2849)
Evert Timberg [Sat, 9 Jul 2016 15:22:25 +0000 (11:22 -0400)] 
Non numeric y (#2849)

* Category scale can now read from the xLabels and yLabels properties.

* Update docs with section regarding the data object.

* Add sample file with non numeric Y and fix animations

9 years agoMerge branch 'amlethojalen-master'
Evert Timberg [Sat, 9 Jul 2016 14:02:08 +0000 (10:02 -0400)] 
Merge branch 'amlethojalen-master'

9 years agoupdate sample to have negative numbers
Evert Timberg [Sat, 9 Jul 2016 14:01:15 +0000 (10:01 -0400)] 
update sample to have negative numbers

9 years agoMerge branch 'master' of https://github.com/amlethojalen/Chart.js into amlethojalen...
Evert Timberg [Sat, 9 Jul 2016 13:39:43 +0000 (09:39 -0400)] 
Merge branch 'master' of https://github.com/amlethojalen/Chart.js into amlethojalen-master

9 years agoMerge pull request #2885 from neilmacintyre/master
Evert Timberg [Sat, 9 Jul 2016 13:12:57 +0000 (09:12 -0400)] 
Merge pull request #2885 from neilmacintyre/master

cached index and length

9 years agoMerge pull request #2905 from shuson/patch-1
Evert Timberg [Sat, 9 Jul 2016 13:12:32 +0000 (09:12 -0400)] 
Merge pull request #2905 from shuson/patch-1

add wikipedia link for DRY explanation

9 years agoFix JSHint warnings and remove commented code 2926/head
Evert Timberg [Sat, 9 Jul 2016 13:08:41 +0000 (09:08 -0400)] 
Fix JSHint warnings and remove commented code

9 years agoAdd polar area start angle setting 2947/head
Evert Timberg [Sat, 9 Jul 2016 13:05:02 +0000 (09:05 -0400)] 
Add polar area start angle setting

9 years agoMerge pull request #2936 from gitname/patch-1
Evert Timberg [Thu, 7 Jul 2016 20:17:20 +0000 (16:17 -0400)] 
Merge pull request #2936 from gitname/patch-1

Fix typo

9 years agoFix typo 2936/head
gitname [Thu, 7 Jul 2016 19:57:49 +0000 (12:57 -0700)] 
Fix typo

9 years agoFixed up formatting suggestions/issues as per comments from zachpanz88 2911/head
amlethojalen [Wed, 6 Jul 2016 22:35:21 +0000 (08:35 +1000)] 
Fixed up formatting suggestions/issues as per comments from zachpanz88

9 years agoFix bar and line controllers to convert strings to numbers when considering the value...
Evert Timberg [Wed, 6 Jul 2016 22:22:42 +0000 (18:22 -0400)] 
Fix bar and line controllers to convert strings to numbers when considering the values for stacking the chart. Simplified the base calculation for the bar charts and added test coverage to ensure that strings will work correctly.

9 years agoMerge pull request #2924 from jeffesp/master
Evert Timberg [Wed, 6 Jul 2016 21:57:31 +0000 (17:57 -0400)] 
Merge pull request #2924 from jeffesp/master

Fixes #2923: swapped tooltip xAlign & yAlign

9 years agoMerge pull request #2870 from chartjs/fix/2802
Evert Timberg [Wed, 6 Jul 2016 21:53:50 +0000 (17:53 -0400)] 
Merge pull request #2870 from chartjs/fix/2802

Consider left and top when determining value from pixel in category s…

9 years agoMerge remote-tracking branch 'chartjs-origin/master'
unknown [Wed, 6 Jul 2016 17:21:57 +0000 (11:21 -0600)] 
Merge remote-tracking branch 'chartjs-origin/master'

9 years agoFixes #2923: swapped tooltip xAlign & yAlign 2924/head
jeffesp [Wed, 6 Jul 2016 13:49:31 +0000 (09:49 -0400)] 
Fixes #2923: swapped tooltip xAlign & yAlign

9 years agoAdded usePointStyle option to label boxes
Shayne Linhart [Wed, 6 Jul 2016 02:08:29 +0000 (20:08 -0600)] 
Added usePointStyle option to label boxes

- Closes #2252
- Allows label boxes to match the shape(pointStyle) of the corresponding data.
* Removed unused varaible from legend's draw()

9 years agoAdded ability to render a line chart as a stepped line chart.
amlethojalen [Wed, 6 Jul 2016 01:13:12 +0000 (11:13 +1000)] 
Added ability to render a line chart as a stepped line chart.
Added option.elements.line.stepped
Added dataset.steppedLine
Added Sample, and updated documentation.

9 years agoFix time point lines. Improved logic for using the value in the time scale. Improve... 2918/head
Evert Timberg [Wed, 6 Jul 2016 00:52:17 +0000 (20:52 -0400)] 
Fix time point lines. Improved logic for using the value in the time scale. Improve value passing in the line and bubble controllers/

9 years agoAdded ability for drawing a Line Chart as a stepped Line Chart. (reverted from commit...
amlethojalen [Wed, 6 Jul 2016 00:25:01 +0000 (10:25 +1000)] 
Added ability for drawing a Line Chart as a stepped Line Chart. (reverted from commit efd06c886de405f00a93c506ef48a4e306c59af0)

9 years agoAdded ability for drawing a Line Chart as a stepped Line Chart.
amlethojalen [Tue, 5 Jul 2016 09:50:53 +0000 (19:50 +1000)] 
Added ability for drawing a Line Chart as a stepped Line Chart.

9 years agoMerge pull request #2867 from mtanda/fix_stacked_scatter_line_chart
Evert Timberg [Tue, 5 Jul 2016 02:57:40 +0000 (22:57 -0400)] 
Merge pull request #2867 from mtanda/fix_stacked_scatter_line_chart

Fix to work stacked scatter line chart (fixes #2878)

9 years agoadd wikipedia link for DRY explanation 2905/head
Shusen Wang [Mon, 4 Jul 2016 06:51:26 +0000 (14:51 +0800)] 
add wikipedia link for DRY explanation

DRY is a short term for Don't Repeat yourself, maybe it is not so well known by front-end developers, but it is well known among back end developers

9 years agoRemoved unused varaible from legend's draw() 2902/head
unknown [Mon, 4 Jul 2016 00:40:15 +0000 (18:40 -0600)] 
Removed unused varaible from legend's draw()

9 years agoMerge remote-tracking branch 'chartjs-origin/master'
unknown [Mon, 4 Jul 2016 00:35:21 +0000 (18:35 -0600)] 
Merge remote-tracking branch 'chartjs-origin/master'

Conflicts:
src/core/core.legend.js

9 years agoMerge pull request #2901 from moberwasserlechner/fix/2894
Evert Timberg [Sun, 3 Jul 2016 21:09:30 +0000 (17:09 -0400)] 
Merge pull request #2901 from moberwasserlechner/fix/2894

Documentation fixes

9 years agoIssue #2252, added usePointStyle option to allow label boxes to match the shape(point...
unknown [Sun, 3 Jul 2016 19:55:43 +0000 (13:55 -0600)] 
Issue #2252, added usePointStyle option to allow label boxes to match the shape(pointStyle) of the corresponding data.

9 years agoAdd more information to README on Building and Testing
zachpanz88 [Sun, 3 Jul 2016 04:24:35 +0000 (00:24 -0400)] 
Add more information to README on Building and Testing

9 years agoFinish incomplete sentence and remove trailing spaces. 2901/head
Michael Oberwasserlechner [Sat, 2 Jul 2016 14:46:29 +0000 (16:46 +0200)] 
Finish incomplete sentence and remove trailing spaces.

9 years agoadd id property to common configuration
Michael Oberwasserlechner [Sat, 2 Jul 2016 14:42:36 +0000 (16:42 +0200)] 
add id property to common configuration

9 years agoremove redundant option entry
Michael Oberwasserlechner [Sat, 2 Jul 2016 14:35:33 +0000 (16:35 +0200)] 
remove redundant option entry

9 years agostepSize defined twice. renaming the first to fixedStepSize
Michael Oberwasserlechner [Sat, 2 Jul 2016 11:55:53 +0000 (13:55 +0200)] 
stepSize defined twice. renaming the first to fixedStepSize

9 years agoMerge pull request #2897 from jimkring/patch-1
Evert Timberg [Sat, 2 Jul 2016 02:49:11 +0000 (22:49 -0400)] 
Merge pull request #2897 from jimkring/patch-1

Update 04-Bar-Chart.md

9 years agoUpdate 04-Bar-Chart.md 2897/head
Jim Kring [Fri, 1 Jul 2016 19:02:55 +0000 (12:02 -0700)] 
Update 04-Bar-Chart.md

Changed second instance of "Options for xAxes" heading to "Options for yAxes" since it is the heading for the y-axis options.

9 years agoMerge pull request #2893 from chartjs/fix/2888
Zach Panzarino [Fri, 1 Jul 2016 13:52:14 +0000 (09:52 -0400)] 
Merge pull request #2893 from chartjs/fix/2888

Fix typo in docs

9 years agoFix typo in docs 2893/head
Evert Timberg [Fri, 1 Jul 2016 12:55:35 +0000 (08:55 -0400)] 
Fix typo in docs

9 years agocached index and length 2885/head
Neil Macintyre [Thu, 30 Jun 2016 22:10:11 +0000 (18:10 -0400)] 
cached index and length

9 years agofix for null tooltip bug 2883/head
Adam Templeton [Thu, 30 Jun 2016 16:34:53 +0000 (11:34 -0500)] 
fix for null tooltip bug

9 years agofix stacked scatter line chart 2867/head
Mitsuhiro Tanda [Wed, 29 Jun 2016 03:35:52 +0000 (12:35 +0900)] 
fix stacked scatter line chart

9 years agoMerge pull request #2865 from nmac143/master
Evert Timberg [Thu, 30 Jun 2016 00:30:02 +0000 (20:30 -0400)] 
Merge pull request #2865 from nmac143/master

Pie chart borders truncated #2844

9 years agoFixed truncation of pie chart borders 2865/head
nmac143 [Tue, 28 Jun 2016 20:33:43 +0000 (16:33 -0400)] 
Fixed truncation of pie chart borders

9 years agoConsider left and top when determining value from pixel in category scale 2870/head
Evert Timberg [Wed, 29 Jun 2016 11:00:09 +0000 (07:00 -0400)] 
Consider left and top when determining value from pixel in category scale

9 years agoMerge pull request #2857 from chartjs/fix/2809
Evert Timberg [Wed, 29 Jun 2016 01:59:39 +0000 (21:59 -0400)] 
Merge pull request #2857 from chartjs/fix/2809

Polar area graph scale should start at 0 by default

9 years agoMerge pull request #2863 from Mesonyx/mesonyx/feature/x-axis-highlighting
Evert Timberg [Tue, 28 Jun 2016 20:15:04 +0000 (16:15 -0400)] 
Merge pull request #2863 from Mesonyx/mesonyx/feature/x-axis-highlighting

adds x-axis hover option for charts and tooltips, highlights all dots…

9 years agomade table more readable 2863/head
Adam Templeton [Tue, 28 Jun 2016 20:05:04 +0000 (15:05 -0500)] 
made table more readable

9 years agoupdated docs with new option
Adam Templeton [Tue, 28 Jun 2016 19:59:22 +0000 (14:59 -0500)] 
updated docs with new option

9 years agoadds x-axis hover option for charts and tooltips, highlights all dots on x-axis upon...
Adam Templeton [Tue, 28 Jun 2016 19:15:12 +0000 (14:15 -0500)] 
adds x-axis hover option for charts and tooltips, highlights all dots on x-axis upon x-axis hover

9 years agoMerge pull request #2555 from simonbrunel/travis-dist
Evert Timberg [Tue, 28 Jun 2016 12:39:57 +0000 (08:39 -0400)] 
Merge pull request #2555 from simonbrunel/travis-dist

Travis GitHub Releases

9 years agoMerge pull request #2860 from azranel/fix-scale-links-in-docs
Evert Timberg [Tue, 28 Jun 2016 11:26:42 +0000 (07:26 -0400)] 
Merge pull request #2860 from azranel/fix-scale-links-in-docs

Change ID links for scales objects

9 years agoMerge pull request #2789 from awallat/awallat-patch-tooltip-multiple-datasets
Evert Timberg [Tue, 28 Jun 2016 11:26:05 +0000 (07:26 -0400)] 
Merge pull request #2789 from awallat/awallat-patch-tooltip-multiple-datasets

Fix tooltip not shown for missing dataset-value

9 years agoChange ID links for scales objects 2860/head
Bartosz Łęcki [Tue, 28 Jun 2016 10:22:41 +0000 (12:22 +0200)] 
Change ID links for scales objects

9 years agoFix tooltip not shown for missing dataset 2789/head
André Wallat [Thu, 16 Jun 2016 11:20:54 +0000 (13:20 +0200)] 
Fix tooltip not shown for missing dataset

When two datasets are shown and they don't have the same x-coordinates, then the position of the tooltip is not calculated correctly, when using mode 'label' or 'dataset'. This fix checks if the position of every dataset exists, before it is used.

9 years agoPolar area graph scale should start at 0 by default 2857/head
Evert Timberg [Tue, 28 Jun 2016 02:25:38 +0000 (22:25 -0400)] 
Polar area graph scale should start at 0 by default

9 years agoMerge pull request #2741 from duerahan/master
William Entriken [Mon, 27 Jun 2016 17:59:41 +0000 (13:59 -0400)] 
Merge pull request #2741 from duerahan/master

time type xAxis height crah in line chart #2622

9 years agoMerge pull request #2848 from ronaldgrn/patch-1
Evert Timberg [Sun, 26 Jun 2016 18:01:01 +0000 (14:01 -0400)] 
Merge pull request #2848 from ronaldgrn/patch-1

Fix line example not showing Scale Titles

9 years agoFix line example not showing Scale Titles 2848/head
ronaldgrn [Sun, 26 Jun 2016 17:07:51 +0000 (14:07 -0300)] 
Fix line example not showing Scale Titles

scaleLabel should have display: true instead of show: true

9 years agoMerge pull request #2841 from chartjs/fix/2839
Zach Panzarino [Fri, 24 Jun 2016 02:32:09 +0000 (22:32 -0400)] 
Merge pull request #2841 from chartjs/fix/2839

Fix typo in tooltip docs

9 years agoFix typo in tooltip docs 2841/head
Evert Timberg [Thu, 23 Jun 2016 23:40:45 +0000 (19:40 -0400)] 
Fix typo in tooltip docs

9 years agoMerge pull request #2823 from chartjs/fix/2708
Evert Timberg [Thu, 23 Jun 2016 02:07:53 +0000 (22:07 -0400)] 
Merge pull request #2823 from chartjs/fix/2708

Use the scale center point for the polar area chart center

9 years agoFix JSHint warning 2823/head
Evert Timberg [Thu, 23 Jun 2016 01:48:49 +0000 (21:48 -0400)] 
Fix JSHint warning

9 years agoMerge pull request #2811 from chartjs/fix/span-gaps-global
Evert Timberg [Thu, 23 Jun 2016 01:46:12 +0000 (21:46 -0400)] 
Merge pull request #2811 from chartjs/fix/span-gaps-global

Span gaps option should be specified at the chart level

9 years agoMerge pull request #2821 from chartjs/fix/2526
Evert Timberg [Thu, 23 Jun 2016 01:46:06 +0000 (21:46 -0400)] 
Merge pull request #2821 from chartjs/fix/2526

Gracefully handle creating the chart with no config

9 years agoMerge pull request #2822 from chartjs/fix/2604
Evert Timberg [Thu, 23 Jun 2016 01:45:59 +0000 (21:45 -0400)] 
Merge pull request #2822 from chartjs/fix/2604

Allow passing a value to the time scale getPixelForValue method

9 years agoMerge pull request #2828 from Kopleman/master
Evert Timberg [Thu, 23 Jun 2016 01:45:32 +0000 (21:45 -0400)] 
Merge pull request #2828 from Kopleman/master

Fix statements in for() in tooltips's getAveragePosition.

9 years agoFix statements in for() in tooltips's getAveragePosition. 2828/head
Alexey Kopleman Dvurechensky [Wed, 22 Jun 2016 11:46:00 +0000 (14:46 +0300)] 
Fix statements in for() in tooltips's getAveragePosition.

9 years agoUse the scale center point for the polar area chart center
Evert Timberg [Tue, 21 Jun 2016 23:57:13 +0000 (19:57 -0400)] 
Use the scale center point for the polar area chart center

9 years agoAllow passing a value to the time scale getPixelForValue method 2822/head
Evert Timberg [Tue, 21 Jun 2016 23:03:30 +0000 (19:03 -0400)] 
Allow passing a value to the time scale getPixelForValue method

9 years agoGracefully handle creating the chart with no config 2821/head
Evert Timberg [Tue, 21 Jun 2016 22:45:22 +0000 (18:45 -0400)] 
Gracefully handle creating the chart with no config

9 years agoMerge pull request #2800 from simonbrunel/#2734
Evert Timberg [Mon, 20 Jun 2016 18:42:59 +0000 (14:42 -0400)] 
Merge pull request #2800 from simonbrunel/#2734

#2734 Fix JSHint warnings