Fixes issue chartjs#3585. Added check for 0 (otherwise would attempt …

…to take log of 0, resulting in no bars drawn)
jeffcarey · Nov 17, 2016 · 7d6b6d0 · 7d6b6d0
1 parent bb71f69
commit 7d6b6d0
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/scales/scale.logarithmic.js b/src/scales/scale.logarithmic.js
@@ -199,9 +199,13 @@ module.exports = function(Chart) {
 						pixel = me.top + paddingTop + innerDimension * 0.02 + (innerDimension * 0.98/ range * (helpers.log10(newVal)-helpers.log10(me.minNotZero)));
 					}
 				} else {
-					range = helpers.log10(me.end) - helpers.log10(start);
-					innerDimension = me.height - (paddingTop + paddingBottom);
-					pixel = (me.bottom - paddingBottom) - (innerDimension / range * (helpers.log10(newVal) - helpers.log10(start)));
+					if (newVal === 0) {
+						pixel = me.bottom - paddingBottom;
+					} else {
+						range = helpers.log10(me.end) - helpers.log10(start);
+						innerDimension = me.height - (paddingTop + paddingBottom);
+						pixel = (me.bottom - paddingBottom) - (innerDimension / range * ( helpers.log10(newVal) - helpers.log10(start)));
+					}
 				}
 			}
 			return pixel;