New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Text Input doesn't use ui-disabled class if disabled attribute is set in markup #4636

Closed
patrick-wolf opened this Issue Jul 3, 2012 · 2 comments

Comments

Projects
None yet
2 participants
@patrick-wolf

For a form element of type "Text Input" the ui-disabled class isn't set during initialization of the widget if the "disabled" attribute is already set in the markup.

See http://jquerymobile.com/test/docs/forms/docs-forms.html (Disabling form elements)

This is inconsistent with the behavior of the textinput API, because

$( "#foo" ).textinput( "disable" );

will disable the text field AND set the ui-disabled class.

@jaspermdegroot

This comment has been minimized.

Show comment
Hide comment
@jaspermdegroot

jaspermdegroot Jul 3, 2012

Member

@patrickwolfat

Thanks for reporting the issues with the disabled textinput. In my comment at #4637 I explained the cause.

Member

jaspermdegroot commented Jul 3, 2012

@patrickwolfat

Thanks for reporting the issues with the disabled textinput. In my comment at #4637 I explained the cause.

@patrick-wolf

This comment has been minimized.

Show comment
Hide comment
@patrick-wolf

patrick-wolf Jul 3, 2012

Thanks for providing a fix that fast!

Thanks for providing a fix that fast!

jaspermdegroot added a commit that referenced this issue Jul 12, 2012

Textinput: added conditional call to disable() to the create function…
… to make sure disabled elements get class ui-disabled. Fixes #4636 and #4637 - Text Input doesn't use ui-disabled class if disabled attribute is set in markup. [1.1.2.]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment