]> git.0d.be Git - empathy.git/blobdiff - libempathy-gtk/empathy-contact-list-view.c
Remove check on GTK version, we depend on >= 2.14.0 now.
[empathy.git] / libempathy-gtk / empathy-contact-list-view.c
index b0bfdf909e70ef42cec60a5eacb31e62f8cdcc88..6cb8b7e8c51ecbbcebd92d45f96ba40a7d8ced56 100644 (file)
 
 #include <string.h>
 
-#include <glib/gi18n.h>
+#include <glib/gi18n-lib.h>
+#include <gdk/gdkkeysyms.h>
 #include <gtk/gtk.h>
 #include <glade/glade.h>
 
 #include <libmissioncontrol/mc-account.h>
 
+#include <libempathy/empathy-call-factory.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 +81,9 @@ typedef struct {
 
 enum {
        PROP_0,
-       PROP_FEATURES
+       PROP_STORE,
+       PROP_LIST_FEATURES,
+       PROP_CONTACT_FEATURES,
 };
 
 enum DndDragType {
@@ -113,6 +115,79 @@ static guint signals[LAST_SIGNAL];
 
 G_DEFINE_TYPE (EmpathyContactListView, empathy_contact_list_view, GTK_TYPE_TREE_VIEW);
 
+static void
+contact_list_view_tooltip_destroy_cb (GtkWidget              *widget,
+                                     EmpathyContactListView *view)
+{
+       EmpathyContactListViewPriv *priv = GET_PRIV (view);
+       
+       if (priv->tooltip_widget) {
+               DEBUG ("Tooltip destroyed");
+               priv->tooltip_widget = NULL;
+               g_object_unref (widget);
+       }
+}
+
+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;
+       static gint                 running = 0;
+       gboolean                    ret = FALSE;
+
+       /* Avoid an infinite loop. See GNOME bug #574377 */
+       if (running > 0) {
+               return FALSE;
+       }
+       running++;
+
+       if (!gtk_tree_view_get_tooltip_context (GTK_TREE_VIEW (view), &x, &y,
+                                               keyboard_mode,
+                                               &model, &path, &iter)) {
+               goto OUT;
+       }
+
+       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) {
+               goto OUT;
+       }
+
+       if (!priv->tooltip_widget) {
+               priv->tooltip_widget = empathy_contact_widget_new (contact,
+                       EMPATHY_CONTACT_WIDGET_FOR_TOOLTIP);
+               g_object_ref (priv->tooltip_widget);
+               g_signal_connect (priv->tooltip_widget, "destroy",
+                                 G_CALLBACK (contact_list_view_tooltip_destroy_cb),
+                                 view);
+       } else {
+               empathy_contact_widget_set_contact (priv->tooltip_widget,
+                                                   contact);
+       }
+
+       gtk_tooltip_set_custom (tooltip, priv->tooltip_widget);
+       ret = TRUE;
+
+       g_object_unref (contact);
+OUT:
+       running--;
+
+       return ret;
+}
+
 static void
 contact_list_view_drag_data_received (GtkWidget         *widget,
                                      GdkDragContext    *context,
@@ -139,13 +214,13 @@ 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 ();
+       factory = empathy_contact_factory_dup_singleton ();
        account = mc_account_lookup (strv[0]);
        if (account) {
                contact = empathy_contact_factory_get_from_id (factory,
@@ -157,7 +232,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 +263,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,93 +462,94 @@ 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);
+       g_slice_free (MenuPopupData, data);
 
-       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);
-
-       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
-contact_list_view_row_activated_cb (EmpathyContactListView *view,
-                                   GtkTreePath            *path,
-                                   GtkTreeViewColumn      *col,
-                                   gpointer                user_data)
+contact_list_view_row_activated (GtkTreeView       *view,
+                                GtkTreePath       *path,
+                                GtkTreeViewColumn *column)
 {
        EmpathyContactListViewPriv *priv = GET_PRIV (view);
        EmpathyContact             *contact;
        GtkTreeModel               *model;
        GtkTreeIter                 iter;
 
-       if (!(priv->features & EMPATHY_CONTACT_LIST_FEATURE_CONTACT_CHAT)) {
+       if (!(priv->contact_features & EMPATHY_CONTACT_FEATURE_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);
+               DEBUG ("Starting a chat");
+               empathy_dispatcher_chat_with_contact (contact, NULL, NULL);
                g_object_unref (contact);
        }
 }
@@ -489,7 +564,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 +578,11 @@ contact_list_view_voip_activated_cb (EmpathyCellRendererActivatable *cell,
                            -1);
 
        if (contact) {
-               empathy_call_with_contact (contact);
+               EmpathyCallFactory *factory;
+
+               factory = empathy_call_factory_get ();
+               empathy_call_factory_new_call (factory, contact);
+
                g_object_unref (contact);
        }
 }
@@ -693,7 +772,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;
        }
 
@@ -724,12 +803,12 @@ contact_list_view_row_has_child_toggled_cb (GtkTreeModel          *model,
                            EMPATHY_CONTACT_LIST_STORE_COL_NAME, &name,
                            -1);
 
