library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Button: Don't react to keydown when disabled.

Browse Source 
Partial fix for #5328 - button disabled only visually when disabled option set, still functional.
This commit is contained in:
Scott González 2010-03-14 17:53:18 +00:00
parent c75abc5e9a
commit 385269e3ea
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
ui/jquery.ui.button.js vendored
View File

@ -101,7 +101,7 @@ $.widget( "ui.button", {
});
}
if ( this.type === "checkbox") {
if ( this.type === "checkbox" ) {
this.buttonElement.bind( "click.button", function() {
if ( options.disabled ) {
return;
@ -109,7 +109,7 @@ $.widget( "ui.button", {
$( this ).toggleClass( "ui-state-active" );
self.buttonElement.attr( "aria-pressed", self.element[0].checked );
});
} else if ( this.type === "radio") {
} else if ( this.type === "radio" ) {
this.buttonElement.bind( "click.button", function() {
if ( options.disabled ) {
return;
@ -145,7 +145,10 @@ $.widget( "ui.button", {
$( this ).removeClass( "ui-state-active" );
})
.bind( "keydown.button", function(event) {
if ( event.keyCode === $.ui.keyCode.SPACE || event.keyCode === $.ui.keyCode.ENTER ) {
if ( options.disabled ) {
return;
}
if ( event.keyCode == $.ui.keyCode.SPACE || event.keyCode == $.ui.keyCode.ENTER ) {
$( this ).addClass( "ui-state-active" );
}
})
@ -166,8 +169,7 @@ $.widget( "ui.button", {
// TODO: pull out $.Widget's handling for the disabled option into
// $.Widget.prototype._setOptionDisabled so it's easy to proxy and can
// be overridden by individual plugins
$.Widget.prototype._setOption.call( this, "disabled", options.disabled );
this._resetButton();
this._setOption( "disabled", options.disabled );
},
_determineButtonType: function() {
@ -326,4 +328,4 @@ $.widget( "ui.buttonset", {
}
});
}( jQuery ));
}( jQuery ) );