Make changePage working properly on PhoneGap with WP7 #5235

Closed
wants to merge 1 commit into
from

4 participants

@gziolo

WP7 with PhoneGap uses different local file name which has form like:
x-wmapp1:/app/www/index.html
Currently navigation helper assumes that every URL has double slash in url which is not the case in this situation. I added fix which always checks if there is double slash necessary for given url. That makes this working for all tradtional urls, as well as for WP7.

@DoobleD

Hi,

I experiment a similar problem using $.mobile.changePage with WP7. However I'm a bit new to github and the pull request system.

How can I use your fix in my code? Can I download your version of js/jquery.mobile.navigation.js and include it in my js files?

@jaspermdegroot
jQuery Foundation member

@DoobleD

I think easiest way to test @gziolo his solution is making the changes manually. If you click at the "Files Changed" tab at the top of this pull request you see the changes he suggest.
Make sure you use latest code for testing: http://jquerymobile.com/test/js/jquery.mobile.js

@DoobleD

Yes this was my first try but I wasn't able to find the code to replace in my JQM js file, which might be because I was using the minified one.

Anyway I tried again using http://jquerymobile.com/test/js/jquery.mobile.js as you suggested and it worked! However I had to change something in the code from @gziolo. Replace var url by var uri in:

"
var url = this.parseUrl( url || location.href ),
hash = uri.hash;
"

Now $.mobile.changePage works correctly on WP7. Thanks @gziolo and @uGoMobi for the help.

I expriment other troubles on WP7 now: my JQM pop up windows are all showed, and at the end of the page. Any chance it could come from the modified getLocation function?

@DoobleD

I confirm this must be related to the getLocation changes since I tested the code on Android and the same bug occurs (which is not the case with th original code).

@arschmitz
jQuery Foundation member

closing this because this function no longer exists

@arschmitz arschmitz closed this Aug 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment