From 34312fd66a3492200b880fac612127c278d282bd Mon Sep 17 00:00:00 2001
From: Jukka Kurkela <jukka.kurkela@gmail.com>
Date: Fri, 6 Nov 2020 15:07:15 +0200
Subject: [PATCH] Remove +-10px limit from bar controller (#8014)

* Remove +-10px limit from Bar controller
* Remove unused import, update tests
---
 src/controllers/controller.bar.js  | 10 ++--------
 test/specs/controller.bar.tests.js | 14 +++++++-------
 2 files changed, 9 insertions(+), 15 deletions(-)

diff --git a/src/controllers/controller.bar.js b/src/controllers/controller.bar.js
index 128f0fc033d..89d4d591802 100644
--- a/src/controllers/controller.bar.js
+++ b/src/controllers/controller.bar.js
@@ -1,7 +1,7 @@
 import DatasetController from '../core/core.datasetController';
 import {
 	clipArea, unclipArea, _arrayUnique, isArray, isNullOrUndef,
-	valueOrDefault, resolveObjectKey, _limitValue, sign
+	valueOrDefault, resolveObjectKey, sign
 } from '../helpers';
 
 function getAllScaleValues(scale) {
@@ -428,14 +428,8 @@ export default class BarController extends DatasetController {
 			start += value;
 		}
 
-		// Limit the bar to only extend up to 10 pixels past scale bounds (chartArea)
-		// So we don't try to draw so huge rectangles.
-		// https://github.com/chartjs/Chart.js/issues/5247
-		// TODO: use borderWidth instead (need to move the parsing from rectangle)
 		const startValue = !isNullOrUndef(baseValue) && !floating ? baseValue : start;
-		let base = _limitValue(vScale.getPixelForValue(startValue),
-			vScale._startPixel - 10,
-			vScale._endPixel + 10);
+		let base = vScale.getPixelForValue(startValue);
 
 		if (this.chart.getDataVisibility(index)) {
 			head = vScale.getPixelForValue(start + length);
diff --git a/test/specs/controller.bar.tests.js b/test/specs/controller.bar.tests.js
index c80ad663d77..6cbd36782d4 100644
--- a/test/specs/controller.bar.tests.js
+++ b/test/specs/controller.bar.tests.js
@@ -630,7 +630,7 @@ describe('Chart.controllers.bar', function() {
 		].forEach(function(expected, i) {
 			expect(meta.data[i].x).toBeCloseToPixel(expected.x);
 			expect(meta.data[i].y).toBeCloseToPixel(expected.y);
-			expect(meta.data[i].base).toBeCloseToPixel(522);
+			expect(meta.data[i].base).toBeCloseToPixel(1024);
 			expect(meta.data[i].width).toBeCloseToPixel(46);
 			expect(meta.data[i].options).toEqual(jasmine.objectContaining({
 				backgroundColor: 'red',
@@ -828,10 +828,10 @@ describe('Chart.controllers.bar', function() {
 		var meta0 = chart.getDatasetMeta(0);
 
 		[
-			{b: 522, w: 92 / 2, x: 38, y: 512},
-			{b: 522, w: 92 / 2, x: 166, y: 819},
-			{b: 522, w: 92 / 2, x: 294, y: 922},
-			{b: 522, w: 92 / 2, x: 422.5, y: 0}
+			{b: 1024, w: 92 / 2, x: 38, y: 512},
+			{b: 1024, w: 92 / 2, x: 166, y: 819},
+			{b: 1024, w: 92 / 2, x: 294, y: 922},
+			{b: 1024, w: 92 / 2, x: 422.5, y: 0}
 		].forEach(function(values, i) {
 			expect(meta0.data[i].base).toBeCloseToPixel(values.b);
 			expect(meta0.data[i].width).toBeCloseToPixel(values.w);
@@ -843,8 +843,8 @@ describe('Chart.controllers.bar', function() {
 
 		[
 			{b: 512, w: 92 / 2, x: 89, y: 0},
-			{b: 522, w: 92 / 2, x: 217, y: 0},
-			{b: 522, w: 92 / 2, x: 345, y: 0},
+			{b: 819.2, w: 92 / 2, x: 217, y: 0},
+			{b: 921.6, w: 92 / 2, x: 345, y: 0},
 			{b: 0, w: 92 / 2, x: 473.5, y: 0}
 		].forEach(function(values, i) {
 			expect(meta1.data[i].base).toBeCloseToPixel(values.b);