Spinner: Adjust height check for IE 6 rendering.

scottgonzalez · scottgonzalez · commit 4694f85d4c1f · 2011-10-13T16:39:14.000-04:00
diff --git a/ui/jquery.ui.spinner.js b/ui/jquery.ui.spinner.js
@@ -151,7 +151,8 @@ $.widget( "ui.spinner", {
 
 		// IE 6 doesn't understand height: 50% for the buttons
 		// unless the wrapper has an explicit height
-		if ( this.buttons.height() === uiSpinner.height() && uiSpinner.height() > 0 ) {
+		if ( this.buttons.height() > Math.ceil( uiSpinner.height() * 0.5 ) &&
+				uiSpinner.height() > 0 ) {
 			uiSpinner.height( uiSpinner.height() );
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -151,7 +151,8 @@ $.widget( "ui.spinner", {`
`151`	`151`
`152`	`152`	`// IE 6 doesn't understand height: 50% for the buttons`
`153`	`153`	`// unless the wrapper has an explicit height`
`154`		`- if ( this.buttons.height() === uiSpinner.height() && uiSpinner.height() > 0 ) {`
	`154`	`+ if ( this.buttons.height() > Math.ceil( uiSpinner.height() * 0.5 ) &&`
	`155`	`+ uiSpinner.height() > 0 ) {`
`155`	`156`	`uiSpinner.height( uiSpinner.height() );`
`156`	`157`	`}`
`157`	`158`