Bug #6911 - Prevent action on disabled elements, both triggering and bound via .live()

This commit is contained in:
Dan Heberden 2011-02-18 10:09:07 -08:00
parent a43a5ca9cf
commit 47abe5e1da
2 changed files with 95 additions and 62 deletions

View File

@ -1099,7 +1099,7 @@ function liveHandler( event ) {
for ( j = 0; j < live.length; j++ ) { for ( j = 0; j < live.length; j++ ) {
handleObj = live[j]; handleObj = live[j];
if ( close.selector === handleObj.selector && (!namespace || namespace.test( handleObj.namespace )) ) { if ( close.selector === handleObj.selector && (!namespace || namespace.test( handleObj.namespace )) && !close.elem.disabled ) {
elem = close.elem; elem = close.elem;
related = null; related = null;

View File

@ -38,17 +38,21 @@
<input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox1"/> <input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox1"/>
<label for="checkbox1">Checkbox 1</label><br/> <label for="checkbox1">Checkbox 1</label><br/>
<input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox2"/> <input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox2"/>
<label for="checkbox2">Checkbox 2</label> <label for="checkbox2">Checkbox 2</label><br />
<input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox3" disabled="disabled"/> <input type="checkbox" class="checkbox_test" name="mycheckbox" id="checkbox3" disabled="disabled"/>
<label for="checkbox3">Checkbox 3</label> <label for="checkbox3">Checkbox 3</label>
</td> </td>
</td> <td>
<button class="button_test" name="mybutton1" id="button1">Button</button><br />
<button class="button_test" name="mybutton1" id="button1"><span>Button w/ child</span></button><br />
<button class="button_test" name="mybutton1" id="button1" disabled="disabled">Button Disabled</button><br />
<button class="button_test" name="mybutton1" id="button1" disabled="disabled"><span disabled="disabled">Button w/ child Dis.</span></button><br />
</td> </td>
<td> <td>
<input type="radio" class="radio_test" name="myradio" id="radio1"/> <input type="radio" class="radio_test" name="myradio" id="radio1"/>
<label for="radio1">Radio1</label><br/> <label for="radio1">Radio1</label><br/>
<input type="radio" class="radio_test" name="myradio" id="radio2"/> <input type="radio" class="radio_test" name="myradio" id="radio2"/>
<label for="radio2">Radio2</label> <label for="radio2">Radio2</label><br />
<input type="radio" class="radio_test" name="myradio" id="radio3" disabled="disabled"/> <input type="radio" class="radio_test" name="myradio" id="radio3" disabled="disabled"/>
<label for="radio3">Radio3</label> <label for="radio3">Radio3</label>
</td> </td>
@ -61,13 +65,17 @@
<td> <td>
<textarea rows='2'></textarea> <textarea rows='2'></textarea>
</td> </td>
<td>$(document).bind('change')</td> </td>
<td>$(document).bind('change')<br />
$(document).bind('click')
</td>
</tr> </tr>
<tr> <tr>
<td>Live:</td> <td>Live:</td>
<td id='select' class="red">SELECT</td> <td id='select' class="red">SELECT</td>
<td id='mselect' class="red">MULTI</td> <td id='mselect' class="red">MULTI</td>
<td id='checkbox' class="red">CHECKBOX</td> <td id='checkbox' class="red">CHECKBOX</td>
<td id='button' class="red">BUTTON</td>
<td id='radio' class="red">RADIO</td> <td id='radio' class="red">RADIO</td>
<td id='file' class="red">FILE</td> <td id='file' class="red">FILE</td>
<td id='text' class="red">TEXT</td> <td id='text' class="red">TEXT</td>
@ -79,6 +87,7 @@
<td id='selectbind' class="red">SELECT</td> <td id='selectbind' class="red">SELECT</td>
<td id='mselectbind' class="red">MULTI</td> <td id='mselectbind' class="red">MULTI</td>
<td id='checkboxbind' class="red">CHECKBOX</td> <td id='checkboxbind' class="red">CHECKBOX</td>
<td id='buttonbind' class="red">BUTTON</td>
<td id='radiobind' class="red">RADIO</td> <td id='radiobind' class="red">RADIO</td>
<td id='filebind' class="red">FILE</td> <td id='filebind' class="red">FILE</td>
<td id='textbind' class="red">TEXT</td> <td id='textbind' class="red">TEXT</td>
@ -89,6 +98,7 @@
<td id='selectfocus' class="red">SELECT</td> <td id='selectfocus' class="red">SELECT</td>
<td id='mselectfocus' class="red">MULTI</td> <td id='mselectfocus' class="red">MULTI</td>
<td id='checkboxfocus' class="red">CHECKBOX</td> <td id='checkboxfocus' class="red">CHECKBOX</td>
<td id='buttonfocus' class="red">BUTTON</td>
<td id='radiofocus' class="red">RADIO</td> <td id='radiofocus' class="red">RADIO</td>
<td id='filefocus' class="red">FILE</td> <td id='filefocus' class="red">FILE</td>
<td id='textfocus' class="red">TEXT</td> <td id='textfocus' class="red">TEXT</td>
@ -100,6 +110,7 @@
<td id='selectblur' class="red">SELECT</td> <td id='selectblur' class="red">SELECT</td>
<td id='mselectblur' class="red">MULTI</td> <td id='mselectblur' class="red">MULTI</td>
<td id='checkboxblur' class="red">CHECKBOX</td> <td id='checkboxblur' class="red">CHECKBOX</td>
<td id='buttonblur' class="red">BUTTON</td>
<td id='radioblur' class="red">RADIO</td> <td id='radioblur' class="red">RADIO</td>
<td id='fileblur' class="red">FILE</td> <td id='fileblur' class="red">FILE</td>
<td id='textblur' class="red">TEXT</td> <td id='textblur' class="red">TEXT</td>
@ -111,6 +122,7 @@
<td id='selectlfocus' class="red">SELECT</td> <td id='selectlfocus' class="red">SELECT</td>
<td id='mselectlfocus' class="red">MULTI</td> <td id='mselectlfocus' class="red">MULTI</td>
<td id='checkboxlfocus' class="red">CHECKBOX</td> <td id='checkboxlfocus' class="red">CHECKBOX</td>
<td id='buttonlfocus' class="red">BUTTON</td>
<td id='radiolfocus' class="red">RADIO</td> <td id='radiolfocus' class="red">RADIO</td>
<td id='filelfocus' class="red">FILE</td> <td id='filelfocus' class="red">FILE</td>
<td id='textlfocus' class="red">TEXT</td> <td id='textlfocus' class="red">TEXT</td>
@ -121,11 +133,24 @@
<td id='selectlblur' class="red">SELECT</td> <td id='selectlblur' class="red">SELECT</td>
<td id='mselectlblur' class="red">MULTI</td> <td id='mselectlblur' class="red">MULTI</td>
<td id='checkboxlblur' class="red">CHECKBOX</td> <td id='checkboxlblur' class="red">CHECKBOX</td>
<td id='buttonlblur' class="red">BUTTON</td>
<td id='radiolblur' class="red">RADIO</td> <td id='radiolblur' class="red">RADIO</td>
<td id='filelblur' class="red">FILE</td> <td id='filelblur' class="red">FILE</td>
<td id='textlblur' class="red">TEXT</td> <td id='textlblur' class="red">TEXT</td>
<td id='textarealblur' class="red">TEXTAREA</td> <td id='textarealblur' class="red">TEXTAREA</td>
</tr> </tr>
<tr>
<td>Live Click:</td>
<td id='selectlclick' class="red">SELECT</td>
<td id='mselectlclick' class="red">MULTI</td>
<td id='checkboxlclick' class="red">CHECKBOX</td>
<td id='buttonlclick' class="red">BUTTON</td>
<td id='radiolclick' class="red">RADIO</td>
<td id='filelclick' class="red">FILE</td>
<td id='textlclick' class="red">TEXT</td>
<td id='textarealclick' class="red">TEXTAREA</td>
<td id='boundClick' class="red">DOCUMENT</td>
</tr>
</table> </table>
<h2>Submit Tests</h2> <h2>Submit Tests</h2>
<table> <table>
@ -162,7 +187,7 @@
<ul id="log"></ul> <ul id="log"></ul>
<script type='text/javascript'> <script type='text/javascript'>
jQuery.fn.addChangeTest = function( id, prevent ) { jQuery.fn.addChangeClickTest = function( id, prevent ) {
this.bind("focusin", function(){ this.bind("focusin", function(){
jQuery(id + "focus").blink(); jQuery(id + "focus").blink();
}).bind("focusout", function(){ }).bind("focusout", function(){
@ -183,9 +208,12 @@
} }
jQuery(id).blink(); jQuery(id).blink();
}).live("click", function(e){
jQuery(id + "lclick").blink();
}); });
}; };
jQuery.fn.addSubmitTest = function( id, prevent ) { jQuery.fn.addSubmitTest = function( id, prevent ) {
return this.live("submit", function(e){ return this.live("submit", function(e){
if ( prevent ) { if ( prevent ) {
@ -211,6 +239,10 @@
jQuery("#boundBlur").blink(); jQuery("#boundBlur").blink();
}); });
$(document).bind("click", function() {
jQuery("#boundClick").blink();
});
$("td.red").live("hover", function(e) { $("td.red").live("hover", function(e) {
if ( e.type === "mouseenter" ) { if ( e.type === "mouseenter" ) {
$(this).css("backgroundColor","green"); $(this).css("backgroundColor","green");
@ -219,13 +251,14 @@
} }
}); });
$(".select_test").addChangeTest("#select"); $(".select_test").addChangeClickTest("#select");
$(".mselect_test").addChangeTest("#mselect"); $(".mselect_test").addChangeClickTest("#mselect");
$(".checkbox_test").addChangeTest("#checkbox"); $(".checkbox_test").addChangeClickTest("#checkbox");
$(".radio_test").addChangeTest("#radio"); $(".radio_test").addChangeClickTest("#radio");
$(".file_test").addChangeTest("#file"); $(".file_test").addChangeClickTest("#file");
$('textarea').addChangeTest("#textarea"); $('textarea').addChangeClickTest("#textarea");
$('#input').addChangeTest("#text"); $('#input').addChangeClickTest("#text");
$('button').addChangeClickTest("#button");
$(document).bind("change", function(){ $(document).bind("change", function(){
jQuery("#boundChange").blink(); jQuery("#boundChange").blink();
}); });