Skip to content
This repository
Browse code

Fix issue 1913 - unnecessary ajax call and duplicate DOM nodes when r…

…efreshing a page with a dialog visible
  • Loading branch information...
commit 83973b429b635db1f1a6667eda73c37b4d8838da 1 parent c81b8cc
Martin Sutherland authored

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

  1. 4  js/jquery.mobile.navigation.js
4  js/jquery.mobile.navigation.js
@@ -156,7 +156,9 @@
156 156
 			convertUrlToDataUrl: function( absUrl ) {
157 157
 				var u = path.parseUrl( absUrl );
158 158
 				if ( path.isEmbeddedPage( u ) ) {
159  
-					return u.hash.replace( /^#/, "" );
  159
+				    // For embedded pages, remove the dialog hash key as in getFilePath(),
  160
+				    // otherwise the Data Url won't match the id of the embedded Page.
  161
+					return u.hash.split( dialogHashKey )[0].replace( /^#/, "" );
160 162
 				} else if ( path.isSameDomain( u, documentBase ) ) {
161 163
 					return u.hrefNoHash.replace( documentBase.domain, "" );
162 164
 				}

0 notes on commit 83973b4

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