[eog] Make EogPluginEngine GSEAL-compatible
- From: Felix Riemann <friemann src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [eog] Make EogPluginEngine GSEAL-compatible
- Date: Sun, 9 May 2010 20:18:53 +0000 (UTC)
commit 15d23daf9b43d9cb632033518ec88458b26d61e8
Author: Felix Riemann <friemann gnome org>
Date: Sun May 9 22:16:12 2010 +0200
Make EogPluginEngine GSEAL-compatible
Has a workaround for the different behaviour of gtk_window_get_group()
that makes sure that the configure dialog and its parent share a window
group of their own as before. Not sure if it's needed.
Part of bug 606883.
src/eog-plugin-engine.c | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
---
diff --git a/src/eog-plugin-engine.c b/src/eog-plugin-engine.c
index b2cede3..9e5d0f1 100644
--- a/src/eog-plugin-engine.c
+++ b/src/eog-plugin-engine.c
@@ -818,9 +818,12 @@ eog_plugin_engine_configure_plugin (EogPluginInfo *info,
gtk_window_set_transient_for (GTK_WINDOW (conf_dlg),
parent);
- wg = parent->group;
+ // Will return a default group if no group is set
+ wg = gtk_window_get_group (parent);
- if (wg == NULL) {
+ // For now assign a dedicated window group if it is
+ // the default one until we know if this is really needed
+ if (wg == gtk_window_get_group (NULL)) {
wg = gtk_window_group_new ();
gtk_window_group_add_window (wg, parent);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]