RCFilters UI: Fix scrolling (once again) so it works in Firefox
authorMoriel Schottlender <moriel@gmail.com>
Mon, 6 Mar 2017 23:40:12 +0000 (15:40 -0800)
committerMoriel Schottlender <moriel@gmail.com>
Mon, 6 Mar 2017 23:40:12 +0000 (15:40 -0800)
In firefox our scrollable container is html, we need to account
for that along with body element.

Change-Id: Ida603d419c85a2bf27ce8a5d46da7da3c1dc00ea

resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FilterWrapperWidget.js

index 886dc43..1b3812c 100644 (file)
        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( {