diff options
| author | Dave Methvin <dave.methvin@gmail.com> | 2011-09-11 21:18:32 -0400 |
|---|---|---|
| committer | timmywil <timmywillisn@gmail.com> | 2011-09-19 15:42:32 -0400 |
| commit | 6ece8f138f0c89d72fd597d95aa87304cdbe4d1d (patch) | |
| tree | f032c269e01ef1d04bf3537b6fd3970f08b3ea49 /test/delegatetest.html | |
| parent | 6c618ddb0d1efd980316c7d268c5531419e67ae0 (diff) | |
| download | jquery-6ece8f138f0c89d72fd597d95aa87304cdbe4d1d.tar.gz jquery-6ece8f138f0c89d72fd597d95aa87304cdbe4d1d.zip | |
Revert "Fix #6593. Don't trigger `change` event twice when simulating it on IE." Add inline handler monitor to delegatetest.html so we can see it fail.
This reverts commit e77686495b6e34938575c9c0fe978cb4a0be6f05.
Diffstat (limited to 'test/delegatetest.html')
| -rw-r--r-- | test/delegatetest.html | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/test/delegatetest.html b/test/delegatetest.html index 6938f7e1f..ef8ea6158 100644 --- a/test/delegatetest.html +++ b/test/delegatetest.html @@ -96,7 +96,7 @@ th, td { <tbody> </tbody> </table> -<p>NOTE: Only IE supports beforeactivate/beforedeactivate; buttons do not support change events.</p> +<p>NOTE: Only IE supports propertychange, beforeactivate, beforedeactivate; buttons do not support change events.</p> <h2>Submit Tests</h2> <table> @@ -136,21 +136,21 @@ $("#version").text(version); $("#fileversion").text($.fn.jquery); // Events we want to track in row-order -var events = "bind-change live-change live-beforeactivate live-focusin bind-focus live-keydown live-beforedeactivate live-focusout bind-blur live-click".split(" "), +var events = "bind-change live-change on-change bind-propertychange live-beforeactivate live-focusin bind-focus live-keydown live-beforedeactivate live-focusout bind-blur live-click".split(" "), counter = 0; blinker = function(event){ if ( !counter ) { $("#changes tbody td").text(""); } - var $els = event.data, - prev = $els.text(); + var $el = event.data, + prev = $el.text(); prev = prev? prev +" | " : ""; - return $els + return $el .text(prev + ++counter+" " + (this.value.replace(/^on$/,"") || this.id || this.checked || "")) .css("backgroundColor","#0f0") - .delay(1000) + .delay(800) .queue(function(next){ - $els.css("backgroundColor","#afa"); + $el.css("backgroundColor","#afa"); --counter; next(); }); @@ -165,7 +165,11 @@ for ( var i=0; i < events.length; i++ ) { $("#changes thead td").each(function(){ var id = "#"+this.id, $cell = $('<td></td>'); - if ( api == "bind" ) { + if ( api == "on" ) { + $(this).find("input, button, select, textarea").each(function(){ + this["on"+type] = function(e){ e = $.event.fix(e||event); e.data = $cell; blinker.call(this, e); }; + }); + } else if ( api == "bind" ) { $(this).find("input, button, select, textarea").bind(type, $cell, blinker); } else { $(id+" input,"+id+" button,"+id+" select,"+id+" textarea").live(type, $cell, blinker); @@ -174,8 +178,6 @@ for ( var i=0; i < events.length; i++ ) { }); $("#changes tbody").append($row); } -// Buttons don't fire change events so show the cell as gray -$("#changes tbody").find("tr:eq(0), tr:eq(1)").find("td:eq(3)").css("background", "#eee"); jQuery.fn.blink = function(){ return this.css("backgroundColor","green").css("border","solid 3px green").delay(700).queue(function(next){ |