Hopiu/jquery-mobile
1
0
Fork 0
mirror of https://github.com/Hopiu/jquery-mobile.git synced 2026-05-19 03:51:06 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Allow checkboxes/radio buttons to be selected with keyboard - updated to use prop()

Browse source
This commit is contained in:
Shana Golden 2011-05-27 09:50:54 -04:00
parent 4ca2fc5287
commit c9d97ef261
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -59,12 +59,12 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
self._cacheVals();
input.attr( "checked", inputtype === "radio" && true || !(input.attr("checked")) );
input.prop( "checked", inputtype === "radio" && true || !(input.prop("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._getInputSet().not(input).prop('checked', false);
self._updateAll();
return false;
@ -81,10 +81,10 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
vclick: function(){
// adds checked attribute to checked input when keyboard is used
if ($(this).is(":checked")) {
$(this).attr( "checked", true);
self._getInputSet().not($(this)).removeAttr('checked');
$(this).prop( "checked", true);
self._getInputSet().not($(this)).prop('checked', false);
} else {
$(this).removeAttr("checked");
$(this).prop("checked", false);
}
self._updateAll();
},
@ -132,7 +132,7 @@ $.widget( "mobile.checkboxradio", $.mobile.widget, {
// input[0].checked expando doesn't always report the proper value
// for checked='checked'
if ( $(input[0]).attr('checked') ) {
if ( $(input[0]).prop('checked') ) {
label.addClass( $.mobile.activeBtnClass );
icon.addClass( this.checkedicon ).removeClass( this.uncheckedicon );