Bug Tracker

Modify ↓

Ticket #1990 (closed bug: duplicate)

Opened 5 years ago

Last modified 5 years ago

Can not have multiple dialogs in latest version of UI

Reported by: rbrackett Owned by: rworth
Priority: major Milestone: 1.2.2
Component: ui Version: 1.2.1
Keywords: dialog Cc:
Blocking: Blocked by:

Description

In fixing Ticket #1684, the ability to have multiple dialogs was lost. They 'new' keyword is no longer being used to create dialog objects and thus all dialogs are sharing the same data. As an example, if I create two dialogs, dialog1 and dialog2, and then call dialog1.dialogOpen();, dialog2 gets opened. This is an easy fix: just add the 'new' keyword to line 12 of ui.dialog.js.

I've included a patch for convenience.

Attachments

multipleDialogsPatch.diff Download (391 bytes) - added by rbrackett 5 years ago.
Fix to let multiple dialogs work again in UI

Change History

Changed 5 years ago by rbrackett

Fix to let multiple dialogs work again in UI

comment:1 Changed 5 years ago by paul

  • Owner set to rworth

comment:2 Changed 5 years ago by Xichekolas

Just want to comment that I made this change and it definitely fixed the multiple dialog closing issue I was having. Thanks rbrackett!

comment:3 Changed 5 years ago by rworth

  • Status changed from new to closed
  • Resolution set to duplicate

Fixed by #2008 in rev4075

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.