From 580f2be463c2ede353491c8ca0ea056b861aec69 Mon Sep 17 00:00:00 2001 From: Cody Russell Date: Thu, 3 Dec 2009 23:23:01 -0600 Subject: API changes, gtk menu support --- example/simple-client.c | 37 +- src/libappindicator/app-indicator.c | 760 +++++++++++++------------------ src/libappindicator/app-indicator.h | 81 ++-- tests/test-libappindicator-dbus-server.c | 1 + tests/test-libappindicator.c | 12 +- tests/test-simple-app.c | 1 + 6 files changed, 388 insertions(+), 504 deletions(-) diff --git a/example/simple-client.c b/example/simple-client.c index 80d6d9c..ab2e24d 100644 --- a/example/simple-client.c +++ b/example/simple-client.c @@ -29,31 +29,32 @@ GMainLoop * mainloop = NULL; int main (int argc, char ** argv) { - g_type_init(); + GtkWidget *menu = NULL; + AppIndicator *ci = NULL; - AppIndicator * ci = APP_INDICATOR(g_object_new(APP_INDICATOR_TYPE, NULL)); - g_assert(ci != NULL); + gtk_init (&argc, &argv); - app_indicator_set_id(ci, "example-simple-client"); - app_indicator_set_category(ci, APP_INDICATOR_CATEGORY_APPLICATION_STATUS); - app_indicator_set_status(ci, APP_INDICATOR_STATUS_ACTIVE); - app_indicator_set_icon(ci, "indicator-messages"); - app_indicator_set_attention_icon(ci, "indicator-messages-new"); + ci = app_indicator_new ("example-simple-client", + "indicator-messages", + APP_INDICATOR_CATEGORY_APPLICATION_STATUS); + + g_assert (IS_APP_INDICATOR (ci)); + g_assert (G_IS_OBJECT (ci)); - DbusmenuMenuitem * root = dbusmenu_menuitem_new(); + app_indicator_set_status (ci, APP_INDICATOR_STATUS_ACTIVE); + app_indicator_set_attention_icon(ci, "indicator-messages-new"); - DbusmenuMenuitem * item = dbusmenu_menuitem_new(); - dbusmenu_menuitem_property_set(item, DBUSMENU_MENUITEM_PROP_LABEL, "Item 1"); - dbusmenu_menuitem_child_append(root, item); + menu = gtk_menu_new (); + GtkWidget *item = gtk_menu_item_new_with_label ("1"); + gtk_menu_shell_append (GTK_MENU_SHELL (menu), item); - item = dbusmenu_menuitem_new(); - dbusmenu_menuitem_property_set(item, DBUSMENU_MENUITEM_PROP_LABEL, "Item 2"); - dbusmenu_menuitem_child_append(root, item); + item = gtk_menu_item_new_with_label ("2"); + gtk_menu_shell_append (GTK_MENU_SHELL (menu), item); - DbusmenuServer * menuservice = dbusmenu_server_new ("/need/a/menu/path"); - dbusmenu_server_set_root(menuservice, root); + item = gtk_menu_item_new_with_label ("3"); + gtk_menu_shell_append (GTK_MENU_SHELL (menu), item); - app_indicator_set_menu(ci, menuservice); + app_indicator_set_menu (ci, GTK_MENU (menu)); mainloop = g_main_loop_new(NULL, FALSE); g_main_loop_run(mainloop); diff --git a/src/libappindicator/app-indicator.c b/src/libappindicator/app-indicator.c index 141e1c0..c6a31ab 100644 --- a/src/libappindicator/app-indicator.c +++ b/src/libappindicator/app-indicator.c @@ -7,22 +7,22 @@ Copyright 2009 Canonical Ltd. Authors: Ted Gould -This program is free software: you can redistribute it and/or modify it +This program is free software: you can redistribute it and/or modify it under the terms of either or both of the following licenses: -1) the GNU Lesser General Public License version 3, as published by the +1) the GNU Lesser General Public License version 3, as published by the Free Software Foundation; and/or -2) the GNU Lesser General Public License version 2.1, as published by +2) the GNU Lesser General Public License version 2.1, as published by the Free Software Foundation. -This program is distributed in the hope that it will be useful, but -WITHOUT ANY WARRANTY; without even the implied warranties of -MERCHANTABILITY, SATISFACTORY QUALITY or FITNESS FOR A PARTICULAR -PURPOSE. See the applicable version of the GNU Lesser General Public +This program is distributed in the hope that it will be useful, but +WITHOUT ANY WARRANTY; without even the implied warranties of +MERCHANTABILITY, SATISFACTORY QUALITY or FITNESS FOR A PARTICULAR +PURPOSE. See the applicable version of the GNU Lesser General Public License for more details. -You should have received a copy of both the GNU Lesser General Public -License version 3 and version 2.1 along with this program. If not, see +You should have received a copy of both the GNU Lesser General Public +License version 3 and version 2.1 along with this program. If not, see */ @@ -54,19 +54,19 @@ License version 3 and version 2.1 along with this program. If not, see All of the private data in an instance of a application indicator. */ -typedef struct _AppIndicatorPrivate AppIndicatorPrivate; struct _AppIndicatorPrivate { /* Properties */ - gchar * id; - AppIndicatorCategory category; - AppIndicatorStatus status; - gchar * icon_name; - gchar * attention_icon_name; - DbusmenuServer * menu; + gchar *id; + AppIndicatorCategory category; + AppIndicatorStatus status; + gchar *icon_name; + gchar *attention_icon_name; + DbusmenuServer *menuservice; + GtkWidget *menu; /* Fun stuff */ - DBusGProxy * watcher_proxy; - DBusGConnection * connection; + DBusGProxy *watcher_proxy; + DBusGConnection *connection; }; /* Signals Stuff */ @@ -86,26 +86,20 @@ enum { PROP_0, PROP_ID, PROP_CATEGORY, - PROP_CATEGORY_ENUM, PROP_STATUS, - PROP_STATUS_ENUM, PROP_ICON_NAME, PROP_ATTENTION_ICON_NAME, PROP_MENU, - PROP_MENU_OBJECT, PROP_CONNECTED }; /* The strings so that they can be slowly looked up. */ #define PROP_ID_S "id" #define PROP_CATEGORY_S "category" -#define PROP_CATEGORY_ENUM_S "category-enum" #define PROP_STATUS_S "status" -#define PROP_STATUS_ENUM_S "status-enum" #define PROP_ICON_NAME_S "icon-name" #define PROP_ATTENTION_ICON_NAME_S "attention-icon-name" #define PROP_MENU_S "menu" -#define PROP_MENU_OBJECT_S "menu-object" #define PROP_CONNECTED_S "connected" /* Private macro, shhhh! */ @@ -143,77 +137,61 @@ app_indicator_class_init (AppIndicatorClass *klass) object_class->get_property = app_indicator_get_property; /* Properties */ - g_object_class_install_property(object_class, PROP_ID, - g_param_spec_string(PROP_ID_S, - "The ID for this indicator", - "An ID that should be unique, but used consistently by this program and it's indicator.", - NULL, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_CATEGORY, - g_param_spec_string(PROP_CATEGORY_S, - "Indicator Category as a string", - "The type of indicator that this represents as a string. For DBus.", - NULL, - G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_CATEGORY_ENUM, - g_param_spec_enum(PROP_CATEGORY_ENUM_S, - "Indicator Category", - "The type of indicator that this represents. Please don't use 'other'. Defaults to 'Application Status'.", - APP_INDICATOR_TYPE_INDICATOR_CATEGORY, - APP_INDICATOR_CATEGORY_APPLICATION_STATUS, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_STATUS, - g_param_spec_string(PROP_STATUS_S, - "Indicator Status as a string", - "The status of the indicator represented as a string. For DBus.", - NULL, - G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_STATUS_ENUM, - g_param_spec_enum(PROP_STATUS_ENUM_S, - "Indicator Status", - "Whether the indicator is shown or requests attention. Defaults to 'off'.", - APP_INDICATOR_TYPE_INDICATOR_STATUS, - APP_INDICATOR_STATUS_PASSIVE, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_ICON_NAME, - g_param_spec_string(PROP_ICON_NAME_S, - "An icon for the indicator", - "The default icon that is shown for the indicator.", - NULL, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_ATTENTION_ICON_NAME, - g_param_spec_string(PROP_ATTENTION_ICON_NAME_S, - "An icon to show when the indicator request attention.", - "If the indicator sets it's status to 'attention' then this icon is shown.", - NULL, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_MENU, - g_param_spec_string(PROP_MENU_S, - "The object path of the menu on DBus.", - "A method for getting the menu path as a string for DBus.", - NULL, - G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_MENU_OBJECT, - g_param_spec_object(PROP_MENU_OBJECT_S, - "The Menu for the indicator", - "A DBus Menu Server object that can have a menu attached to it. The object from this menu will be sent across the bus for the client to connect to and signal.", - DBUSMENU_TYPE_SERVER, - G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); - - g_object_class_install_property(object_class, PROP_CONNECTED, - g_param_spec_boolean(PROP_CONNECTED_S, - "Whether we're conneced to a watcher", - "Pretty simple, true if we have a reasonable expectation of being displayed through this object. You should hide your TrayIcon if so.", - FALSE, - G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); + g_object_class_install_property (object_class, + PROP_ID, + g_param_spec_string(PROP_ID_S, + "The ID for this indicator", + "An ID that should be unique, but used consistently by this program and it's indicator.", + NULL, + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS | G_PARAM_CONSTRUCT_ONLY)); + + g_object_class_install_property (object_class, + PROP_CATEGORY, + g_param_spec_string (PROP_CATEGORY_S, + "Indicator Category", + "The type of indicator that this represents. Please don't use 'other'. Defaults to 'Application Status'.", + NULL, + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS | G_PARAM_CONSTRUCT_ONLY)); + + g_object_class_install_property (object_class, + PROP_STATUS, + g_param_spec_string (PROP_STATUS_S, + "Indicator Status", + "Whether the indicator is shown or requests attention. Defaults to 'off'.", + NULL, + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); + + g_object_class_install_property(object_class, + PROP_ICON_NAME, + g_param_spec_string (PROP_ICON_NAME_S, + "An icon for the indicator", + "The default icon that is shown for the indicator.", + NULL, + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); + + g_object_class_install_property (object_class, + PROP_ATTENTION_ICON_NAME, + g_param_spec_string (PROP_ATTENTION_ICON_NAME_S, + "An icon to show when the indicator request attention.", + "If the indicator sets it's status to 'attention' then this icon is shown.", + NULL, + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); + + g_object_class_install_property(object_class, + PROP_MENU, + g_param_spec_string (PROP_MENU_S, + "The object path of the menu on DBus.", + "A method for getting the menu path as a string for DBus.", + NULL, + G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); + + g_object_class_install_property (object_class, + PROP_CONNECTED, + g_param_spec_boolean (PROP_CONNECTED_S, + "Whether we're conneced to a watcher", + "Pretty simple, true if we have a reasonable expectation of being displayed through this object. You should hide your TrayIcon if so.", + FALSE, + G_PARAM_READABLE | G_PARAM_STATIC_STRINGS)); /* Signals */ @@ -221,7 +199,7 @@ app_indicator_class_init (AppIndicatorClass *klass) /** AppIndicator::new-icon: @arg0: The #AppIndicator object - + Signaled when there is a new icon set for the object. */ @@ -236,7 +214,7 @@ app_indicator_class_init (AppIndicatorClass *klass) /** AppIndicator::new-attention-icon: @arg0: The #AppIndicator object - + Signaled when there is a new attention icon set for the object. */ @@ -252,7 +230,7 @@ app_indicator_class_init (AppIndicatorClass *klass) AppIndicator::new-status: @arg0: The #AppIndicator object @arg1: The string value of the #AppIndicatorStatus enum. - + Signaled when the status of the indicator changes. */ signals[NEW_STATUS] = g_signal_new (APP_INDICATOR_SIGNAL_NEW_STATUS, @@ -261,13 +239,14 @@ app_indicator_class_init (AppIndicatorClass *klass) G_STRUCT_OFFSET (AppIndicatorClass, new_status), NULL, NULL, g_cclosure_marshal_VOID__STRING, - G_TYPE_NONE, 1, G_TYPE_STRING, G_TYPE_NONE); + G_TYPE_NONE, 1, + G_TYPE_STRING); /** AppIndicator::connection-changed: @arg0: The #AppIndicator object @arg1: Whether we're connected or not - + Signaled when we connect to a watcher, or when it drops away. */ @@ -278,7 +257,7 @@ app_indicator_class_init (AppIndicatorClass *klass) NULL, NULL, g_cclosure_marshal_VOID__BOOLEAN, G_TYPE_NONE, 1, G_TYPE_BOOLEAN, G_TYPE_NONE); - + /* Initialize the object as a DBus type */ dbus_g_object_type_install_info(APP_INDICATOR_TYPE, &dbus_glib__notification_item_server_object_info); @@ -314,6 +293,8 @@ app_indicator_init (AppIndicator *self) "/need/a/path", G_OBJECT(self)); + self->priv = priv; + return; } @@ -322,10 +303,8 @@ app_indicator_init (AppIndicator *self) static void app_indicator_dispose (GObject *object) { - AppIndicator * self = APP_INDICATOR(object); - g_return_if_fail(self != NULL); - - AppIndicatorPrivate * priv = APP_INDICATOR_GET_PRIVATE(self); + AppIndicator *self = APP_INDICATOR (object); + AppIndicatorPrivate *priv = self->priv; if (priv->status != APP_INDICATOR_STATUS_PASSIVE) { app_indicator_set_status(self, APP_INDICATOR_STATUS_PASSIVE); @@ -351,10 +330,8 @@ app_indicator_dispose (GObject *object) static void app_indicator_finalize (GObject *object) { - AppIndicator * self = APP_INDICATOR(object); - g_return_if_fail(self != NULL); - - AppIndicatorPrivate * priv = APP_INDICATOR_GET_PRIVATE(self); + AppIndicator * self = APP_INDICATOR(object); + AppIndicatorPrivate *priv = self->priv; if (priv->status != APP_INDICATOR_STATUS_PASSIVE) { g_warning("Finalizing Application Status with the status set to: %d", priv->status); @@ -385,117 +362,70 @@ app_indicator_finalize (GObject *object) static void app_indicator_set_property (GObject * object, guint prop_id, const GValue * value, GParamSpec * pspec) { - AppIndicator * self = APP_INDICATOR(object); - g_return_if_fail(self != NULL); + AppIndicator *self = APP_INDICATOR (object); + AppIndicatorPrivate *priv = self->priv; + gboolean changed = FALSE; + const gchar *instr; + GEnumValue *enum_val; - AppIndicatorPrivate * priv = APP_INDICATOR_GET_PRIVATE(self); + switch (prop_id) { + case PROP_ID: + if (priv->id != NULL) { + g_warning ("Resetting ID value when I already had a value of: %s", priv->id); + g_free (priv->id); + priv->id = NULL; + } - switch (prop_id) { - /* *********************** */ - case PROP_ID: - if (G_VALUE_HOLDS_STRING(value)) { - if (priv->id != NULL) { - g_warning("Resetting ID value when I already had a value of: %s", priv->id); - g_free(priv->id); - priv->id = NULL; - } - priv->id = g_strdup(g_value_get_string(value)); - } else { - WARN_BAD_TYPE(PROP_ID_S, value); - } - check_connect(self); - break; - /* *********************** */ - case PROP_CATEGORY_ENUM: - if (G_VALUE_HOLDS_ENUM(value)) { - priv->category = g_value_get_enum(value); - } else { - WARN_BAD_TYPE(PROP_CATEGORY_ENUM_S, value); - } - break; - /* *********************** */ - case PROP_STATUS_ENUM: { - gboolean changed = FALSE; - if (G_VALUE_HOLDS_ENUM(value)) { - if (priv->status != g_value_get_enum(value)) { - changed = TRUE; - } - priv->status = g_value_get_enum(value); - } else { - WARN_BAD_TYPE(PROP_STATUS_ENUM_S, value); - } - if (changed) { - GParamSpecEnum * enumspec = G_PARAM_SPEC_ENUM(pspec); - if (enumspec != NULL) { - GEnumValue * enumval = g_enum_get_value(enumspec->enum_class, priv->status); - g_signal_emit(object, signals[NEW_STATUS], 0, enumval->value_nick, TRUE); - } - } - break; - } - /* *********************** */ - case PROP_ICON_NAME: - if (G_VALUE_HOLDS_STRING(value)) { - const gchar * instr = g_value_get_string(value); - gboolean changed = FALSE; - if (priv->icon_name == NULL) { - priv->icon_name = g_strdup(instr); - changed = TRUE; - } else if (!g_strcmp0(instr, priv->icon_name)) { - changed = FALSE; - } else { - g_free(priv->icon_name); - priv->icon_name = g_strdup(instr); - changed = TRUE; - } - if (changed) { - g_signal_emit(object, signals[NEW_ICON], 0, TRUE); - } - } else { - WARN_BAD_TYPE(PROP_ICON_NAME_S, value); - } - check_connect(self); - break; - /* *********************** */ - case PROP_ATTENTION_ICON_NAME: - if (G_VALUE_HOLDS_STRING(value)) { - const gchar * instr = g_value_get_string(value); - gboolean changed = FALSE; - if (priv->attention_icon_name == NULL) { - priv->attention_icon_name = g_strdup(instr); - changed = TRUE; - } else if (!g_strcmp0(instr, priv->attention_icon_name)) { - changed = FALSE; - } else { - g_free(priv->attention_icon_name); - priv->attention_icon_name = g_strdup(instr); - changed = TRUE; - } - if (changed) { - g_signal_emit(object, signals[NEW_ATTENTION_ICON], 0, TRUE); - } - } else { - WARN_BAD_TYPE(PROP_ATTENTION_ICON_NAME_S, value); - } - break; - /* *********************** */ - case PROP_MENU_OBJECT: - if (G_VALUE_HOLDS_OBJECT(value)) { - if (priv->menu != NULL) { - g_object_unref(G_OBJECT(priv->menu)); - } - priv->menu = DBUSMENU_SERVER(g_value_get_object(value)); - g_object_ref(G_OBJECT(priv->menu)); - } else { - WARN_BAD_TYPE(PROP_MENU_S, value); - } - check_connect(self); - break; - /* *********************** */ - default: - G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); - break; - } + priv->id = g_strdup (g_value_get_string (value)); + + check_connect (self); + break; + + case PROP_CATEGORY: + enum_val = g_enum_get_value_by_nick ((GEnumClass *) g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_CATEGORY), + g_value_get_string (value)); + + if (priv->category != enum_val->value) + { + priv->category = enum_val->value; + changed = TRUE; + } + + break; + + case PROP_STATUS: + enum_val = g_enum_get_value_by_nick ((GEnumClass *) g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_STATUS), + g_value_get_string (value)); + + app_indicator_set_status (APP_INDICATOR (object), + enum_val->value); + break; + + case PROP_ICON_NAME: + instr = g_value_get_string (value); + + if (g_strcmp0 (priv->icon_name, instr) != 0) + { + if (priv->icon_name) + g_free (priv->icon_name); + + priv->icon_name = g_strdup (instr); + + g_signal_emit (self, signals[NEW_ICON], 0, TRUE); + } + + check_connect (self); + break; + + case PROP_ATTENTION_ICON_NAME: + app_indicator_set_attention_icon (APP_INDICATOR (object), + g_value_get_string (value)); + break; + + default: + G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); + break; + } return; } @@ -504,115 +434,51 @@ app_indicator_set_property (GObject * object, guint prop_id, const GValue * valu static void app_indicator_get_property (GObject * object, guint prop_id, GValue * value, GParamSpec * pspec) { - AppIndicator * self = APP_INDICATOR(object); - g_return_if_fail(self != NULL); - - AppIndicatorPrivate * priv = APP_INDICATOR_GET_PRIVATE(self); - - switch (prop_id) { - /* *********************** */ - case PROP_ID: - if (G_VALUE_HOLDS_STRING(value)) { - g_value_set_string(value, priv->id); - } else { - WARN_BAD_TYPE(PROP_ID_S, value); - } - break; - /* *********************** */ - case PROP_CATEGORY: - if (G_VALUE_HOLDS_STRING(value)) { - GParamSpec * spec_for_enum = g_object_class_find_property(G_OBJECT_GET_CLASS(object), PROP_CATEGORY_ENUM_S); - GParamSpecEnum * enumspec = G_PARAM_SPEC_ENUM(spec_for_enum); - if (enumspec != NULL) { - GEnumValue * enumval = g_enum_get_value(enumspec->enum_class, priv->category); - g_value_set_string(value, enumval->value_nick); - } else { - g_assert_not_reached(); - } - } else { - WARN_BAD_TYPE(PROP_CATEGORY_S, value); - } - break; - /* *********************** */ - case PROP_CATEGORY_ENUM: - if (G_VALUE_HOLDS_ENUM(value)) { - /* We want the enum value */ - g_value_set_enum(value, priv->category); - } else { - WARN_BAD_TYPE(PROP_CATEGORY_ENUM_S, value); - } - break; - /* *********************** */ - case PROP_STATUS: - if (G_VALUE_HOLDS_STRING(value)) { - GParamSpec * spec_for_enum = g_object_class_find_property(G_OBJECT_GET_CLASS(object), PROP_STATUS_ENUM_S); - GParamSpecEnum * enumspec = G_PARAM_SPEC_ENUM(spec_for_enum); - if (enumspec != NULL) { - GEnumValue * enumval = g_enum_get_value(enumspec->enum_class, priv->status); - g_value_set_string(value, enumval->value_nick); - } else { - g_assert_not_reached(); - } - } else { - WARN_BAD_TYPE(PROP_STATUS_S, value); - } - break; - /* *********************** */ - case PROP_STATUS_ENUM: - if (G_VALUE_HOLDS_ENUM(value)) { - /* We want the enum value */ - g_value_set_enum(value, priv->status); - } else { - WARN_BAD_TYPE(PROP_STATUS_ENUM_S, value); - } - break; - /* *********************** */ - case PROP_ICON_NAME: - if (G_VALUE_HOLDS_STRING(value)) { - g_value_set_string(value, priv->icon_name); - } else { - WARN_BAD_TYPE(PROP_ICON_NAME_S, value); - } - break; - /* *********************** */ - case PROP_ATTENTION_ICON_NAME: - if (G_VALUE_HOLDS_STRING(value)) { - g_value_set_string(value, priv->attention_icon_name); - } else { - WARN_BAD_TYPE(PROP_ATTENTION_ICON_NAME_S, value); - } - break; - /* *********************** */ - case PROP_MENU: - if (G_VALUE_HOLDS_STRING(value)) { - if (priv->menu != NULL) { - g_object_get_property(G_OBJECT(priv->menu), DBUSMENU_SERVER_PROP_DBUS_OBJECT, value); - } - } else { - WARN_BAD_TYPE(PROP_MENU_S, value); - } - break; - /* *********************** */ - case PROP_MENU_OBJECT: - if (G_VALUE_HOLDS_OBJECT(value)) { - g_value_set_object(value, priv->menu); - } else { - WARN_BAD_TYPE(PROP_MENU_OBJECT_S, value); - } - break; - /* *********************** */ - case PROP_CONNECTED: - if (G_VALUE_HOLDS_BOOLEAN(value)) { - g_value_set_boolean(value, priv->watcher_proxy != NULL ? TRUE : FALSE); - } else { - WARN_BAD_TYPE(PROP_CONNECTED_S, value); - } - break; - /* *********************** */ - default: - G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); - break; - } + AppIndicator *self = APP_INDICATOR (object); + AppIndicatorPrivate *priv = self->priv; + GEnumValue *enum_value; + + switch (prop_id) { + case PROP_ID: + g_value_set_string (value, priv->id); + break; + + case PROP_CATEGORY: + enum_value = g_enum_get_value ((GEnumClass *) g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_CATEGORY), priv->category); + g_value_set_string (value, enum_value->value_nick); + break; + + case PROP_STATUS: + enum_value = g_enum_get_value ((GEnumClass *) g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_STATUS), priv->status); + g_value_set_string (value, enum_value->value_nick); + break; + + case PROP_ICON_NAME: + g_value_set_string (value, priv->icon_name); + break; + + case PROP_ATTENTION_ICON_NAME: + g_value_set_string (value, priv->attention_icon_name); + break; + + case PROP_MENU: + if (G_VALUE_HOLDS_STRING(value)) { + if (priv->menuservice != NULL) { + g_object_get_property (G_OBJECT (priv->menuservice), DBUSMENU_SERVER_PROP_DBUS_OBJECT, value); + } + } else { + WARN_BAD_TYPE(PROP_MENU_S, value); + } + break; + + case PROP_CONNECTED: + g_value_set_boolean (value, priv->watcher_proxy != NULL ? TRUE : FALSE); + break; + + default: + G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec); + break; + } return; } @@ -621,9 +487,9 @@ app_indicator_get_property (GObject * object, guint prop_id, GValue * value, GPa connect to things. If we do, and we're not connected, it connects for us. */ static void -check_connect (AppIndicator * self) +check_connect (AppIndicator *self) { - AppIndicatorPrivate * priv = APP_INDICATOR_GET_PRIVATE(self); + AppIndicatorPrivate *priv = self->priv; /* We're alreadying connecting or trying to connect. */ if (priv->watcher_proxy != NULL) return; @@ -664,43 +530,32 @@ register_service_cb (DBusGProxy * proxy, GError * error, gpointer data) return; } +static const gchar * +category_from_enum (AppIndicatorCategory category) +{ + GEnumValue *value; + + value = g_enum_get_value ((GEnumClass *)g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_CATEGORY), category); + return value->value_nick; +} + /* ************************* */ /* Public Functions */ /* ************************* */ -/** - app_indicator_set_id: - @ci: The #AppIndicator object to use - @id: ID to set for this indicator - - Wrapper function for property #AppIndicator::id. -*/ -void -app_indicator_set_id (AppIndicator * ci, const gchar * id) +AppIndicator * +app_indicator_new (const gchar *id, + const gchar *icon_name, + AppIndicatorCategory category) { - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_value_set_string(&value, id); - g_object_set_property(G_OBJECT(ci), PROP_ID_S, &value); - return; -} - -/** - app_indicator_set_category: - @ci: The #AppIndicator object to use - @category: The category to set for this indicator + AppIndicator *indicator = g_object_new (APP_INDICATOR_TYPE, + "id", id, + "category", category_from_enum (category), + "icon-name", icon_name, + NULL); - Wrapper function for property #AppIndicator::category. -*/ -void -app_indicator_set_category (AppIndicator * ci, AppIndicatorCategory category) -{ - GValue value = {0}; - g_value_init(&value, APP_INDICATOR_TYPE_INDICATOR_CATEGORY); - g_value_set_enum(&value, category); - g_object_set_property(G_OBJECT(ci), PROP_CATEGORY_ENUM_S, &value); - return; + return indicator; } /** @@ -711,29 +566,17 @@ app_indicator_set_category (AppIndicator * ci, AppIndicatorCategory category) Wrapper function for property #AppIndicator::status. */ void -app_indicator_set_status (AppIndicator * ci, AppIndicatorStatus status) +app_indicator_set_status (AppIndicator *self, AppIndicatorStatus status) { - GValue value = {0}; - g_value_init(&value, APP_INDICATOR_TYPE_INDICATOR_STATUS); - g_value_set_enum(&value, status); - g_object_set_property(G_OBJECT(ci), PROP_STATUS_ENUM_S, &value); - return; -} + g_return_if_fail (IS_APP_INDICATOR (self)); -/** - app_indicator_set_icon: - @ci: The #AppIndicator object to use - @icon_name: The name of the icon to set for this indicator + if (self->priv->status != status) + { + GEnumValue *value = g_enum_get_value ((GEnumClass *) g_type_class_ref (APP_INDICATOR_TYPE_INDICATOR_STATUS), status); - Wrapper function for property #AppIndicator::icon. -*/ -void app_indicator_set_icon (AppIndicator * ci, const gchar * icon_name) -{ - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_value_set_string(&value, icon_name); - g_object_set_property(G_OBJECT(ci), PROP_ICON_NAME_S, &value); - return; + self->priv->status = status; + g_signal_emit (self, signals[NEW_STATUS], 0, value->value_nick); + } } /** @@ -744,30 +587,91 @@ void app_indicator_set_icon (AppIndicator * ci, const gchar * icon_name) Wrapper function for property #AppIndicator::attention-icon. */ void -app_indicator_set_attention_icon (AppIndicator * ci, const gchar * icon_name) +app_indicator_set_attention_icon (AppIndicator *self, const gchar *icon_name) { - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_value_set_string(&value, icon_name); - g_object_set_property(G_OBJECT(ci), PROP_ATTENTION_ICON_NAME_S, &value); - return; + g_return_if_fail (IS_APP_INDICATOR (self)); + g_return_if_fail (icon_name != NULL); + + if (g_strcmp0 (self->priv->attention_icon_name, icon_name) != 0) + { + if (self->priv->attention_icon_name) + g_free (self->priv->attention_icon_name); + + self->priv->attention_icon_name = g_strdup (icon_name); + + g_signal_emit (self, signals[NEW_ATTENTION_ICON], 0, TRUE); + } } -/** - app_indicator_set_menu: - @ci: The #AppIndicator object to use - @menu: The object with the menu for the indicator +void +app_indicator_set_icon (AppIndicator *self, const gchar *icon_name) +{ + g_return_if_fail (IS_APP_INDICATOR (self)); + g_return_if_fail (icon_name != NULL); + + if (g_strcmp0 (self->priv->attention_icon_name, icon_name) != 0) + { + if (self->priv->attention_icon_name) + g_free (self->priv->attention_icon_name); + + self->priv->attention_icon_name = g_strdup (icon_name); + + g_signal_emit (self, signals[NEW_ATTENTION_ICON], 0, TRUE); + } +} + +static void +container_iterate (GtkWidget *widget, + gpointer data) +{ + DbusmenuMenuitem *root = (DbusmenuMenuitem *)data; + DbusmenuMenuitem *child; + + child = dbusmenu_menuitem_new (); + dbusmenu_menuitem_property_set (child, + DBUSMENU_MENUITEM_PROP_LABEL, + gtk_menu_item_get_label (GTK_MENU_ITEM (widget))); + dbusmenu_menuitem_child_append (root, child); +} + +static void +setup_dbusmenu (AppIndicator *self) +{ + AppIndicatorPrivate *priv; + DbusmenuMenuitem *root; + + priv = self->priv; + root = dbusmenu_menuitem_new (); + + gtk_container_forall (GTK_CONTAINER (priv->menu), + container_iterate, + root); + + priv->menuservice = dbusmenu_server_new ("/need/a/menu/path"); + dbusmenu_server_set_root (priv->menuservice, root); +} - Wrapper function for property #AppIndicator::menu. -*/ void -app_indicator_set_menu (AppIndicator * ci, DbusmenuServer * menu) +app_indicator_set_menu (AppIndicator *self, GtkMenu *menu) { - GValue value = {0}; - g_value_init(&value, G_TYPE_OBJECT); - g_value_set_object(&value, G_OBJECT(menu)); - g_object_set_property(G_OBJECT(ci), PROP_MENU_OBJECT_S, &value); - return; + AppIndicatorPrivate *priv; + + g_return_if_fail (IS_APP_INDICATOR (self)); + g_return_if_fail (GTK_IS_MENU (menu)); + + priv = self->priv; + + if (priv->menu != NULL) + { + g_object_unref (priv->menu); + } + + priv->menu = GTK_WIDGET (menu); + g_object_ref (priv->menu); + + setup_dbusmenu (self); + + check_connect (self); } /** @@ -779,12 +683,11 @@ app_indicator_set_menu (AppIndicator * ci, DbusmenuServer * menu) Return value: The current ID */ const gchar * -app_indicator_get_id (AppIndicator * ci) +app_indicator_get_id (AppIndicator *self) { - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_object_get_property(G_OBJECT(ci), PROP_ID_S, &value); - return g_value_get_string(&value); + g_return_val_if_fail (IS_APP_INDICATOR (self), NULL); + + return self->priv->id; } /** @@ -796,12 +699,11 @@ app_indicator_get_id (AppIndicator * ci) Return value: The current category. */ AppIndicatorCategory -app_indicator_get_category (AppIndicator * ci) +app_indicator_get_category (AppIndicator *self) { - GValue value = {0}; - g_value_init(&value, APP_INDICATOR_TYPE_INDICATOR_CATEGORY); - g_object_get_property(G_OBJECT(ci), PROP_CATEGORY_ENUM_S, &value); - return g_value_get_enum(&value); + g_return_val_if_fail (IS_APP_INDICATOR (self), APP_INDICATOR_CATEGORY_APPLICATION_STATUS); + + return self->priv->category; } /** @@ -813,12 +715,11 @@ app_indicator_get_category (AppIndicator * ci) Return value: The current status. */ AppIndicatorStatus -app_indicator_get_status (AppIndicator * ci) +app_indicator_get_status (AppIndicator *self) { - GValue value = {0}; - g_value_init(&value, APP_INDICATOR_TYPE_INDICATOR_STATUS); - g_object_get_property(G_OBJECT(ci), PROP_STATUS_ENUM_S, &value); - return g_value_get_enum(&value); + g_return_val_if_fail (IS_APP_INDICATOR (self), APP_INDICATOR_STATUS_PASSIVE); + + return self->priv->status; } /** @@ -830,12 +731,11 @@ app_indicator_get_status (AppIndicator * ci) Return value: The current icon name. */ const gchar * -app_indicator_get_icon (AppIndicator * ci) +app_indicator_get_icon (AppIndicator *self) { - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_object_get_property(G_OBJECT(ci), PROP_ICON_NAME_S, &value); - return g_value_get_string(&value); + g_return_val_if_fail (IS_APP_INDICATOR (self), NULL); + + return self->priv->icon_name; } /** @@ -847,29 +747,9 @@ app_indicator_get_icon (AppIndicator * ci) Return value: The current attention icon name. */ const gchar * -app_indicator_get_attention_icon (AppIndicator * ci) +app_indicator_get_attention_icon (AppIndicator *self) { - GValue value = {0}; - g_value_init(&value, G_TYPE_STRING); - g_object_get_property(G_OBJECT(ci), PROP_ATTENTION_ICON_NAME_S, &value); - return g_value_get_string(&value); -} - -/** - app_indicator_get_menu: - @ci: The #AppIndicator object to use - - Wrapper function for property #AppIndicator::menu. + g_return_val_if_fail (IS_APP_INDICATOR (self), NULL); - Return value: The current menu being used. -*/ -DbusmenuServer * -app_indicator_get_menu (AppIndicator * ci) -{ - GValue value = {0}; - g_value_init(&value, G_TYPE_OBJECT); - g_object_get_property(G_OBJECT(ci), PROP_MENU_OBJECT_S, &value); - return DBUSMENU_SERVER(g_value_get_object(&value)); + return self->priv->attention_icon_name; } - - diff --git a/src/libappindicator/app-indicator.h b/src/libappindicator/app-indicator.h index f955506..42b7cbe 100644 --- a/src/libappindicator/app-indicator.h +++ b/src/libappindicator/app-indicator.h @@ -7,31 +7,29 @@ Copyright 2009 Canonical Ltd. Authors: Ted Gould -This program is free software: you can redistribute it and/or modify it +This program is free software: you can redistribute it and/or modify it under the terms of either or both of the following licenses: -1) the GNU Lesser General Public License version 3, as published by the +1) the GNU Lesser General Public License version 3, as published by the Free Software Foundation; and/or -2) the GNU Lesser General Public License version 2.1, as published by +2) the GNU Lesser General Public License version 2.1, as published by the Free Software Foundation. -This program is distributed in the hope that it will be useful, but -WITHOUT ANY WARRANTY; without even the implied warranties of -MERCHANTABILITY, SATISFACTORY QUALITY or FITNESS FOR A PARTICULAR -PURPOSE. See the applicable version of the GNU Lesser General Public +This program is distributed in the hope that it will be useful, but +WITHOUT ANY WARRANTY; without even the implied warranties of +MERCHANTABILITY, SATISFACTORY QUALITY or FITNESS FOR A PARTICULAR +PURPOSE. See the applicable version of the GNU Lesser General Public License for more details. -You should have received a copy of both the GNU Lesser General Public -License version 3 and version 2.1 along with this program. If not, see +You should have received a copy of both the GNU Lesser General Public +License version 3 and version 2.1 along with this program. If not, see */ #ifndef __APP_INDICATOR_H__ #define __APP_INDICATOR_H__ -#include -#include -#include +#include G_BEGIN_DECLS @@ -83,8 +81,9 @@ typedef enum { /*< prefix=APP_INDICATOR_STATUS >*/ APP_INDICATOR_STATUS_ATTENTION } AppIndicatorStatus; -typedef struct _AppIndicator AppIndicator; -typedef struct _AppIndicatorClass AppIndicatorClass; +typedef struct _AppIndicator AppIndicator; +typedef struct _AppIndicatorClass AppIndicatorClass; +typedef struct _AppIndicatorPrivate AppIndicatorPrivate; /** AppIndicatorClass: @@ -106,13 +105,13 @@ struct _AppIndicatorClass { GObjectClass parent_class; /* DBus Signals */ - void (* new_icon) (AppIndicator * indicator, - gpointer user_data); - void (* new_attention_icon) (AppIndicator * indicator, - gpointer user_data); - void (* new_status) (AppIndicator * indicator, - gchar * status_string, - gpointer user_data); + void (* new_icon) (AppIndicator *indicator, + gpointer user_data); + void (* new_attention_icon) (AppIndicator *indicator, + gpointer user_data); + void (* new_status) (AppIndicator *indicator, + const gchar *status, + gpointer user_data); /* Local Signals */ void (* connection_changed) (AppIndicator * indicator, @@ -137,33 +136,33 @@ struct _AppIndicatorClass { */ struct _AppIndicator { GObject parent; - /* None. We're a very private object. */ + + AppIndicatorPrivate *priv; }; /* GObject Stuff */ -GType app_indicator_get_type (void); +GType app_indicator_get_type (void) G_GNUC_CONST; + +AppIndicator *app_indicator_new (const gchar *id, + const gchar *icon_name, + AppIndicatorCategory category); /* Set properties */ -void app_indicator_set_id (AppIndicator * ci, - const gchar * id); -void app_indicator_set_category (AppIndicator * ci, - AppIndicatorCategory category); -void app_indicator_set_status (AppIndicator * ci, - AppIndicatorStatus status); -void app_indicator_set_icon (AppIndicator * ci, - const gchar * icon_name); -void app_indicator_set_attention_icon (AppIndicator * ci, - const gchar * icon_name); -void app_indicator_set_menu (AppIndicator * ci, - DbusmenuServer * menu); +void app_indicator_set_status (AppIndicator *self, + AppIndicatorStatus status); +void app_indicator_set_attention_icon (AppIndicator *self, + const gchar *icon_name); +void app_indicator_set_menu (AppIndicator *self, + GtkMenu *menu); +void app_indicator_set_icon (AppIndicator *self, + const gchar *icon_name); /* Get properties */ -const gchar * app_indicator_get_id (AppIndicator * ci); -AppIndicatorCategory app_indicator_get_category (AppIndicator * ci); -AppIndicatorStatus app_indicator_get_status (AppIndicator * ci); -const gchar * app_indicator_get_icon (AppIndicator * ci); -const gchar * app_indicator_get_attention_icon (AppIndicator * ci); -DbusmenuServer * app_indicator_get_menu (AppIndicator * ci); +const gchar * app_indicator_get_id (AppIndicator *self); +AppIndicatorCategory app_indicator_get_category (AppIndicator *self); +AppIndicatorStatus app_indicator_get_status (AppIndicator *self); +const gchar * app_indicator_get_icon (AppIndicator *self); +const gchar * app_indicator_get_attention_icon (AppIndicator *self); G_END_DECLS diff --git a/tests/test-libappindicator-dbus-server.c b/tests/test-libappindicator-dbus-server.c index 6216c94..b316456 100644 --- a/tests/test-libappindicator-dbus-server.c +++ b/tests/test-libappindicator-dbus-server.c @@ -24,6 +24,7 @@ with this program. If not, see . #include #include #include +#include #include #include "test-defines.h" diff --git a/tests/test-libappindicator.c b/tests/test-libappindicator.c index 8b76d6e..c0ec1ff 100644 --- a/tests/test-libappindicator.c +++ b/tests/test-libappindicator.c @@ -42,7 +42,10 @@ test_libappindicator_prop_signals_helper (AppIndicator * ci, gboolean * signalac void test_libappindicator_prop_signals (void) { - AppIndicator * ci = APP_INDICATOR(g_object_new(APP_INDICATOR_TYPE, NULL)); + AppIndicator * ci = app_indicator_new ("test-app-indicator", + "indicator-messages", + APP_INDICATOR_CATEGORY_APPLICATION_STATUS); + g_assert(ci != NULL); gboolean signaled = FALSE; @@ -109,13 +112,12 @@ test_libappindicator_prop_signals (void) void test_libappindicator_init_set_props (void) { - AppIndicator * ci = APP_INDICATOR(g_object_new(APP_INDICATOR_TYPE, NULL)); + AppIndicator * ci = app_indicator_new ("my-id", + "my-name", + APP_INDICATOR_CATEGORY_APPLICATION_STATUS); g_assert(ci != NULL); - app_indicator_set_id(ci, "my-id"); - app_indicator_set_category(ci, APP_INDICATOR_CATEGORY_APPLICATION_STATUS); app_indicator_set_status(ci, APP_INDICATOR_STATUS_ACTIVE); - app_indicator_set_icon(ci, "my-name"); app_indicator_set_attention_icon(ci, "my-attention-name"); g_assert(!g_strcmp0("my-id", app_indicator_get_id(ci))); diff --git a/tests/test-simple-app.c b/tests/test-simple-app.c index 60ed88f..f5957c8 100644 --- a/tests/test-simple-app.c +++ b/tests/test-simple-app.c @@ -23,6 +23,7 @@ with this program. If not, see . #include #include #include +#include #include static GMainLoop * mainloop = NULL; -- cgit v1.2.3