Skip to main content

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 davidserduke comment:1

component: coreui

Changed December 05, 2007 09:35AM UTC by rdworth comment:2

owner: → rworth
status: newassigned

Changed December 08, 2007 11:39AM UTC by rdworth comment:3

resolution: → fixed
status: assignedclosed

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 paul comment:5

milestone: → 1.5