]> git.0d.be Git - empathy.git/blobdiff - libempathy-gtk/empathy-contact-list-view.c
Move the event manager to src/
[empathy.git] / libempathy-gtk / empathy-contact-list-view.c
index b0bfdf909e70ef42cec60a5eacb31e62f8cdcc88..dd3dc54a6d5d5d682189711a42c6cd85e490c2fe 100644 (file)
@@ -28,6 +28,7 @@
 #include <string.h>
 
 #include <glib/gi18n.h>
+#include <gdk/gdkkeysyms.h>
 #include <gtk/gtk.h>
 #include <glade/glade.h>
 
 #include <libempathy/empathy-contact-factory.h>
 #include <libempathy/empathy-contact-list.h>
 #include <libempathy/empathy-contact-groups.h>
-#include <libempathy/empathy-debug.h>
+#include <libempathy/empathy-dispatcher.h>
 #include <libempathy/empathy-utils.h>
 
 #include "empathy-contact-list-view.h"
 #include "empathy-contact-list-store.h"
-#include "empathy-contact-menu.h"
 #include "empathy-images.h"
 #include "empathy-cell-renderer-expander.h"
 #include "empathy-cell-renderer-text.h"
 #include "empathy-gtk-enum-types.h"
 #include "empathy-gtk-marshal.h"
 
-#define DEBUG_DOMAIN "ContactListView"
-
-/* Flashing delay for icons (milliseconds). */
-#define FLASH_TIMEOUT 500
+#define DEBUG_FLAG EMPATHY_DEBUG_CONTACT
+#include <libempathy/empathy-debug.h>
 
 /* Active users are those which have recently changed state
  * (e.g. online, offline or from normal to a busy state).
  */
 
