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 112841 - [multi-head]screenshot pops error dialog on non default head
[multi-head]screenshot pops error dialog on non default head
Status: RESOLVED INCOMPLETE
Product: eel
Classification: Deprecated
Component: general
2.14.x
Other All
: High minor
: 2.14.x
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-05-12 14:56 UTC by Leena Gunda
Modified: 2008-07-20 18:12 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
patch which fixes the bug (452 bytes, patch)
2003-05-13 05:04 UTC, Leena Gunda
reviewed Details | Review

Description Leena Gunda 2003-05-12 14:56:46 UTC
Have two heads configured; Screen 0 (the primary screen) and Screen 1(the
second head). 

1. Invoke screenshot on Screen 1 (Select Actions:Screenshot from panel)
2. The screenshot comesup with a screenshot.png file.
3. drag and drop the screeshshot.png file to desktop,
4. the screenshot.png file is droped on to desktop.
5. repeat the step 3 again with the file name as screenshot.png
6. the dialog "File already exists would you like to replace it?"  comes up
on screen 0.

Expected Behavior - The dialog "File already exists would you like to
replace it?"  should come up on screen 1.
Comment 1 Leena Gunda 2003-05-13 05:02:49 UTC
This problem is because the screen where to display the message is not
fetched.
So the solution is to obtain the screen where to display the message
in eel_run_simple_dialog().
Attaching the patch.

Comment 2 Leena Gunda 2003-05-13 05:04:35 UTC
Created attachment 16480 [details] [review]
patch which fixes the bug
Comment 3 Christian Neumair 2006-03-18 11:38:48 UTC
Thanks for your bug report and your patch, and sorry for the late response!

Maybe you could send your patch to the nautilus mailing list [1] for review?

[1] http://mail.gnome.org/mailman/listinfo/nautilus-list
Comment 4 Alexander Larsson 2006-08-08 14:49:27 UTC
I don't think this is needed. We already set transient_for the parent, and that should handle this (these days).

Can someone verify this?
Comment 5 Christian Neumair 2006-08-08 16:10:28 UTC
I can confirm that gtk_window_set_transient_for() calls

      gtk_window_set_screen (window, parent->screen);

and eel_alert_dialog_new() calls gtk_window_set_transient_for(). However I'm a bit reluctant to close this bug report as the above call chain has been present since 2002 [1], and this report is from 2003, so we might have overlooked something.

[1] http://cvs.gnome.org/viewcvs/gtk%2B/gtk/gtkwindow.c?r1=1.227&r2=1.228
Comment 6 Cosimo Cecchi 2008-07-20 18:12:01 UTC
Closing this bug report as no further information has been provided. Please feel free to reopen this bug if you can provide the information asked for.
Thanks!