From 2b25bfe65dd8963b42ff502f47565f41e335ec20 Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 24 Sep 2012 13:54:33 -0700 Subject: [PATCH] Revert "Autoscroll sidebar" This reverts commit 3df0cfa6417622aa55fcf1ac927470eee1fd47e0. See D3473 for discussion. This causes some issues with scrolling on Mountain Lion. --- .../application/core/behavior-active-nav.js | 20 ++++--------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/webroot/rsrc/js/application/core/behavior-active-nav.js b/webroot/rsrc/js/application/core/behavior-active-nav.js index a753e15184..43fcc3328f 100644 --- a/webroot/rsrc/js/application/core/behavior-active-nav.js +++ b/webroot/rsrc/js/application/core/behavior-active-nav.js @@ -28,11 +28,8 @@ JX.behavior('phabricator-active-nav', function(config) { link, 'phabricator-active-nav-focus', selected); - if (selected && link.scrollIntoView) { - link.scrollIntoView(false); - } } - }; + } /** @@ -75,24 +72,15 @@ JX.behavior('phabricator-active-nav', function(config) { // If we get above the first marker, select it. selectnav(active && JX.Stratcom.getData(active).anchor); - }; + } var pending = null; var onviewportchange = function(e) { pending && clearTimeout(pending); pending = setTimeout(updateposition, 100); - }; + } - var onscroll = function(e) { - if (e.getNode('tag:body')) { - // If we are inside then it means we just scrolled some