Hopiu/jquery-mobile
1
0
Fork 0
mirror of https://github.com/Hopiu/jquery-mobile.git synced 2026-05-26 14:53:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

remove checked state from radio button inputs that are not checked. use jquery attr accessor instead of expando to guarantee consistent values. Fixes #1532.

Browse source
This commit is contained in:
John Bender 2011-05-02 00:53:51 -07:00
parent d9e47627ed
commit 377c0067a7
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
js/jquery.mobile.forms.checkboxradio.js
View file

@ -58,7 +58,14 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
}
self._cacheVals();
input.attr( "checked", inputtype === "radio" && true || !input.is( ":checked" ) );
// input set for common radio buttons will contain all the radio
// buttons, but will not for checkboxes. clearing the checked status
// of other radios ensures the active button state is applied properly
self._getInputSet().not(input).removeAttr('checked');
self._updateAll();
return false;
}
@ -116,7 +123,9 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
label = this.label,
icon = label.find( ".ui-icon" );
if ( input[0].checked ) {
// input[0].checked expando doesn't always report the proper value
// for checked='checked'
if ( $(input[0]).attr('checked') ) {
label.addClass( $.mobile.activeBtnClass );
icon.addClass( this.checkedicon ).removeClass( this.uncheckedicon );