Search and Top Navigation
#2008 closed bug (fixed)
Opened December 05, 2007 06:19AM UTC
Closed December 08, 2007 11:39AM UTC
Last modified February 26, 2009 11:19AM UTC
[PATCH] Open and Close Multiple Dialogs
Reported by: | Xichekolas | Owned by: | rdworth |
---|---|---|---|
Priority: | major | Milestone: | 1.5 |
Component: | ui.core | Version: | 1.2.1 |
Keywords: | ui dialog open close | Cc: | |
Blocked by: | Blocking: |
Description
Two problems:
1. All dialogs were opening with the same data. This made all the close buttons only close the most recently opened dialog. Fixed this by adding the 'new' keyword before the call to $.ui.dialogInit
2. Closing a dialog only sets it to display:none. However, reopening the dialog actually creates a whole duplicate dialog, rather than unhiding the existing one. This is because the if statement that checked to see if the div on which we call .dialog() is already a dialog was checking for the wrong css class (due to the way the dialog is built, the original div gets the .ui-dialog-content class, not the .ui-dialog class.
Attachments (0)
Change History (5)
Changed December 05, 2007 07:33AM UTC by comment:1
component: | core → ui |
---|
Changed December 05, 2007 09:35AM UTC by comment:2
owner: | → rworth |
---|---|
status: | new → assigned |
Changed December 08, 2007 11:39AM UTC by comment:3
resolution: | → fixed |
---|---|
status: | assigned → closed |
Thanks for the patch. Fixed in rev4075.
Changed May 24, 2008 03:39AM UTC by comment:4
milestone: | 1.2.2 |
---|
Milestone 1.2.2 deleted
Changed February 26, 2009 11:19AM UTC by comment:5
milestone: | → 1.5 |
---|