From 7c9976f1d984f48153cdc432ed9580710f75ac7a Mon Sep 17 00:00:00 2001 From: Jermolene Date: Thu, 2 Apr 2015 15:21:49 +0100 Subject: [PATCH] Partially revert to 5.1.7 scroll behaviour MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit We’re keeping the new logic that prevents unnecessary scrolling at startup Fixes #1543 --- core/modules/utils/dom/scroller.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/core/modules/utils/dom/scroller.js b/core/modules/utils/dom/scroller.js index 91e66a0ae..cc0a3d271 100644 --- a/core/modules/utils/dom/scroller.js +++ b/core/modules/utils/dom/scroller.js @@ -73,14 +73,10 @@ PageScroller.prototype.scrollIntoView = function(element) { // returns: new position of the scroll viewport var getEndPos = function(targetPos,targetSize,currentPos,currentSize) { var newPos = currentPos; - // If the target is entirely above/left of the current view, then scroll to its top/left - if((targetPos + targetSize) <= (currentPos + 50)) { newPos = targetPos; // If the target is smaller than the window and the scroll position is too far up, then scroll till the target is at the bottom of the window } else if(targetSize < currentSize && currentPos < (targetPos + targetSize - currentSize)) { newPos = targetPos + targetSize - currentSize; - // If the target is out of view below/right, then just scroll to the top/left - } else if(targetPos > (currentPos + currentSize - 50)) { newPos = targetPos; } // If we are scrolling within 50 pixels of the top/left then snap to zero