From: Moriel Schottlender <moriel@gmail.com>
Date: Mon, 6 Mar 2017 23:40:12 +0000 (-0800)
Subject: RCFilters UI: Fix scrolling (once again) so it works in Firefox
X-Git-Tag: 1.31.0-rc.0~3868
X-Git-Url: http://git.cyclocoop.org//%27%40script%40/%27?a=commitdiff_plain;h=2c8b1ec8208e2bd7801f855126839d062f515a87;p=lhc%2Fweb%2Fwiklou.git

RCFilters UI: Fix scrolling (once again) so it works in Firefox

In firefox our scrollable container is html, we need to account
for that along with body element.

Change-Id: Ida603d419c85a2bf27ce8a5d46da7da3c1dc00ea
---

diff --git a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterWrapperWidget.js b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterWrapperWidget.js
index 886dc43a6c..1b3812c86b 100644
--- a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterWrapperWidget.js
+++ b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterWrapperWidget.js
@@ -187,7 +187,7 @@
 	mw.rcfilters.ui.FilterWrapperWidget.prototype.scrollToTop = function ( $element, marginFromTop ) {
 		var container = OO.ui.Element.static.getClosestScrollableContainer( $element[ 0 ], 'y' ),
 			pos = OO.ui.Element.static.getRelativePosition( $element, $( container ) ),
-			containerScrollTop = $( container ).is( 'body' ) ? 0 : $( container ).scrollTop();
+			containerScrollTop = $( container ).is( 'body, html' ) ? 0 : $( container ).scrollTop();
 
 		// Scroll to item
 		$( container ).animate( {