Taking the absolute value is actually not necessary

This commit is contained in:
Thomas Khyn 2016-04-14 12:25:50 +12:00
parent b0a38741df
commit 2cac43ef39

View File

@ -54,7 +54,7 @@ module.exports = function (element, axis, value) {
if (axis === 'top' && value >= i.contentHeight - i.containerHeight) {
// don't allow scroll past container
value = i.contentHeight - i.containerHeight;
if (Math.abs(value - element.scrollTop) <= 1) {
if (value - element.scrollTop <= 1) {
// mitigates rounding errors on non-subpixel scroll values
value = element.scrollTop;
} else {
@ -66,7 +66,7 @@ module.exports = function (element, axis, value) {
if (axis === 'left' && value >= i.contentWidth - i.containerWidth) {
// don't allow scroll past container
value = i.contentWidth - i.containerWidth;
if (Math.abs(value - element.scrollLeft) <= 1) {
if (value - element.scrollLeft <= 1) {
// mitigates rounding errors on non-subpixel scroll values
value = element.scrollLeft;
} else {