#4439 closed bug (duplicate)

Opened April 06, 2009 03:34PM UTC

Closed April 08, 2009 12:52AM UTC

Last modified October 11, 2012 09:15PM UTC

Second modal dialog disables input fields in first modal dialog.

Reported by: nbubna Owned by:
Priority: minor Milestone:
Component: ui.dialog Version: 1.7
Keywords: Cc:
Blocked by: Blocking:
Description

Open one modal dialog with form to create a new Foo. Entering invalid data triggers second modal dialog with error message. Once the second one is closed, you can no longer edit the fields in the first modal dialog to fix the error. Only tested in firefox.

Seems i'm not the first to find this:

http://groups.google.com/group/jquery-ui/browse_thread/thread/96e731e6a49fae5/53f54c04279f5427?lnk=gst&q=modal+dialogs+edit+fields#53f54c04279f5427

Attachments (0)
Change History (9)

Changed April 06, 2009 04:30PM UTC by nbubna comment:1

Appears to me that the $.ui.dialog.overlay.maxZ and $.ui.dialog.maxZ don't get corrected once the second dialog is closed. I'm not sure how to fix that.

Changed April 06, 2009 04:47PM UTC by nbubna comment:2

As a workaround, i called $firstModalDialog.dialog('moveToTop', true) once the second one is closed. That's about all the time i have for this today.

Changed April 06, 2009 05:12PM UTC by nbubna comment:3

Ah, but just as i was about to move on, i noticed that moveToTop(true) is always called on open. So, on close, it is safe to always just revert the changes to the maxZ values when closing the topmost dialog. Just add this to the end of the close function:

if this was topmost, revert the last moveToTop call

http://dev.jqueryui.com/ticket/4439

if ($.ui.dialog.maxZ == self.uiDialog.css('z-index')) {

$.ui.dialog.maxZ = --$.ui.dialog.overlay.maxZ;

--$.ui.dialog.overlay.maxZ;

}

Changed April 06, 2009 05:13PM UTC by nbubna comment:4

hmm. formatting issues...

        // if this was topmost, revert the last moveToTop call
        //  http://dev.jqueryui.com/ticket/4439
        if ($.ui.dialog.maxZ == self.uiDialog.css('z-index')) {
            $.ui.dialog.maxZ = --$.ui.dialog.overlay.maxZ;
            --$.ui.dialog.overlay.maxZ;
        }

Changed April 06, 2009 06:32PM UTC by nbubna comment:5

Ok, i tested the bug and the fix in Internet Explorer also.

Changed April 08, 2009 12:52AM UTC by scottgonzalez comment:6

resolution: → duplicate
status: new → closed

Duplicate of #4309.

Changed May 06, 2011 06:53PM UTC by captdragon comment:7

This is still a problem, please re-open this ticket.

Changed May 09, 2011 12:53PM UTC by scottgonzalez comment:8

Please provide a reduced test case showing that the problem exists.

Changed October 11, 2012 09:15PM UTC by scottgonzalez comment:9

milestone: TBD

Milestone TBD deleted