$(select).selectmenu('disable') causes issues on iOS for native selects #1816

Closed
gerges opened this Issue Jun 10, 2011 · 6 comments

Projects

None yet

3 participants

@gerges
gerges commented Jun 10, 2011

On iOS, I've found that native select menus disabled with .selectmenu('disable') are still opened, and can cause some odd behavior on the device. Once opened, the native menu cannot be closed unless the browser is killed or the user refreshes/leaves the page.

The issue, oddly enough, appears to be caused by setting the following style

.ui-disabled,
.ui-disabled a {
    cursor: default !important;
}

I was able to verify this using the latest jQuery mobile, on both iPad, iPhone and the emulators for both. Please let me know if you have any issues replicating the issue, or would like me to provide an example. As the majority of the devices supported don't show a cursor, I'd assume the fix would just be to remove, or modify this style so it does not apply to select menus. I would also assume there's an underlying iOS issue here.

Please let me know if you have any questions.

@toddparker
Contributor

Interesting. Could you post up a JSBin demo page with this demo? We'll take a look.

@gerges
gerges commented Jun 10, 2011

Of course. http://jsbin.com/atola5/.

Also, here are screenshots of the emulators
http://imgur.com/a/IvhOl

I think this may be iOS version dependent, definitely an issue in iOS 4...but possibly not in 3. Needs more research

@gerges gerges closed this Jun 10, 2011
@gerges gerges reopened this Jun 10, 2011
@gerges
gerges commented Jun 10, 2011

Woops...They really need to move that comment and close button

@gseguin gseguin pushed a commit to gseguin/jquery-mobile that referenced this issue Jun 16, 2011
Ghislain Seguin Fix for issue #1816: $(select).selectmenu('disable') causes issues on…
… iOS for native selects
c3e5015
@jblas
Contributor
jblas commented Jun 16, 2011

Landed @gseguin's fix:

6e7e4d0

Thank you @gseguin!

@jblas jblas closed this Jun 16, 2011
@gerges
gerges commented Jun 17, 2011

Just as a follow-up, I no longer have this issue on latest. Thanks guys!

@jblas
Contributor
jblas commented Jun 17, 2011

@gerges

Thanks for the follow-up confirmation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment