https://www.mediawiki.org/wiki/Special:Code/MediaWiki/114786

Revision: 114786
Author:   raindrift
Date:     2012-04-06 23:16:13 +0000 (Fri, 06 Apr 2012)
Log Message:
-----------
fix for browser differences in scrollTop
followup to r114785

Modified Paths:
--------------
    
trunk/extensions/PageTriage/modules/ext.pageTriage.views/ext.pageTriage.listControlNav.js

Modified: 
trunk/extensions/PageTriage/modules/ext.pageTriage.views/ext.pageTriage.listControlNav.js
===================================================================
--- 
trunk/extensions/PageTriage/modules/ext.pageTriage.views/ext.pageTriage.listControlNav.js
   2012-04-06 23:08:08 UTC (rev 114785)
+++ 
trunk/extensions/PageTriage/modules/ext.pageTriage.views/ext.pageTriage.listControlNav.js
   2012-04-06 23:16:13 UTC (rev 114786)
@@ -100,8 +100,11 @@
                                event.stopPropagation();
                        } );
                        
-                       //$.waypoints('refresh'); // this ends up happening a 
lot. :(
-                       if( $( '#mwe-pt-list-view' ).offset().top > 
$('body').scrollTop() ) {
+                       // Different browsers represent the document's scroll 
position differently.
+                       // I would expect jQuery to deal with this in some 
graceful fashion, but nooo...
+                       var scrollTop = $('body').scrollTop() || 
$('html').scrollTop() || $(window).scrollTop();
+                       
+                       if( $( '#mwe-pt-list-view' ).offset().top > scrollTop ) 
{
                                // turn off floating nav, bring the bar back 
into the list.
                                $( '#mwe-pt-list-control-nav' 
).parent().removeClass('stickyTop');
                                this.floatNav = false;


_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to