Cached window in show/hide functions, since I was in the neighborhood.

aapostoliadis · Jan 23, 2012 · 201058b · 201058b
1 parent 1609ea1
commit 201058b
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/js/jquery.mobile.fixedToolbar.js b/js/jquery.mobile.fixedToolbar.js
@@ -132,10 +132,11 @@ define( [ "jquery", "jquery.mobile.widget", "jquery.mobile.core", "jquery.mobile
 		show: function(){
 			var hideClass = "ui-fixed-hidden",
 				$el = this.element,
-				scroll = $( window ).scrollTop(),
+				$win = $( window ),
+				scroll = $win.scrollTop(),
 				elHeight = $el.height(),
 				pHeight = $el.closest( ".ui-page" ).height(),
-				viewportHeight = Math.min( screen.height, $( window ).height() ),
+				viewportHeight = Math.min( screen.height, $win.height() ),
 				tbtype = $el.is( ".ui-header" ) ? "header" : "footer";
 
 				if( this.options.transition && this.options.transition !== "none" &&
@@ -156,10 +157,11 @@ define( [ "jquery", "jquery.mobile.widget", "jquery.mobile.core", "jquery.mobile
 		hide: function(){
 			var hideClass = "ui-fixed-hidden",
 				$el = this.element,
-				scroll = $( window ).scrollTop(),
+				$win = $( window ),
+				scroll = $win.scrollTop(),
 				elHeight = $el.height(),
 				pHeight = $el.closest( ".ui-page" ).height(),
-				viewportHeight = Math.min( screen.height, $( window ).height() ),
+				viewportHeight = Math.min( screen.height, $win.height() ),
 				tbtype = $el.is( ".ui-header" ) ? "header" : "footer";
 
 				if( this.options.transition && this.options.transition !== "none" &&