Merge pull request #2325 from adobe/jasonsanjose/issue-2130

Use scrollTop instead of scrollIntoView for #2130
adobe · Dec 11, 2012 · 7ba8722 · 7ba8722
2 parents 211f899 + c39ff01
commit 7ba8722
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/src/utils/ViewUtils.js b/src/utils/ViewUtils.js
@@ -302,12 +302,18 @@ define(function (require, exports, module) {
             elementOffset = $element.offset();
 
         // scroll minimum amount
-        if (elementOffset.top + $element.height() >= (viewOffset.top + $view.height())) {
+        var delta = (elementOffset.top + $element.height()) - (viewOffset.top + $view.height());
+
+        if (delta > 0) {
             // below viewport
-            element.scrollIntoView(false);
-        } else if (elementOffset.top <= viewOffset.top) {
-            // above viewport
-            element.scrollIntoView(true);
+            $view.scrollTop($view.scrollTop() + delta);
+        } else {
+            delta = viewOffset.top - elementOffset.top;
+
+            if (delta > 0) {
+                // above viewport
+                $view.scrollTop($view.scrollTop() - delta);
+            }
         }
 
         if (scrollHorizontal) {