Unable to disable vertical Scroll [since v3.0.0-rc.20-89461bc]

[jercoh]

This commit 89461bc broke the ability to disable vertical scrolling.

Last unstable version (NOT WORKING):
http://plnkr.co/edit/k46LqpiBc1cesOP8xdIM?p=preview

Unstable version v3.0.0-rc.20-d4a784f - 2015-04-21 (WORKING):
http://plnkr.co/edit/35szFnzVSPGinvzJgR8y?p=preview

Also, the scrolling cannot be disabled if the grid has already been scrolled.

[jercoh]

Hi,

Any update on this issue?

Thanks!

[PaulL1]

@swalters: is this in your area?

[mryellow]

All I can do is laugh. This (what should have been a) 2 minute job is going to be my life now.

[mryellow]

This solution is good.

Overloading atTop and atBottom so that this preventDefault() codepath is under your control again.

ui-grid/packages/core/src/js/directives/ui-grid-render-container.js

Lines 106 to 112 in 8892f1d

	// Let the parent container scroll if the grid is already at the top/bottom
	if (!((event.deltaY !== 0 && (scrollEvent.atTop(scrollTop) || scrollEvent.atBottom(scrollTop))) ||
	(event.deltaX !== 0 && (scrollEvent.atLeft(scrollLeft) || scrollEvent.atRight(scrollLeft))))) {
	event.preventDefault();
	event.stopPropagation();
	scrollEvent.fireThrottledScrollingEvent('', scrollEvent);
	}

ScrollEvent.prototype.atTop = function(scrollTop) {
	return true;
};

ScrollEvent.prototype.atBottom = function(scrollTop) {
	return true;
};

https://stackoverflow.com/a/44020606/2438830