ayatana-commits team mailing list archive

[Michael Terry <michael.terry@xxxxxxxxxxxxx>]

Michael Terry has proposed merging lp:~mterry/indicator-session/correct-printer-cmd into lp:indicator-session.

Requested reviews:
  Indicator Applet Developers (indicator-applet-developers)

For more details, see:
https://code.launchpad.net/~mterry/indicator-session/correct-printer-cmd/+merge/70189

Ubuntu doesn't use gnome-control-center's upstream printing panel.  Instead, it uses 'system-config-printer'.
-- 
https://code.launchpad.net/~mterry/indicator-session/correct-printer-cmd/+merge/70189
Your team ayatana-commits is subscribed to branch lp:indicator-session.

=== modified file 'src/device-menu-mgr.c'
--- src/device-menu-mgr.c	2011-07-22 14:36:59 +0000
+++ src/device-menu-mgr.c	2011-08-02 16:21:47 +0000
@@ -441,6 +441,19 @@
 }                                   
 
 static void
+show_printer_properties (DbusmenuMenuitem * mi,
+                         guint timestamp,
+                         gchar * type)
+{
+  GError * error = NULL;
+  if (!g_spawn_command_line_async("system-config-printer", &error))
+  {
+    g_warning("Unable to show dialog: %s", error->message);
+    g_error_free(error);
+  }
+}
+
+static void
 show_system_settings_with_context (DbusmenuMenuitem * mi,
                                    guint timestamp,
                                    gchar * type)
@@ -539,7 +552,7 @@
                                   _("Printers"));
   g_signal_connect (G_OBJECT(printers_menuitem),
                     DBUSMENU_MENUITEM_SIGNAL_ITEM_ACTIVATED,
-                    G_CALLBACK(show_system_settings_with_context),
+                    G_CALLBACK(show_printer_properties),
                     "printers");
   dbusmenu_menuitem_child_add_position(self->root_item,
                                        printers_menuitem,