From 149b55f8aeac76c2efdab1373797ef0b7a7de81a Mon Sep 17 00:00:00 2001 From: Vincent Petry Date: Wed, 15 Oct 2014 10:14:20 +0200 Subject: [PATCH] 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 --- apps/files/js/filelist.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) 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 }, { -- 2.39.5