jQuery: The Write Less, Do More JavaScript Library

Ticket #2028 (closed enhancement: fixed)

Opened 8 months ago

Last modified 8 months ago

X/Y parameters to dialogs, example included

Reported by: jontsa Assigned to: anonymous
Type: enhancement Priority: minor
Milestone: 1.2.2 Component: ui
Version: 1.2.1 Keywords:
Cc: Needs: Review

Description

Please add X/Y parameters to dialog options. The current "position" parameter is too limited so I use the following patch:

Index: /resources/trunk/js/jquery/ui/dialog.js
===================================================================
--- /resources/trunk/js/jquery/ui/dialog.js (revision 8114)
+++ /resources/trunk/js/jquery/ui/dialog.js (revision 8168)
@@ -36,5 +36,7 @@
 			drag: true,
 			resize: true,
-			shadow: false // It's quite slow
+			shadow: false, // It's quite slow
+			x: false,
+			y: false
 		};
 		var o = o || {}; $.extend(options, o); //Extend and copy options
@@ -108,5 +110,5 @@
 					break;
 			}
-			uiDialog.css({top: top, left: left});
+			uiDialog.css({top: options.y?options.y:top, left: options.x?options.x:left});
 			uiDialog.appendTo('body').show();

Attachments

Change History

Changed 8 months ago by rworth

  • status changed from new to closed
  • resolution set to fixed

Fixed in [4076]. position option now accepts an array ([x, y]) in addition to strings.

Note: See TracTickets for help on using tickets.