Skip to content
This repository
Browse code

disable popup history managment in old ie by default, Fixes #4784

  • Loading branch information...
commit 46a65c6bd28bbaac59b3c49a7097f66296d72cc0 1 parent fef0eea
John Bender authored August 27, 2012

Showing 1 changed file with 6 additions and 1 deletion. Show diff stats Hide diff stats

  1. 7  js/widgets/popup.js
7  js/widgets/popup.js
@@ -7,6 +7,7 @@
7 7
 
8 8
 define( [ "jquery",
9 9
 	"../jquery.mobile.widget",
  10
+	"../jquery.mobile.support",
10 11
 	"../jquery.mobile.navigation",
11 12
 	"depend!../jquery.hashchange[jquery]" ], function( $ ) {
12 13
 //>>excludeEnd("jqmBuildExclude");
@@ -51,7 +52,11 @@ define( [ "jquery",
51 52
 			closeLinkEvents: "click.popup",
52 53
 			navigateEvents: "navigate.popup",
53 54
 			closeEvents: "navigate.popup pagebeforechange.popup",
54  
-			history: true
  55
+
  56
+			// NOTE Windows Phone 7 has a scroll position caching issue that
  57
+			//      requires us to disable popup history management by default
  58
+			//      https://github.com/jquery/jquery-mobile/issues/4784
  59
+			history: !$.mobile.browser.ie
55 60
 		},
56 61
 
57 62
 		_eatEventAndClose: function( e ) {

0 notes on commit 46a65c6

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