-#define GET_PRIV(obj) (G_TYPE_INSTANCE_GET_PRIVATE ((obj), EMPATHY_TYPE_CONTACT_LIST_VIEW, EmpathyContactListViewPriv))
-
+#define GET_PRIV(obj) EMPATHY_GET_PRIV (obj, EmpathyContactListView)
 typedef struct {
-       EmpathyContactListStore    *store;
-       GtkTreeRowReference        *drag_row;
-       EmpathyContactListFeatures  features;
+       EmpathyContactListStore        *store;
+       GtkTreeRowReference            *drag_row;
+       EmpathyContactListFeatureFlags  list_features;
+       EmpathyContactFeatureFlags      contact_features;
+       GtkWidget                      *tooltip_widget;
 } EmpathyContactListViewPriv;
 
 typedef struct {
@@ -81,7 +80,9 @@ typedef struct {
 
 enum {
        PROP_0,
-       PROP_FEATURES
+       PROP_STORE,
+       PROP_LIST_FEATURES,
+       PROP_CONTACT_FEATURES,
 };
 
 enum DndDragType {
@@ -113,6 +114,58 @@ static guint signals[LAST_SIGNAL];
 
 G_DEFINE_TYPE (EmpathyContactListView, empathy_contact_list_view, GTK_TYPE_TREE_VIEW);
 
+static gboolean
+contact_list_view_query_tooltip_cb (EmpathyContactListView *view,
+                                   gint                    x,
+                                   gint                    y,
+                                   gboolean                keyboard_mode,
+                                   GtkTooltip             *tooltip,
+                                   gpointer                user_data)
+{
+       EmpathyContactListViewPriv *priv = GET_PRIV (view);
+       EmpathyContact             *contact;
+       GtkTreeModel               *model;
+       GtkTreeIter                 iter;
+       GtkTreePath                *path;
+
+       /* FIXME: We need GTK version >= 2.12.10. See GNOME bug #504087 */
+       if (gtk_check_version (2, 12, 10)) {
+               return FALSE;
+       }
+
+       if (!gtk_tree_view_get_tooltip_context (GTK_TREE_VIEW (view), &x, &y,
+                                               keyboard_mode,
+                                               &model, &path, &iter)) {
+               return FALSE;
+       }
+
+       gtk_tree_view_set_tooltip_row (GTK_TREE_VIEW (view), tooltip, path);
+       gtk_tree_path_free (path);
+
+       gtk_tree_model_get (model, &iter,
+                           EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, &contact,
+                           -1);
+       if (!contact) {
+               return FALSE;
+       }
+
+       if (!priv->tooltip_widget) {
+               priv->tooltip_widget = empathy_contact_widget_new (contact,
+                       EMPATHY_CONTACT_WIDGET_EDIT_NONE);
+               g_object_add_weak_pointer (G_OBJECT (priv->tooltip_widget),
+                                          (gpointer) &priv->tooltip_widget);
+       } else {
+               empathy_contact_widget_set_contact (priv->tooltip_widget,
+                                                   contact);
+       }
+
+       gtk_tooltip_set_custom (tooltip, priv->tooltip_widget);
+
+       g_object_unref (contact);
+
+       return TRUE;
+}
+
 static void
 contact_list_view_drag_data_received (GtkWidget         *widget,
                                      GdkDragContext    *context,
@@ -139,10 +192,10 @@ contact_list_view_drag_data_received (GtkWidget         *widget,
        priv = GET_PRIV (widget);
 
        id = (const gchar*) selection->data;
-       empathy_debug (DEBUG_DOMAIN, "Received %s%s drag & drop contact from roster with id:'%s'",
-                     context->action == GDK_ACTION_MOVE ? "move" : "",
-                     context->action == GDK_ACTION_COPY ? "copy" : "",
-                     id);
+       DEBUG ("Received %s%s drag & drop contact from roster with id:'%s'",
+               context->action == GDK_ACTION_MOVE ? "move" : "",
+               context->action == GDK_ACTION_COPY ? "copy" : "",
+               id);
 
        strv = g_strsplit (id, "/", 2);
        factory = empathy_contact_factory_new ();
@@ -157,7 +210,7 @@ contact_list_view_drag_data_received (GtkWidget         *widget,
        g_strfreev (strv);
 
        if (!contact) {
-               empathy_debug (DEBUG_DOMAIN, "No contact found associated with drag & drop");
+               DEBUG ("No contact found associated with drag & drop");
                return;
        }
 
@@ -188,11 +241,10 @@ contact_list_view_drag_data_received (GtkWidget         *widget,
                gtk_tree_path_free (path);
        }
 
-       empathy_debug (DEBUG_DOMAIN,
-                     "contact %s (%d) dragged from '%s' to '%s'",
-                     empathy_contact_get_id (contact),
-                     empathy_contact_get_handle (contact),
-                     old_group, new_group);
+       DEBUG ("contact %s (%d) dragged from '%s' to '%s'",
+               empathy_contact_get_id (contact),
+               empathy_contact_get_handle (contact),
+               old_group, new_group);
 
        list = empathy_contact_list_store_get_list_iface (priv->store);
        if (new_group) {
@@ -388,67 +440,69 @@ contact_list_view_drag_drop (GtkWidget      *widget,
        return FALSE;
 }
 
+typedef struct {
+       EmpathyContactListView *view;
+       guint                   button;
+       guint32                 time;
+} MenuPopupData;
+
 static gboolean
-contact_list_view_button_press_event_cb (EmpathyContactListView *view,
-                                        GdkEventButton        *event,
-                                        gpointer               user_data)
+contact_list_view_popup_menu_idle_cb (gpointer user_data)
 {
-       EmpathyContactListViewPriv *priv;
-       EmpathyContact             *contact;
-       GtkTreePath               *path;
-       GtkTreeSelection          *selection;
-       GtkTreeModel              *model;
-       GtkTreeIter                iter;
-       gboolean                   row_exists;
-       GtkWidget                 *menu;
+       MenuPopupData *data = user_data;
+       GtkWidget     *menu;
 
-       priv = GET_PRIV (view);
-
-       if (event->button != 3) {
-               return FALSE;
+       menu = empathy_contact_list_view_get_contact_menu (data->view);
+       if (!menu) {
+               menu = empathy_contact_list_view_get_group_menu (data->view);
        }
 
-       selection = gtk_tree_view_get_selection (GTK_TREE_VIEW (view));
-       model = gtk_tree_view_get_model (GTK_TREE_VIEW (view));
-
-       gtk_widget_grab_focus (GTK_WIDGET (view));
-
-       row_exists = gtk_tree_view_get_path_at_pos (GTK_TREE_VIEW (view),
-                                                   event->x, event->y,
-                                                   &path,
-                                                   NULL, NULL, NULL);
-       if (!row_exists) {
-               return FALSE;
+       if (menu) {
+               gtk_widget_show (menu);
+               gtk_menu_popup (GTK_MENU (menu),
+                               NULL, NULL, NULL, NULL,
+                               data->button, data->time);
        }
 
-       gtk_tree_selection_unselect_all (selection);
-       gtk_tree_selection_select_path (selection, path);
-
-       gtk_tree_model_get_iter (model, &iter, path);
-       gtk_tree_path_free (path);
-
-       gtk_tree_model_get (model, &iter,
-                           EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, &contact,
-                           -1);
+       g_slice_free (MenuPopupData, data);
 
-       if (contact) {
-               menu = empathy_contact_list_view_get_contact_menu (view, contact);
-               g_object_unref (contact);
-       } else {
-               menu = empathy_contact_list_view_get_group_menu (view);
-       }
+       return FALSE;
+}
 
-       if (!menu) {
-               return FALSE;
+static gboolean
+contact_list_view_button_press_event_cb (EmpathyContactListView *view,
+                                        GdkEventButton         *event,
+                                        gpointer                user_data)
+{
+       if (event->button == 3) {
+               MenuPopupData *data;
+
+               data = g_slice_new (MenuPopupData);
+               data->view = view;
+               data->button = event->button;
+               data->time = event->time;
+               g_idle_add (contact_list_view_popup_menu_idle_cb, data);
        }
 
-       gtk_widget_show (menu);
+       return FALSE;
+}
 
-       gtk_menu_popup (GTK_MENU (menu),
-                       NULL, NULL, NULL, NULL,
-                       event->button, event->time);
+static gboolean
+contact_list_view_key_press_event_cb (EmpathyContactListView *view,
+                                     GdkEventKey            *event,
+                                     gpointer                user_data)
+{
+       if (event->keyval == GDK_Menu) {
+               MenuPopupData *data;
+
+               data = g_slice_new (MenuPopupData);
+               data->view = view;
+               data->button = 0;
+               data->time = event->time;
+               g_idle_add (contact_list_view_popup_menu_idle_cb, data);
+       }
 
-       return TRUE;
+       return FALSE;
 }
 
 static void
@@ -462,21 +516,22 @@ contact_list_view_row_activated_cb (EmpathyContactListView *view,
        GtkTreeModel               *model;
        GtkTreeIter                 iter;
 
-       if (!(priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CHAT)) {
-               return;
-       }
-
-       model = gtk_tree_view_get_model (GTK_TREE_VIEW (view));
-
+       model = GTK_TREE_MODEL (priv->store);
        gtk_tree_model_get_iter (model, &iter, path);
        gtk_tree_model_get (model, &iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, &contact,
                            -1);
 
-       if (contact) {
-               empathy_chat_with_contact (contact);
-               g_object_unref (contact);
+       if (!contact) {
+               return;
+       }
+
+       /* There is no event for the contact, default action is starting a chat */
+       if (priv->contact_features & EMPATHY_CONTACT_FEATURE_CHAT) {
+               empathy_dispatcher_chat_with_contact (contact);
        }
+
+       g_object_unref (contact);
 }
 
 static void
@@ -489,7 +544,7 @@ contact_list_view_voip_activated_cb (EmpathyCellRendererActivatable *cell,
        GtkTreeIter                 iter;
        EmpathyContact             *contact;
 
-       if (!(priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CALL)) {
+       if (!(priv->contact_features & EMPATHY_CONTACT_FEATURE_CALL)) {
                return;
        }
 
@@ -503,7 +558,7 @@ contact_list_view_voip_activated_cb (EmpathyCellRendererActivatable *cell,
                            -1);
 
        if (contact) {
-               empathy_call_with_contact (contact);
+               empathy_dispatcher_call_with_contact (contact);
                g_object_unref (contact);
        }
 }
@@ -693,7 +748,7 @@ contact_list_view_row_expand_or_collapse_cb (EmpathyContactListView *view,
        gchar                      *name;
        gboolean                    expanded;
 
-       if (!(priv->features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_SAVE)) {
+       if (!(priv->list_features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_SAVE)) {
                return;
        }
 
@@ -729,7 +784,7 @@ contact_list_view_row_has_child_toggled_cb (GtkTreeModel          *model,
                return;
        }
 
-       if (!(priv->features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_SAVE) ||
+       if (!(priv->list_features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_SAVE) ||
            empathy_contact_group_get_expanded (name)) {
                g_signal_handlers_block_by_func (view,
                                                 contact_list_view_row_expand_or_collapse_cb,
@@ -864,6 +919,45 @@ contact_list_view_setup (EmpathyContactListView *view)
        }
 }
 
+static void
+contact_list_view_set_list_features (EmpathyContactListView         *view,
+                                    EmpathyContactListFeatureFlags  features)
+{
+       EmpathyContactListViewPriv *priv = GET_PRIV (view);
+       gboolean                    has_tooltip;
+
+       g_return_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view));
+
+       priv->list_features = features;
+
+       /* Update DnD source/dest */
+       if (features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_DRAG) {
+               gtk_drag_source_set (GTK_WIDGET (view),
+                                    GDK_BUTTON1_MASK,
+                                    drag_types_source,
+                                    G_N_ELEMENTS (drag_types_source),
+                                    GDK_ACTION_MOVE | GDK_ACTION_COPY);
+       } else {
+               gtk_drag_source_unset (GTK_WIDGET (view));
+
+       }
+
+       if (features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_DROP) {
+               gtk_drag_dest_set (GTK_WIDGET (view),
+                                  GTK_DEST_DEFAULT_ALL,
+                                  drag_types_dest,
+                                  G_N_ELEMENTS (drag_types_dest),
+                                  GDK_ACTION_MOVE | GDK_ACTION_COPY);
+       } else {
+               /* FIXME: URI could still be droped depending on FT feature */
+               gtk_drag_dest_unset (GTK_WIDGET (view));
+       }
+
+       /* Update has-tooltip */
+       has_tooltip = (features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_TOOLTIP) != 0;
+       gtk_widget_set_has_tooltip (GTK_WIDGET (view), has_tooltip);
+}
+
 static void
 contact_list_view_finalize (GObject *object)
 {
@@ -889,8 +983,14 @@ contact_list_view_get_property (GObject    *object,
        priv = GET_PRIV (object);
 
        switch (param_id) {
-       case PROP_FEATURES:
-               g_value_set_flags (value, priv->features);
+       case PROP_STORE:
+               g_value_set_object (value, priv->store);
+               break;
+       case PROP_LIST_FEATURES:
+               g_value_set_flags (value, priv->list_features);
+               break;
+       case PROP_CONTACT_FEATURES:
+               g_value_set_flags (value, priv->contact_features);
                break;
        default:
                G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
@@ -905,13 +1005,18 @@ contact_list_view_set_property (GObject      *object,
                                GParamSpec   *pspec)
 {
        EmpathyContactListView     *view = EMPATHY_CONTACT_LIST_VIEW (object);
-       EmpathyContactListViewPriv *priv;
-
-       priv = GET_PRIV (object);
+       EmpathyContactListViewPriv *priv = GET_PRIV (object);
 
        switch (param_id) {
-       case PROP_FEATURES:
-               empathy_contact_list_view_set_features (view, g_value_get_flags (value));
+       case PROP_STORE:
+               priv->store = g_value_dup_object (value);
+               contact_list_view_setup (view);
+               break;
+       case PROP_LIST_FEATURES:
+               contact_list_view_set_list_features (view, g_value_get_flags (value));
+               break;
+       case PROP_CONTACT_FEATURES:
+               priv->contact_features = g_value_get_flags (value);
                break;
        default:
                G_OBJECT_WARN_INVALID_PROPERTY_ID (object, param_id, pspec);
@@ -951,12 +1056,27 @@ empathy_contact_list_view_class_init (EmpathyContactListViewClass *klass)
                              3, EMPATHY_TYPE_CONTACT, G_TYPE_STRING, G_TYPE_STRING);
 
        g_object_class_install_property (object_class,
-                                        PROP_FEATURES,
-                                        g_param_spec_flags ("features",
+                                        PROP_STORE,
+                                        g_param_spec_object ("store",
+                                                            "The store of the view",
+                                                            "The store of the view",
+                                                             EMPATHY_TYPE_CONTACT_LIST_STORE,
+                                                             G_PARAM_CONSTRUCT_ONLY | G_PARAM_READWRITE));
+       g_object_class_install_property (object_class,
+                                        PROP_LIST_FEATURES,
+                                        g_param_spec_flags ("list-features",
                                                             "Features of the view",
                                                             "Falgs for all enabled features",
-                                                             EMPATHY_TYPE_CONTACT_LIST_FEATURES,
-                                                             0,
+                                                             EMPATHY_TYPE_CONTACT_LIST_FEATURE_FLAGS,
+                                                             EMPATHY_CONTACT_LIST_FEATURE_NONE,
+                                                             G_PARAM_READWRITE));
+       g_object_class_install_property (object_class,
+                                        PROP_CONTACT_FEATURES,
+                                        g_param_spec_flags ("contact-features",
+                                                            "Features of the contact menu",
+                                                            "Falgs for all enabled features for the menu",
+                                                             EMPATHY_TYPE_CONTACT_FEATURE_FLAGS,
+                                                             EMPATHY_CONTACT_FEATURE_NONE,
                                                              G_PARAM_READWRITE));
 
        g_type_class_add_private (object_class, sizeof (EmpathyContactListViewPriv));
@@ -965,6 +1085,10 @@ empathy_contact_list_view_class_init (EmpathyContactListViewClass *klass)
 static void
 empathy_contact_list_view_init (EmpathyContactListView *view)
 {
+       EmpathyContactListViewPriv *priv = G_TYPE_INSTANCE_GET_PRIVATE (view,
+               EMPATHY_TYPE_CONTACT_LIST_VIEW, EmpathyContactListViewPriv);
+
+       view->priv = priv;
        /* Get saved group states. */
        empathy_contact_groups_get_all ();
 
@@ -973,88 +1097,38 @@ empathy_contact_list_view_init (EmpathyContactListView *view)
                                              NULL, NULL);
 
        /* Connect to tree view signals rather than override. */
-       g_signal_connect (view,
-                         "button-press-event",
+       g_signal_connect (view, "button-press-event",
                          G_CALLBACK (contact_list_view_button_press_event_cb),
                          NULL);
-       g_signal_connect (view,
-                         "row-activated",
+       g_signal_connect (view, "key-press-event",
+                         G_CALLBACK (contact_list_view_key_press_event_cb),
+                         NULL);
+       g_signal_connect (view, "row-activated",
                          G_CALLBACK (contact_list_view_row_activated_cb),
                          NULL);
-       g_signal_connect (view,
-                         "row-expanded",
+       g_signal_connect (view, "row-expanded",
                          G_CALLBACK (contact_list_view_row_expand_or_collapse_cb),
                          GINT_TO_POINTER (TRUE));
-       g_signal_connect (view,
-                         "row-collapsed",
+       g_signal_connect (view, "row-collapsed",
                          G_CALLBACK (contact_list_view_row_expand_or_collapse_cb),
                          GINT_TO_POINTER (FALSE));
+       g_signal_connect (view, "query-tooltip",
+                         G_CALLBACK (contact_list_view_query_tooltip_cb),
+                         NULL);
 }
 
 EmpathyContactListView *
-empathy_contact_list_view_new (EmpathyContactListStore    *store,
-                              EmpathyContactListFeatures  features)
+empathy_contact_list_view_new (EmpathyContactListStore        *store,
+                              EmpathyContactListFeatureFlags  list_features,
+                              EmpathyContactFeatureFlags      contact_features)
 {
-       EmpathyContactListView     *view;
-       EmpathyContactListViewPriv *priv;
-
        g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_STORE (store), NULL);
        
-       view = g_object_new (EMPATHY_TYPE_CONTACT_LIST_VIEW,
-                           "features", features,
-                           NULL);
-
-       priv = GET_PRIV (view);
-       priv->store = g_object_ref (store);
-       contact_list_view_setup (EMPATHY_CONTACT_LIST_VIEW (view));
-
-       return view;
-}
-
-void
-empathy_contact_list_view_set_features (EmpathyContactListView     *view,
-                                       EmpathyContactListFeatures  features)
-{
-       EmpathyContactListViewPriv *priv = GET_PRIV (view);
-
-       g_return_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view));
-
-       priv->features = features;
-
-       /* Update DnD source/dest */
-       if (features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_DRAG) {
-               gtk_drag_source_set (GTK_WIDGET (view),
-                                    GDK_BUTTON1_MASK,
-                                    drag_types_source,
-                                    G_N_ELEMENTS (drag_types_source),
-                                    GDK_ACTION_MOVE | GDK_ACTION_COPY);
-       } else {
-               gtk_drag_source_unset (GTK_WIDGET (view));
-
-       }
-
-       if (features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_DROP) {
-               gtk_drag_dest_set (GTK_WIDGET (view),
-                                  GTK_DEST_DEFAULT_ALL,
-                                  drag_types_dest,
-                                  G_N_ELEMENTS (drag_types_dest),
-                                  GDK_ACTION_MOVE | GDK_ACTION_COPY);
-       } else {
-               /* FIXME: URI could still be  droped depending on FT feature */
-               gtk_drag_dest_unset (GTK_WIDGET (view));
-       }
-
-       g_object_notify (G_OBJECT (view), "features");
-}
-
-EmpathyContactListFeatures
-empathy_contact_list_view_get_features (EmpathyContactListView  *view)
-{
-       EmpathyContactListViewPriv *priv = GET_PRIV (view);
-
-       g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view), FALSE);
-
-       return priv->features;
+       return g_object_new (EMPATHY_TYPE_CONTACT_LIST_VIEW,
+                            "store", store,
+                            "contact-features", contact_features,
+                            "list-features", list_features,
+                            NULL);
 }
 
 EmpathyContact *
@@ -1124,8 +1198,8 @@ contact_list_view_remove_dialog_show (GtkWindow   *parent,
        
        dialog = gtk_dialog_new_with_buttons (window_title, parent,
                                              GTK_DIALOG_MODAL,
+                                             GTK_STOCK_CANCEL, GTK_RESPONSE_NO,
                                              GTK_STOCK_DELETE, GTK_RESPONSE_YES,
-                                             GTK_STOCK_CANCEL, GTK_RESPONSE_NO,
                                              NULL);
        gtk_dialog_set_has_separator (GTK_DIALOG(dialog), FALSE);
         
@@ -1161,7 +1235,7 @@ contact_list_view_group_remove_activate_cb (GtkMenuItem            *menuitem,
                gchar     *text;
                GtkWindow *parent;
 
-               text = g_strdup_printf (_("Do you really want to remove the group '%s' ?"), group);
+               text = g_strdup_printf (_("Do you really want to remove the group '%s'?"), group);
                parent = empathy_get_toplevel_window (GTK_WIDGET (view));
                if (contact_list_view_remove_dialog_show (parent, _("Removing group"), text)) {
                        EmpathyContactList *list;
@@ -1180,14 +1254,20 @@ GtkWidget *
 empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
 {
        EmpathyContactListViewPriv *priv = GET_PRIV (view);
+       gchar                      *group;
        GtkWidget                  *menu;
        GtkWidget                  *item;
        GtkWidget                  *image;
 
        g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view), NULL);
 
-       if (!(priv->features & (EMPATHY_CONTACT_LIST_FEATURE_GROUPS_RENAME |
-                               EMPATHY_CONTACT_LIST_FEATURE_GROUPS_REMOVE))) {
+       if (!(priv->list_features & (EMPATHY_CONTACT_LIST_FEATURE_GROUPS_RENAME |
+                                    EMPATHY_CONTACT_LIST_FEATURE_GROUPS_REMOVE))) {
+               return NULL;
+       }
+
+       group = empathy_contact_list_view_get_selected_group (view);
+       if (!group) {
                return NULL;
        }
 
@@ -1203,7 +1283,7 @@ empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
                                  view);
        }*/
 
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_REMOVE) {
+       if (priv->list_features & EMPATHY_CONTACT_LIST_FEATURE_GROUPS_REMOVE) {
                item = gtk_image_menu_item_new_with_mnemonic (_("_Remove"));
                image = gtk_image_new_from_icon_name (GTK_STOCK_REMOVE,
                                                      GTK_ICON_SIZE_MENU);
@@ -1215,6 +1295,8 @@ empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
                                  view);
        }
 
+       g_free (group);
+
        return menu;
 }
 
@@ -1232,7 +1314,7 @@ contact_list_view_remove_activate_cb (GtkMenuItem            *menuitem,
                GtkWindow *parent;
 
                parent = empathy_get_toplevel_window (GTK_WIDGET (view));
-               text = g_strdup_printf (_("Do you really want to remove the contact '%s' ?"),
+               text = g_strdup_printf (_("Do you really want to remove the contact '%s'?"),
                                        empathy_contact_get_name (contact));                                            
                if (contact_list_view_remove_dialog_show (parent, _("Removing contact"), text)) {
                        EmpathyContactList *list;
@@ -1248,89 +1330,52 @@ contact_list_view_remove_activate_cb (GtkMenuItem            *menuitem,
 }
 
 GtkWidget *
-empathy_contact_list_view_get_contact_menu (EmpathyContactListView *view,
-                                           EmpathyContact         *contact)
+empathy_contact_list_view_get_contact_menu (EmpathyContactListView *view)
 {
        EmpathyContactListViewPriv *priv = GET_PRIV (view);
+       EmpathyContact             *contact;
        GtkWidget                  *menu;
-       GtkMenuShell               *shell;
        GtkWidget                  *item;
        GtkWidget                  *image;
 
        g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view), NULL);
-       g_return_val_if_fail (EMPATHY_IS_CONTACT (contact), NULL);
-
-       if (!(priv->features & (EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CHAT |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CALL |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_LOG |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_FT |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_INVITE |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_EDIT |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_INFO |
-                               EMPATHY_CONTACT_LIST_FEATURE_CONTACT_REMOVE))) {
-               return NULL;
-       }
 
-       menu = gtk_menu_new ();
-       shell = GTK_MENU_SHELL (menu);
-
-       /* Main items */
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CHAT) {
-               item = empathy_contact_chat_menu_item_new (contact);
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
-       }
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CALL) {
-               item = empathy_contact_call_menu_item_new (contact);
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
-       }
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_LOG) {
-               item = empathy_contact_log_menu_item_new (contact);
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
+       contact = empathy_contact_list_view_get_selected (view);
+       if (!contact) {
+               return NULL;
        }
 
-       /* Separator */
-       if (priv->features & (EMPATHY_CONTACT_LIST_FEATURE_CONTACT_EDIT |
-                             EMPATHY_CONTACT_LIST_FEATURE_CONTACT_INFO)) {
-               item = gtk_separator_menu_item_new ();
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
-       }
+       menu = empathy_contact_menu_new (contact, priv->contact_features);
 
-       /* More items */
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_EDIT) {
-               item = empathy_contact_edit_menu_item_new (contact);
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
-       }
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_INFO) {
-               item = empathy_contact_info_menu_item_new (contact);
-               gtk_menu_shell_append (shell, item);
-               gtk_widget_show (item);
+       if (!(priv->list_features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_REMOVE)) {
+               g_object_unref (contact);
+               return menu;
        }
 
-       /* Separator */
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_REMOVE) {
+       if (menu) {
+               /* Separator */
                item = gtk_separator_menu_item_new ();
-               gtk_menu_shell_append (shell, item);
+               gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
                gtk_widget_show (item);
+       } else {
+               menu = gtk_menu_new ();
        }
 
-       /* Custom items */
-       if (priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_REMOVE) {
+       /* Remove contact */
+       if (priv->list_features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_REMOVE) {
                item = gtk_image_menu_item_new_with_mnemonic (_("_Remove"));
                image = gtk_image_new_from_icon_name (GTK_STOCK_REMOVE,
                                                      GTK_ICON_SIZE_MENU);
                gtk_image_menu_item_set_image (GTK_IMAGE_MENU_ITEM (item), image);
-               gtk_menu_shell_append (shell, item);
+               gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
                gtk_widget_show (item);
                g_signal_connect (item, "activate",
                                  G_CALLBACK (contact_list_view_remove_activate_cb),
                                  view);
        }
 
+       g_object_unref (contact);
+
        return menu;
 }