All: Remove workaround for .offset() setter bug from older versions of jQuery.

scottgonzalez · scottgonzalez · commit ca0ac5a64f66 · 2010-12-11T15:19:25.000-05:00
diff --git a/ui/jquery.ui.autocomplete.js b/ui/jquery.ui.autocomplete.js
@@ -201,8 +201,6 @@ $.widget( "ui.autocomplete", {
 				}
 			})
 			.zIndex( this.element.zIndex() + 1 )
-			// workaround for jQuery bug #5781 http://dev.jquery.com/ticket/5781
-			.css({ top: 0, left: 0 })
 			.hide()
 			.data( "menu" );
 		if ( $.fn.bgiframe ) {
diff --git a/ui/jquery.ui.dialog.js b/ui/jquery.ui.dialog.js
@@ -518,10 +518,7 @@ $.widget("ui.dialog", {
 		if (!isVisible) {
 			this.uiDialog.show();
 		}
-		this.uiDialog
-			// workaround for jQuery bug #5781 http://dev.jquery.com/ticket/5781
-			.css({ top: 0, left: 0 })
-			.position(position);
+		this.uiDialog.position(position);
 		if (!isVisible) {
 			this.uiDialog.hide();
 		}

Original file line number	Diff line number	Diff line change
`@@ -201,8 +201,6 @@ $.widget( "ui.autocomplete", {`
`201`	`201`	`}`
`202`	`202`	`})`
`203`	`203`	`.zIndex( this.element.zIndex() + 1 )`
`204`		`- // workaround for jQuery bug #5781 http://dev.jquery.com/ticket/5781`
`205`		`- .css({ top: 0, left: 0 })`
`206`	`204`	`.hide()`
`207`	`205`	`.data( "menu" );`
`208`	`206`	`if ( $.fn.bgiframe ) {`
Original file line number	Diff line number	Diff line change
`@@ -518,10 +518,7 @@ $.widget("ui.dialog", {`
`518`	`518`	`if (!isVisible) {`
`519`	`519`	`this.uiDialog.show();`
`520`	`520`	`}`
`521`		`- this.uiDialog`
`522`		`- // workaround for jQuery bug #5781 http://dev.jquery.com/ticket/5781`
`523`		`- .css({ top: 0, left: 0 })`
`524`		`- .position(position);`
	`521`	`+ this.uiDialog.position(position);`
`525`	`522`	`if (!isVisible) {`
`526`	`523`	`this.uiDialog.hide();`
`527`	`524`	`}`