From: Vincent Petry <pvince81@owncloud.com>
Date: Wed, 15 Oct 2014 08:14:20 +0000 (+0200)
Subject: Use body element when animating scroll in public page
X-Git-Tag: v7.0.3RC1~7
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=149b55f8aeac76c2efdab1373797ef0b7a7de81a;p=nextcloud-server.git

Use body element when animating scroll in public page

In the public page the scroll container is the window instead of a div.
The $(window) object doesn't support animating the scroll property, so
the $('body') element is used instead.

Backport of 704ffaa6a37730f02a3359e8e8f0a0070f0e0d31 from master
---

diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 34d4885cdce..d08e77f5a5a 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -1897,7 +1897,13 @@
 
 			// Animation
 			var _this = this;
-			this.$container.animate({
+			var $scrollContainer = this.$container;
+			if ($scrollContainer[0] === window) {
+				// need to use "body" to animate scrolling
+				// when the scroll container is the window
+				$scrollContainer = $('body');
+			}
+			$scrollContainer.animate({
 				// Scrolling to the top of the new element
 				scrollTop: currentOffset + $fileRow.offset().top - $fileRow.height() * 2 - additionalOffset
 			}, {