Button: add explanatory comment for when we bind to blur

mikesherov · mikesherov · commit 509259a66e28 · 2012-11-21T19:58:59.000-05:00
diff --git a/ui/jquery.ui.button.js b/ui/jquery.ui.button.js
@@ -193,6 +193,8 @@ $.widget( "ui.button", {
 						$( this ).addClass( "ui-state-active" );
 					}
 				})
+				// see #8559, we bind to blur here in case the button element loses
+				// focus between keydown and keyup, it would be left in an "active" state
 				.bind( "keyup" + this.eventNamespace + " blur" + this.eventNamespace, function() {
 					$( this ).removeClass( "ui-state-active" );
 				});

Original file line number	Diff line number	Diff line change
`@@ -193,6 +193,8 @@ $.widget( "ui.button", {`
`193`	`193`	`$( this ).addClass( "ui-state-active" );`
`194`	`194`	`}`
`195`	`195`	`})`
	`196`	`+ // see #8559, we bind to blur here in case the button element loses`
	`197`	`+ // focus between keydown and keyup, it would be left in an "active" state`
`196`	`198`	`.bind( "keyup" + this.eventNamespace + " blur" + this.eventNamespace, function() {`
`197`	`199`	`$( this ).removeClass( "ui-state-active" );`
`198`	`200`	`});`