beedesk / jQTouch

jQT + actively maintained + exclusive features (iPad layout, iscroll, tons of css enhancements, page events, param passing between pages, radio button, grid, mail, improved page history management etc.)
http://bit.ly/beedesk-jqt
MIT License
72 stars 13 forks source link

External links opening two copies, two windows #12

Closed only1chip closed 13 years ago

only1chip commented 13 years ago

Click the 'external' link to 'homepage' on the demo:

http://dev.beedesk.com/dropbox/jqt/demos/main/#ui

From my iPad it asks the question about opening in a new window twice, and then (assuming you click "OK") it opens the requested url in two separate windows. Sometimes it even opens a third copy when you return focus to the original div/pane in the demo app.

only1chip commented 13 years ago

Should have mentioned that you have to have an ipad or iphone to see the double action - doesn't seem to happen in safari.

Also, I love jqtouch and your fork - please keep up the great work!

Chip

thomasyip commented 13 years ago

I can reprod it when it open as a webpage.

But, it runs fine in full-screen mode. Can you confirm it is what you are seeing too?

only1chip commented 13 years ago

In Safari (v5.0.2) running under Windows, I only get a single new window with the external link, however the right hand panel (in the jQtouch demo) also changes. On the iPad the right hand panel doesn't change - but you get two warning messages, and two new windows. Some times a third when you return focus to the jQtouch app.

Perhaps the HTML is opening one window, and an AJAX call is opening the second? Just a guess...

thomasyip commented 13 years ago

I remembered an email to the jqtouch mailing list about a click pass-thru from the dialog. Can't find the email at the moment.

I will investigate and let you know.

barts2108 commented 13 years ago

Looks a bit like an issue that I had before

https://github.com/senchalabs/jQTouch/issues/151

thomasyip commented 13 years ago

Should have fixed now. Sorry for taking so long.

https://github.com/beedesk/jQTouch/commit/37b6d9beb1538968587449ab1e3b48572018ca92