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 323759 - about dialogue not above
about dialogue not above
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Browser plugin (obsolete)
unspecified
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2005-12-10 23:45 UTC by Christian Persch
Modified: 2006-01-23 18:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
totem-parent-about-box.patch (3.88 KB, patch)
2005-12-23 13:20 UTC, Bastien Nocera
none Details | Review

Description Christian Persch 2005-12-10 23:45:15 UTC
Steps:

0) Load a web page which embeds a totem plugin
1) Right click, About

Result:
The about dialogue isn't parented to the window that contains the web page
containing the plugin; if you click somewhere on the page the about dialogue
disappears and you have to minimise all windows before you can find it again.
Comment 1 Bastien Nocera 2005-12-23 02:56:23 UTC
gtk_widget_get_parent_window() on the video widget fails. I'm not too sure how to get the parent window of the plugin from another process, even though it's embedded.
Comment 2 Christian Persch 2005-12-23 09:43:23 UTC
I think there's code in bonobo that does that, bonobo_control_set_transient_for.
Comment 3 Bastien Nocera 2005-12-23 13:20:29 UTC
Created attachment 56330 [details] [review]
totem-parent-about-box.patch

This doesn't work, and I'm really wondering why.
Comment 4 Bastien Nocera 2006-01-23 18:08:15 UTC
It didn't work because I'm an idiot and forgot to remove the gtk_window_set_transient call below.
Comment 5 Bastien Nocera 2006-01-23 18:42:05 UTC
2006-01-23  Bastien Nocera  <hadess@hadess.net>

        * src/totem-mozilla-viewer.c: (totem_gtk_plug_get_toplevel),
        (on_about1_activate), (main): Add a totem_gtk_plug_get_toplevel ()
        function, and correctly set the about dialogue as a transient for the
        Mozilla toplevel window (Closes: #323759)