Skip to content
This repository
Browse code

Eliminates need for an IE6/7-specific workaround class, as well as ad…

…ditional scripting.
  • Loading branch information...
commit 82e6fe7393538391d34d680ad979116f6298ffd5 1 parent 7376b45
Mat Marquis authored August 21, 2012
10  css/structure/jquery.mobile.popup.css
... ...
@@ -1,19 +1,15 @@
1 1
 .ui-popup-screen {
  2
+	background-image: url(data:image/gif;base64,R0lGODlhAQABAID/AMDAwAAAACH5BAEAAAAALAAAAAABAAEAAAICRAEAOw==); /* Necessary to set some form of background to ensure element is clickable in IE6/7. While legacy IE won’t understand the data-URI’d image, it ensures no additional requests occur in all other browsers with little overhead. */
2 3
 	top: 0px;
3 4
 	left: 0px;
4 5
 	right: 0px;
5 6
 	bottom: 1px;
6 7
 	position: absolute;
7  
-	filter: Alpha(Opacity=50);
  8
+	filter: Alpha(Opacity=0);
8 9
 	opacity: 0;
9 10
 	z-index: 1099;
10 11
 }
11 12
 
12  
-.ui-popup-screen-background-hack {
13  
-	background-color: black;
14  
-	filter: Alpha(Opacity=0);
15  
-}
16  
-
17 13
 @-webkit-keyframes popup-fadein {
18 14
     from { opacity: 0; }
19 15
     to { opacity: 0.5; }
@@ -36,12 +32,14 @@
36 32
 
37 33
 .ui-popup-screen.fade.in {
38 34
 	opacity: 0.5;
  35
+	filter: Alpha(Opacity=50);
39 36
 	-webkit-animation-name: popup-fadein;
40 37
 	-moz-animation-name: popup-fadein;
41 38
 }
42 39
 
43 40
 .ui-popup-screen.fade.out {
44 41
 	opacity: 0;
  42
+	filter: Alpha(Opacity=0);
45 43
 	-webkit-animation-name: popup-fadeout;
46 44
 	-moz-animation-name: popup-fadeout;
47 45
 }
9  js/widgets/popup.js
@@ -222,15 +222,6 @@ define( [ "jquery",
222 222
 		_setOverlayTheme: function( value ) {
223 223
 			this._applyTheme( this._ui.screen, value, "overlay" );
224 224
 
225  
-			if ( $.mobile.browser.ie ) {
226  
-				this._ui.screen.toggleClass(
227  
-					"ui-popup-screen-background-hack",
228  
-					( this._ui.screen.css( "background-color" ) === "transparent" &&
229  
-						this._ui.screen.css( "background-image" ) === "none" &&
230  
-						( this._ui.screen.css( "background" ) === undefined ||
231  
-							this._ui.screen.css( "background" ) === "" ) ) );
232  
-			}
233  
-
234 225
 			if ( this._isOpen ) {
235 226
 				this._ui.screen.addClass( "in" );
236 227
 			}

0 notes on commit 82e6fe7

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