Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

[JQM 1.2.0 beta 1] Issue with custom select menus in Safari (iOS 5.1.1, iPad 2 and iPad 3) #4949

Closed
Mitch64 opened this issue Sep 6, 2012 · 6 comments

Comments

@Mitch64
Copy link

Mitch64 commented Sep 6, 2012

I did a little test driving with the brand new JQM 1.2.0 beta 1 and noticed, that in the demos & docs the demos for the the custom select menus did behave a little strange on my iPads, both running with iOS 5.1.1. You can see it for yourself at

http://jquerymobile.com/demos/1.2.0-beta.1/docs/forms/selects/custom.html

When clicking on the first custom select menu (the small list for the shipping method) at first the menu pops up and then after approx. 50 ms jumps to the center of the screen (i.e. is being centered)

I did a little research then and found out that in 1.2.0 alpha 1 the behaviour was correct (no jumping).
You can also test this for yourself at

http://jquerymobile.com/demos/1.2.0-alpha.1/docs/forms/selects/custom.html

I also managed to write a little test program and found out that it's not a problem with the demos but with the custom select menus. Because I had some older (latest) versions of JQM I could make sure that the problematic code came in to the codebase probably after 08/28/2012. After looking a little bit into the JS I suppose that after the opening of the menu JQM erroneously thinks it has to resize and that causes the centering.

This issue only occurs on my iPads running iOS 5.1.1. It does not occur on my 7'' Galaxy Tab running Android 2.3.6, my Samsung Galaxy S III running Android 4.0.4 and not on any of my desktop browsers (FF 15, Chrome 21.0.1180.89, Safari 5.1.7)

@toddparker
Copy link
Contributor

We spotted this yesterday with normal popups too, but only if you navigated in from the docs homepage on an iPad and didn't scroll the page (bizarre). Looking into it now.

@ghost ghost assigned gabrielschulhof Sep 6, 2012
@jesperveldhuizen
Copy link

I can confirm that this problem also occurs on my iPad 1 running on 5.1.1, but not on my Galaxy Tab 2 10.1 (running on android 4.0.4).

@CKiOne
Copy link

CKiOne commented Sep 7, 2012

I think, its a Problem with "Page Resize Event". Sometimes Popups trigger this event and then the popup will be centered in the middle of the screen.

@gabrielschulhof
Copy link

I don't understand why the popup triggers a window resize event ...

@gabrielschulhof
Copy link

I have identified the commit that breaks things: 7a14e15. The problem is, this is a fairly extensive commit, pretty much re-writing the popup's open/close logic.

@johnbender
Copy link
Contributor

We have to take another look at this, I wrote some poor tests for the "historyless" popups and this change breaks them (obviously without failing the tests). The tests are fixed though.

@johnbender johnbender reopened this Sep 7, 2012
arschmitz pushed a commit to arschmitz/jquery-mobile that referenced this issue Oct 16, 2012
arschmitz pushed a commit to arschmitz/jquery-mobile that referenced this issue Oct 16, 2012
…en acquired -- Fixes jquery-archive#4949"

This reverts commit 3b40d03.

This change doesn't call _open when history alteration is disabled breaking
popups in ie7 by default and anywhere that $.mobile.popup.prototype.history is used
arschmitz pushed a commit to arschmitz/jquery-mobile that referenced this issue Oct 16, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants