Closed Bug 5292 Opened 26 years ago Closed 26 years ago

Dialogs must behave as transient windows

Categories

(Core :: XUL, defect, P3)

x86
Linux
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: slogan, Assigned: danm.moz)

Details

In X11, dialog windows should be marked transient so that they will iconify if the toplevel window is iconified by the user. In Gtk+, this is accomplished in one of two ways. First, a window can be created as a dialog window by calling gtk_window_new( GTK_WINDOW_DIALOG ). The other way is to create a window of any type and then call gtk_window_set_transient_for( GtkWindow *...) and mark it transient, doing this before the window is realized. As it turns out, there is a bug in the implementation of Gtk+ dialog windows -- it assigns the root window as the dialog's transient parent, when it really should be using the toplevel (main) window as the transient parent (actually, there is no way for Gtk+ to tell since gtk_window_new() does not accept a parent window argument). The fix is to make a call to gtk_window_transient_for() if a window is a dialog window.
Moving all Widget Set bugs, past and present, to new HTML Form Controls component per request from karnaze. Widget Set component will be retired shortly.
moving to XP Toolkit/Widgets since these are not HTML form controls
Assignee: syd → pavlov
Assignee: pavlov → pavlov
Status: NEW → ASSIGNED
dialogs are currently passing NULL in as their parents. Once this is fixed, this should be resolved as I have added code to make windows transient in the code.
Assignee: pavlov → danm
Status: ASSIGNED → NEW
when dialogs are created they need a parent. danm is working on this so i am reassinging this to him
Target Milestone: M10
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
You'll get a transient/popup window if you ask for it in the call to openDialog by specifying the "dependent" feature. Transient, popup, dependent ... you know what we mean.
Sorry for spam, re-assigning phillip's QA contact XPToolkit/XPWidget bugs to claudius due to restructure
Status: RESOLVED → VERIFIED
marking VERIFIED with 1999100615 build although it was fixed much earlier
You need to log in before you can comment on or make changes to this bug.