After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 598685 - Set a parent on the remove confirmation dialog
Set a parent on the remove confirmation dialog
Status: RESOLVED FIXED
Product: gnome-bluetooth
Classification: Core
Component: moblin
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-bluetooth-general-maint@gnome.bugs
gnome-bluetooth-general-maint@gnome.bugs
Depends on:
Blocks:
 
 
Reported: 2009-10-16 14:28 UTC by Joshua Lock
Modified: 2009-10-16 15:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Set a parent on the remove confirmation dialog (1.81 KB, patch)
2009-10-16 14:28 UTC, Joshua Lock
none Details | Review
Set a parent on the remove confirmation dialog (1.82 KB, patch)
2009-10-16 14:50 UTC, Joshua Lock
committed Details | Review

Description Joshua Lock 2009-10-16 14:28:32 UTC
With a NULL parent on Moblin the dialog ends up stacked below other windows
and can't accept any user input.
Modify the internal method show_confirm_dialog to take a BluetoothChooser
as an argument and set the toplevel window of the chooser as a parent for
the dialog.

This makes things work *much* nicer in Moblin and doesn't seem to have any
adverse affect in Gnome.
Comment 1 Joshua Lock 2009-10-16 14:28:35 UTC
Created attachment 145606 [details] [review]
Set a parent on the remove confirmation dialog
Comment 2 Joshua Lock 2009-10-16 14:34:07 UTC
Comment on attachment 145606 [details] [review]
Set a parent on the remove confirmation dialog

This patch is wrong, let me attach the correct one.
Comment 3 Joshua Lock 2009-10-16 14:50:10 UTC
Created attachment 145614 [details] [review]
Set a parent on the remove confirmation dialog

With a NULL parent on Moblin the dialog ends up stacked below other windows
and can't accept any user input.
Modify the internal method show_confirm_dialog to take a BluetoothChooser
as an argument and set the toplevel window of the chooser as a parent for
the dialog.

This makes things work *much* nicer in Moblin and doesn't seem to have any
adverse affect in Gnome.
Comment 4 Bastien Nocera 2009-10-16 15:08:32 UTC
Attachment 145614 [details] pushed as 126b526 - Set a parent on the remove confirmation dialog