Skip to content
This repository
Browse code

Fixes #3723

  • Loading branch information...
commit 43f2d61cf8b760ba39b1a9d03d2d721ccd05776c 1 parent e42b785
John Bender authored

Showing 1 changed file with 5 additions and 5 deletions. Show diff stats Hide diff stats

  1. 10  js/jquery.mobile.init.js
10  js/jquery.mobile.init.js
@@ -40,6 +40,8 @@ define( [ "jquery", "./jquery.mobile.core", "./jquery.mobile.support", "./jquery
40 40
 
41 41
 	// For non-fixed supportin browsers. Position at y center (if scrollTop supported), above the activeBtn (if defined), or just 100px from top
42 42
 	function fakeFixLoader(){
  43
+		var activeBtn = $( "." + $.mobile.activeBtnClass ).first();
  44
+
43 45
 		$loader
44 46
 			.css({
45 47
 				top: $.support.scrollTop && $window.scrollTop() + $window.height() / 2 ||
@@ -64,18 +66,16 @@ define( [ "jquery", "./jquery.mobile.core", "./jquery.mobile.support", "./jquery
64 66
 		$html.removeClass( "ui-mobile-rendering" );
65 67
 	}
66 68
 
67  
-
68 69
 	$.extend($.mobile, {
69 70
 		// turn on/off page loading message.
70 71
 		showPageLoadingMsg: function( theme, msgText, textonly ) {
71 72
 			$html.addClass( "ui-loading" );
72 73
 
73 74
 			if ( $.mobile.loadingMessage ) {
74  
-				var activeBtn = $( "." + $.mobile.activeBtnClass ).first(),
75  
-					theme = theme || $.mobile.loadingMessageTheme,
76  
-					// text visibility from argument takes priority
77  
-					textVisible = textonly || $.mobile.loadingMessageTextVisible;
  75
+				// text visibility from argument takes priority
  76
+				var textVisible = textonly || $.mobile.loadingMessageTextVisible;
78 77
 
  78
+				theme = theme || $.mobile.loadingMessageTheme,
79 79
 
80 80
 				$loader
81 81
 					.attr( "class", loaderClass + " ui-corner-all ui-body-" + ( theme || "a" ) + " ui-loader-" + ( textVisible ? "verbose" : "default" ) + ( textonly ? " ui-loader-textonly" : "" ) )

0 notes on commit 43f2d61

Please sign in to comment.
Something went wrong with that request. Please try again.