-       if (!is_group || G_STR_EMPTY (name)) {
+       if (!is_group || EMP_STR_EMPTY (name)) {
                g_free (name);
                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 +943,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)
 {
@@ -874,6 +992,9 @@ contact_list_view_finalize (GObject *object)
        if (priv->store) {
                g_object_unref (priv->store);
        }
+       if (priv->tooltip_widget) {
+               gtk_widget_destroy (priv->tooltip_widget);
+       }
 
        G_OBJECT_CLASS (empathy_contact_list_view_parent_class)->finalize (object);
 }
@@ -889,8 +1010,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 +1032,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);
@@ -922,8 +1054,9 @@ contact_list_view_set_property (GObject      *object,
 static void
 empathy_contact_list_view_class_init (EmpathyContactListViewClass *klass)
 {
-       GObjectClass   *object_class = G_OBJECT_CLASS (klass);
-       GtkWidgetClass *widget_class = GTK_WIDGET_CLASS (klass);
+       GObjectClass     *object_class = G_OBJECT_CLASS (klass);
+       GtkWidgetClass   *widget_class = GTK_WIDGET_CLASS (klass);
+       GtkTreeViewClass *tree_view_class = GTK_TREE_VIEW_CLASS (klass);
 
        object_class->finalize = contact_list_view_finalize;
        object_class->get_property = contact_list_view_get_property;
@@ -934,12 +1067,13 @@ empathy_contact_list_view_class_init (EmpathyContactListViewClass *klass)
        widget_class->drag_begin         = contact_list_view_drag_begin;
        widget_class->drag_data_get      = contact_list_view_drag_data_get;
        widget_class->drag_end           = contact_list_view_drag_end;
-       /* FIXME: noticed but when you drag the row over the treeview
-        * fast, it seems to stop redrawing itself, if we don't
-        * connect this signal, all is fine.
-        */
        widget_class->drag_motion        = contact_list_view_drag_motion;
 
+       /* We use the class method to let user of this widget to connect to
+        * the signal and stop emission of the signal so the default handler
+        * won't be called. */
+       tree_view_class->row_activated = contact_list_view_row_activated;
+
        signals[DRAG_CONTACT_RECEIVED] =
                g_signal_new ("drag-contact-received",
                              G_OBJECT_CLASS_TYPE (klass),
@@ -951,12 +1085,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 +1114,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 +1126,35 @@ 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_CALLBACK (contact_list_view_row_activated_cb),
+       g_signal_connect (view, "key-press-event",
+                         G_CALLBACK (contact_list_view_key_press_event_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 *
@@ -1116,31 +1216,22 @@ empathy_contact_list_view_get_selected_group (EmpathyContactListView *view)
 
 static gboolean
 contact_list_view_remove_dialog_show (GtkWindow   *parent, 
-                                     const gchar *window_title, 
-                                     const gchar *text)
+                                     const gchar *message, 
+                                     const gchar *secondary_text)
 {
-       GtkWidget *dialog, *label, *image, *hbox;
+       GtkWidget *dialog;
        gboolean res;
        
-       dialog = gtk_dialog_new_with_buttons (window_title, parent,
-                                             GTK_DIALOG_MODAL,
-                                             GTK_STOCK_DELETE, GTK_RESPONSE_YES,
-                                             GTK_STOCK_CANCEL, GTK_RESPONSE_NO,
-                                             NULL);
-       gtk_dialog_set_has_separator (GTK_DIALOG(dialog), FALSE);
-        
-       label = gtk_label_new (text);
-       image = gtk_image_new_from_stock (GTK_STOCK_DIALOG_QUESTION, GTK_ICON_SIZE_DIALOG);
+       dialog = gtk_message_dialog_new (parent, GTK_DIALOG_MODAL,
+                                        GTK_MESSAGE_QUESTION, GTK_BUTTONS_NONE,
+                                        "%s", message);
+       gtk_dialog_add_buttons (GTK_DIALOG (dialog),
+                               GTK_STOCK_CANCEL, GTK_RESPONSE_NO,
+                               GTK_STOCK_DELETE, GTK_RESPONSE_YES,
+                               NULL);
+       gtk_message_dialog_format_secondary_text (GTK_MESSAGE_DIALOG (dialog),
+                                                 "%s", secondary_text);
         
-       hbox = gtk_hbox_new (FALSE, 5);
-       gtk_container_set_border_width (GTK_CONTAINER (hbox), 5);
-       gtk_box_pack_start_defaults (GTK_BOX (hbox), image);
-       gtk_box_pack_start_defaults (GTK_BOX (hbox), label);     
-       gtk_box_pack_start_defaults (GTK_BOX(GTK_DIALOG(dialog)->vbox), hbox);
-
-       gtk_widget_show (image);
-       gtk_widget_show (label);
-       gtk_widget_show (hbox);
        gtk_widget_show (dialog);
         
        res = gtk_dialog_run (GTK_DIALOG (dialog));
@@ -1161,7 +1252,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 +1271,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 +1300,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 +1312,8 @@ empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
                                  view);
        }
 
+       g_free (group);
+
        return menu;
 }
 
@@ -1232,7 +1331,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 +1347,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;
 }