fix bug 4239, 'Preferences: Text Options Header Display modal is not modal' (sic)
[claws.git] / src / importldif.c
index e06364ebcab70bb5d843d5066d04a8e319611e10..5b523f425ddc3aab8bc7a6f813e4f9df6de81773 100644 (file)
@@ -1,6 +1,6 @@
 /*
- * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 2001-2007 Match Grun and the Claws Mail team
+ * Claws Mail -- a GTK+ based, lightweight, and fast e-mail client
+ * Copyright (C) 2001-2015 Match Grun and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -14,7 +14,6 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
- * 
  */
 
 /*
@@ -23,6 +22,7 @@
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #include "defs.h"
 #define IMPORTLDIF_WIDTH           390
 #define IMPORTLDIF_HEIGHT          300
 
-#define FIELDS_N_COLS              4
 #define FIELDS_COL_WIDTH_RESERVED  10
 #define FIELDS_COL_WIDTH_SELECT    10
 #define FIELDS_COL_WIDTH_FIELD     140
 #define FIELDS_COL_WIDTH_ATTRIB    140
 
 typedef enum {
-       FIELD_COL_RESERVED = 0,
-       FIELD_COL_SELECT   = 1,
-       FIELD_COL_FIELD    = 2,
-       FIELD_COL_ATTRIB   = 3
+       FIELD_COL_RESERVED,
+       FIELD_COL_SELECT,
+       FIELD_COL_FIELD,
+       FIELD_COL_ATTRIB,
+       FIELD_COL_PTR,
+       FIELDS_N_COLS
 } ImpLdif_FieldColPos;
 
 /**
@@ -74,7 +75,7 @@ static struct _ImpLdif_Dlg {
        GtkWidget *notebook;
        GtkWidget *entryFile;
        GtkWidget *entryName;
-       GtkWidget *clist_field;
+       GtkWidget *view_fields;
        GtkWidget *entryField;
        GtkWidget *entryAttrib;
        GtkWidget *checkSelect;
@@ -88,7 +89,6 @@ static struct _ImpLdif_Dlg {
        GtkWidget *btnCancel;
        GtkWidget *statusbar;
        gint      status_cid;
-       gint      rowIndSelect;
        gint      rowCount;
        gchar     *nameBook;
        gchar     *fileName;
@@ -100,8 +100,7 @@ static AddressBookFile *_importedBook_;
 static AddressIndex *_imp_addressIndex_;
 static LdifFile *_ldifFile_ = NULL;
 
-static GdkPixmap *markxpm;
-static GdkBitmap *markxpmmask;
+static GdkPixbuf *markxpm;
 
 /**
  * Structure of error message table.
@@ -195,42 +194,21 @@ static void imp_ldif_message( void ) {
 }
 
 /**
- * Update list with data for current row.
- * \param clist List to update.
+ * Update the line (represented by the GtkTreeIter) with data
+ * from the Ldif_FieldRec.
  */
-static void imp_ldif_update_row( GtkCMCList *clist ) {
-       Ldif_FieldRec *rec;
-       gchar *text[ FIELDS_N_COLS ];
-       gint row;
-
-       if( impldif_dlg.rowIndSelect < 0 ) return;
-       row = impldif_dlg.rowIndSelect;
-
-       rec = gtk_cmclist_get_row_data( clist, row );
-       text[ FIELD_COL_RESERVED ] = "";
-       text[ FIELD_COL_SELECT   ] = "";
-       text[ FIELD_COL_FIELD    ] = rec->tagName;
-       text[ FIELD_COL_ATTRIB   ] = rec->userName;
-
-       gtk_cmclist_freeze( clist );
-       gtk_cmclist_remove( clist, row );
-       if( row == impldif_dlg.rowCount - 1 ) {
-               gtk_cmclist_append( clist, text );
-       }
-       else {
-               gtk_cmclist_insert( clist, row, text );
-       }
-       if( rec->selected ) {
-               gtk_cmclist_set_pixmap(
-                       clist, row, FIELD_COL_SELECT, markxpm, markxpmmask );
-       }
-       if( rec->reserved ) {
-               gtk_cmclist_set_pixmap(
-                       clist, row, FIELD_COL_RESERVED, markxpm, markxpmmask );
-       }
-
-       gtk_cmclist_set_row_data( clist, row, rec );
-       gtk_cmclist_thaw( clist );
+static void _populate_iter(GtkListStore *store, GtkTreeIter *iter,
+               Ldif_FieldRec *rec)
+{
+       gtk_list_store_set(store, iter,
+                       FIELD_COL_FIELD, rec->tagName,
+                       FIELD_COL_ATTRIB, rec->userName,
+                       FIELD_COL_PTR, rec,
+                       -1);
+       gtk_list_store_set(store, iter,
+                       FIELD_COL_SELECT, rec->selected ? markxpm : NULL, -1);
+       gtk_list_store_set(store, iter,
+                       FIELD_COL_RESERVED, rec->reserved ? markxpm : NULL, -1);
 }
 
 /**
@@ -238,34 +216,24 @@ static void imp_ldif_update_row( GtkCMCList *clist ) {
  * \param ldf LDIF control data.
  */
 static void imp_ldif_load_fields( LdifFile *ldf ) {
-       GtkCMCList *clist = GTK_CMCLIST(impldif_dlg.clist_field);
+       GtkWidget *view = impldif_dlg.view_fields;
+       GtkTreeModel *model = gtk_tree_view_get_model(GTK_TREE_VIEW(view));
+       GtkTreeIter iter;
        GList *node, *list;
-       gchar *text[ FIELDS_N_COLS ];
 
-       impldif_dlg.rowIndSelect = -1;
        impldif_dlg.rowCount = 0;
+
        if( ! ldf->accessFlag ) return;
-       gtk_cmclist_clear( clist );
+
+       gtk_list_store_clear(GTK_LIST_STORE(model));
+
        list = ldif_get_fieldlist( ldf );
        node = list;
        while( node ) {
                Ldif_FieldRec *rec = node->data;
-               gint row;
-
-               text[ FIELD_COL_RESERVED ] = "";
-               text[ FIELD_COL_SELECT   ] = "";
-               text[ FIELD_COL_FIELD    ] = rec->tagName;
-               text[ FIELD_COL_ATTRIB   ] = rec->userName;
-               row = gtk_cmclist_append( clist, text );
-               gtk_cmclist_set_row_data( clist, row, rec );
-               if( rec->selected ) {
-                       gtk_cmclist_set_pixmap( clist, row,
-                               FIELD_COL_SELECT, markxpm, markxpmmask );
-               }
-               if( rec->reserved ) {
-                       gtk_cmclist_set_pixmap( clist, row,
-                               FIELD_COL_RESERVED, markxpm, markxpmmask );
-               }
+
+               gtk_list_store_append(GTK_LIST_STORE(model), &iter);
+               _populate_iter(GTK_LIST_STORE(model), &iter, rec);
                impldif_dlg.rowCount++;
                node = g_list_next( node );
        }
@@ -274,117 +242,6 @@ static void imp_ldif_load_fields( LdifFile *ldf ) {
        ldif_set_accessed( ldf, FALSE );
 }
 
-/**
- * Callback function when list item is selected.
- * \param clist List widget.
- * \param row   Row.
- * \param col   Column.
- * \param event Event object.
- * \param data  User data.
- */
-static void imp_ldif_field_list_selected(
-               GtkCMCList *clist, gint row, gint column, GdkEvent *event,
-               gpointer data )
-{
-       Ldif_FieldRec *rec = gtk_cmclist_get_row_data( clist, row );
-
-       impldif_dlg.rowIndSelect = row;
-       gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryAttrib), "" );
-       if( rec ) {
-               /* Update widget contents */
-               gtk_label_set_text(
-                       GTK_LABEL(impldif_dlg.entryField), rec->tagName );
-               if( rec->userName )
-                       gtk_entry_set_text(
-                               GTK_ENTRY(impldif_dlg.entryAttrib), rec->userName );
-               gtk_toggle_button_set_active(
-                       GTK_TOGGLE_BUTTON( impldif_dlg.checkSelect),
-                       rec->selected );
-
-               /* Disable widgets for reserved fields */
-               gtk_widget_set_sensitive(
-                       impldif_dlg.entryAttrib, ! rec->reserved );
-               gtk_widget_set_sensitive(
-                       impldif_dlg.checkSelect, ! rec->reserved );
-               gtk_widget_set_sensitive(
-                       impldif_dlg.btnModify,   ! rec->reserved );
-       }
-       gtk_widget_grab_focus(impldif_dlg.entryAttrib);
-}
-
-/**
- * Callback function to toggle selected LDIF field.
- * \param clist List to update.
- * \param event Event object.
- * \param data  Data.
- */
-static gboolean imp_ldif_field_list_toggle(
-               GtkCMCList *clist, GdkEventButton *event, gpointer data )
-{
-       Ldif_FieldRec *rec;
-       gboolean toggle = FALSE;
-
-       if( ! event ) return FALSE;
-       if( impldif_dlg.rowIndSelect < 0 ) return FALSE;
-       if( event->button == 1 ) {
-               /* If single click in select column */
-               if( event->type == GDK_BUTTON_PRESS ) {
-                       gint x = event->x;
-                       gint y = event->y;
-                       gint row, col;
-
-                       gtk_cmclist_get_selection_info( clist, x, y, &row, &col );
-                       if( col != FIELD_COL_SELECT ) return FALSE;
-                       if( row > impldif_dlg.rowCount ) return FALSE;
-
-                       /* Set row */
-                       impldif_dlg.rowIndSelect = row;
-                       toggle = TRUE;
-               }
-
-               /* If double click anywhere in row */
-               else if( event->type == GDK_2BUTTON_PRESS ) {
-                       toggle = TRUE;
-               }
-       }
-       /* Toggle field selection */
-       if( toggle ) {
-               rec = gtk_cmclist_get_row_data(
-                       clist, impldif_dlg.rowIndSelect );
-               if( rec ) {
-                       ldif_field_toggle( rec );
-                       imp_ldif_update_row( clist );
-               }
-       }
-       return FALSE;
-}
-
-/**
- * Callback function to update LDIF field data from input fields.
- * \param widget Widget (button).
- * \param data   User data.
- */
-static void imp_ldif_modify_pressed( GtkWidget *widget, gpointer data ) {
-       GtkCMCList *clist = GTK_CMCLIST(impldif_dlg.clist_field);
-       Ldif_FieldRec *rec;
-       gint row;
-
-       if( impldif_dlg.rowIndSelect < 0 ) return;
-       row = impldif_dlg.rowIndSelect;
-       rec = gtk_cmclist_get_row_data( clist, impldif_dlg.rowIndSelect );
-
-       ldif_field_set_name( rec, gtk_editable_get_chars(
-               GTK_EDITABLE(impldif_dlg.entryAttrib), 0, -1 ) );
-       ldif_field_set_selected( rec, gtk_toggle_button_get_active(
-               GTK_TOGGLE_BUTTON( impldif_dlg.checkSelect) ) );
-       imp_ldif_update_row( clist );
-       gtk_cmclist_select_row( clist, row, 0 );
-       gtk_label_set_text( GTK_LABEL(impldif_dlg.entryField), "" );
-       gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryAttrib), "" );
-       gtk_toggle_button_set_active(
-               GTK_TOGGLE_BUTTON( impldif_dlg.checkSelect), FALSE );
-}
-
 /**
  * Test whether we can move off fields page.
  * \return <i>TRUE</i> if OK to move off page.
@@ -418,6 +275,36 @@ static gboolean imp_ldif_field_move() {
        return retVal;
 }
 
+static void _update_selected_row()
+{
+       GtkTreeModel *model;
+       GtkTreeIter iter;
+       Ldif_FieldRec *rec;
+
+       rec = gtkut_tree_view_get_selected_pointer(
+                       GTK_TREE_VIEW(impldif_dlg.view_fields), FIELD_COL_PTR,
+                       &model, NULL, &iter);
+       if (rec == NULL)
+               return;
+
+       ldif_field_set_name(rec, gtk_entry_get_text(
+                               GTK_ENTRY(impldif_dlg.entryAttrib)));
+       ldif_field_set_selected(rec, gtk_toggle_button_get_active(
+                               GTK_TOGGLE_BUTTON(impldif_dlg.checkSelect)));
+
+       _populate_iter(GTK_LIST_STORE(model), &iter, rec);
+}
+
+static void imp_ldif_modify_pressed(GtkButton *widget, gpointer user_data)
+{
+       _update_selected_row();
+}
+
+static void imp_ldif_entryattrib_activate(GtkEntry *entry, gpointer user_data)
+{
+       _update_selected_row();
+}
+
 /**
  * Test whether we can move off file page.
  * \return <i>TRUE</i> if OK to move off page.
@@ -430,10 +317,10 @@ static gboolean imp_ldif_file_move() {
        gboolean errFlag = FALSE;
 
        sFile = gtk_editable_get_chars( GTK_EDITABLE(impldif_dlg.entryFile), 0, -1 );
-       g_strchug( sFile ); g_strchomp( sFile );
+       g_strstrip( sFile );
 
        sName = gtk_editable_get_chars( GTK_EDITABLE(impldif_dlg.entryName), 0, -1 );
-       g_strchug( sName ); g_strchomp( sName );
+       g_strstrip( sName );
 
        g_free( impldif_dlg.nameBook );
        g_free( impldif_dlg.fileName );
@@ -611,7 +498,7 @@ static gint imp_ldif_delete_event( GtkWidget *widget, GdkEventAny *event, gpoint
  * \param data   User data.
  */
 static gboolean imp_ldif_key_pressed( GtkWidget *widget, GdkEventKey *event, gpointer data ) {
-       if (event && event->keyval == GDK_Escape) {
+       if (event && event->keyval == GDK_KEY_Escape) {
                imp_ldif_cancel( widget, data );
        }
        return FALSE;
@@ -629,7 +516,6 @@ static void imp_ldif_page_file( gint pageNum, gchar *pageLbl ) {
        GtkWidget *entryFile;
        GtkWidget *entryName;
        GtkWidget *btnFile;
-       CLAWS_TIP_DECL();
        gint top;
 
        vbox = gtk_vbox_new(FALSE, 8);
@@ -697,6 +583,66 @@ static void imp_ldif_page_file( gint pageNum, gchar *pageLbl ) {
 
 }
 
+static void imp_ldif_field_list_cursor_changed(GtkTreeView *view,
+               gpointer user_data)
+{
+       Ldif_FieldRec *rec;
+
+       gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryAttrib), "" );
+
+       rec = gtkut_tree_view_get_selected_pointer(view, FIELD_COL_PTR,
+                       NULL, NULL, NULL);
+
+       if( rec != NULL) {
+               /* Update widget contents */
+               gtk_label_set_text(
+                       GTK_LABEL(impldif_dlg.entryField), rec->tagName );
+               if( rec->userName )
+                       gtk_entry_set_text(
+                               GTK_ENTRY(impldif_dlg.entryAttrib), rec->userName );
+               gtk_toggle_button_set_active(
+                       GTK_TOGGLE_BUTTON(impldif_dlg.checkSelect),
+                       rec->selected );
+
+               /* Disable widgets for reserved fields */
+               gtk_widget_set_sensitive(
+                       impldif_dlg.entryAttrib, ! rec->reserved );
+               gtk_widget_set_sensitive(
+                       impldif_dlg.checkSelect, ! rec->reserved );
+               gtk_widget_set_sensitive(
+                       impldif_dlg.btnModify,   ! rec->reserved );
+       }
+       gtk_widget_grab_focus(impldif_dlg.entryAttrib);
+}
+
+static void imp_ldif_field_list_row_activated(GtkTreeView *view,
+               GtkTreePath *path, GtkTreeViewColumn *col,
+               gpointer user_data)
+{
+       GtkTreeModel *model = gtk_tree_view_get_model(view);
+       GtkTreeIter iter;
+       gboolean ok;
+       Ldif_FieldRec *rec;
+
+       ok = gtk_tree_model_get_iter(model, &iter, path);
+       if (!ok) {
+               return; /* Huh? */
+       }
+
+       gtk_tree_model_get(model, &iter, FIELD_COL_PTR, &rec, -1);
+       cm_return_if_fail(rec != NULL);
+
+       /* Flip the "selected" state for the record, and update the
+        * "selected" column in the list view, as well as the
+        * "selected" checkbox. */
+       ldif_field_toggle(rec);
+       gtk_list_store_set(GTK_LIST_STORE(model), &iter,
+                       FIELD_COL_SELECT, rec->selected ? markxpm : NULL, -1);
+       gtk_toggle_button_set_active(
+               GTK_TOGGLE_BUTTON(impldif_dlg.checkSelect),
+               rec->selected );
+}
+
 /**
  * Format notebook fields page.
  * \param pageNum Page (tab) number.
@@ -708,23 +654,18 @@ static void imp_ldif_page_fields( gint pageNum, gchar *pageLbl ) {
        GtkWidget *vboxb;
        GtkWidget *table;
        GtkWidget *label;
-       GtkWidget *clist_swin;
-       GtkWidget *clist_field;
+       GtkWidget *scrollwin;
+       GtkWidget *view_fields;
        GtkWidget *entryField;
        GtkWidget *entryAttrib;
        GtkWidget *checkSelect;
        GtkWidget *btnModify;
        GtkWidget *eventBox;
-       CLAWS_TIP_DECL();
        gint top;
-
-       gchar *titles[ FIELDS_N_COLS ];
-       gint i;
-
-       titles[ FIELD_COL_RESERVED ] = _("R");
-       titles[ FIELD_COL_SELECT   ] = _("S");
-       titles[ FIELD_COL_FIELD    ] = _("LDIF Field Name");
-       titles[ FIELD_COL_ATTRIB   ] = _("Attribute Name");
+       GtkListStore *store;
+       GtkCellRenderer *rdr;
+       GtkTreeViewColumn *col;
+       GtkTreeSelection *sel;
 
        vbox = gtk_vbox_new(FALSE, 8);
        gtk_container_add( GTK_CONTAINER( impldif_dlg.notebook ), vbox );
@@ -741,31 +682,48 @@ static void imp_ldif_page_fields( gint pageNum, gchar *pageLbl ) {
        vboxt = gtk_vbox_new( FALSE, 4 );
        gtk_container_add( GTK_CONTAINER( vbox ), vboxt );
 
-       clist_swin = gtk_scrolled_window_new( NULL, NULL );
-       gtk_container_add( GTK_CONTAINER(vboxt), clist_swin );
-       gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(clist_swin),
+       scrollwin = gtk_scrolled_window_new( NULL, NULL );
+       gtk_container_add( GTK_CONTAINER(vboxt), scrollwin );
+       gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrollwin),
                                       GTK_POLICY_AUTOMATIC,
                                       GTK_POLICY_AUTOMATIC);
 
-       clist_field = gtk_cmclist_new_with_titles( FIELDS_N_COLS, titles );
-       gtk_container_add( GTK_CONTAINER(clist_swin), clist_field );
-       gtk_cmclist_set_selection_mode(
-               GTK_CMCLIST(clist_field), GTK_SELECTION_BROWSE );
-       gtk_cmclist_set_column_width( GTK_CMCLIST(clist_field),
-               FIELD_COL_RESERVED, FIELDS_COL_WIDTH_RESERVED );
-       gtk_cmclist_set_column_width( GTK_CMCLIST(clist_field),
-               FIELD_COL_SELECT, FIELDS_COL_WIDTH_SELECT );
-       gtk_cmclist_set_column_width( GTK_CMCLIST(clist_field),
-               FIELD_COL_FIELD, FIELDS_COL_WIDTH_FIELD );
-       gtk_cmclist_set_column_width( GTK_CMCLIST(clist_field),
-               FIELD_COL_ATTRIB, FIELDS_COL_WIDTH_ATTRIB );
-
-       /* Remove focus capability for column headers */
-       for( i = 0; i < FIELDS_N_COLS; i++ ) {
-               GTK_WIDGET_UNSET_FLAGS(
-                       GTK_CMCLIST(clist_field)->column[i].button,
-                       GTK_CAN_FOCUS);
-       }
+       store = gtk_list_store_new(FIELDS_N_COLS,
+                       GDK_TYPE_PIXBUF, GDK_TYPE_PIXBUF,
+                       G_TYPE_STRING, G_TYPE_STRING,
+                       G_TYPE_POINTER,
+                       -1);
+
+       view_fields = gtk_tree_view_new_with_model(GTK_TREE_MODEL(store));
+       g_object_unref(store);
+       gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(view_fields), TRUE);
+       gtk_tree_view_set_reorderable(GTK_TREE_VIEW(view_fields), FALSE);
+       sel = gtk_tree_view_get_selection(GTK_TREE_VIEW(view_fields));
+       gtk_tree_selection_set_mode(sel, GTK_SELECTION_BROWSE);
+
+       rdr = gtk_cell_renderer_pixbuf_new();
+       col = gtk_tree_view_column_new_with_attributes(_("R"), rdr,
+                       "pixbuf", FIELD_COL_RESERVED, NULL);
+       gtk_tree_view_column_set_min_width(col, FIELD_COL_RESERVED);
+       gtk_tree_view_append_column(GTK_TREE_VIEW(view_fields), col);
+
+       col = gtk_tree_view_column_new_with_attributes(_("S"), rdr,
+                       "pixbuf", FIELD_COL_SELECT, NULL);
+       gtk_tree_view_column_set_min_width(col, FIELD_COL_SELECT);
+       gtk_tree_view_append_column(GTK_TREE_VIEW(view_fields), col);
+
+       rdr = gtk_cell_renderer_text_new();
+       col = gtk_tree_view_column_new_with_attributes(_("LDIF Field Name"), rdr,
+                       "markup", FIELD_COL_FIELD, NULL);
+       gtk_tree_view_column_set_min_width(col, FIELD_COL_FIELD);
+       gtk_tree_view_append_column(GTK_TREE_VIEW(view_fields), col);
+
+       col = gtk_tree_view_column_new_with_attributes(_("Attribute Name"), rdr,
+                       "markup", FIELD_COL_ATTRIB, NULL);
+       gtk_tree_view_column_set_min_width(col, FIELD_COL_ATTRIB);
+       gtk_tree_view_append_column(GTK_TREE_VIEW(view_fields), col);
+
+       gtk_container_add( GTK_CONTAINER(scrollwin), view_fields );
 
        /* Lower area - Edit area */
        vboxb = gtk_vbox_new( FALSE, 4 );
@@ -840,14 +798,16 @@ static void imp_ldif_page_fields( gint pageNum, gchar *pageLbl ) {
        gtk_widget_show_all(vbox);
 
        /* Event handlers */
-       g_signal_connect( G_OBJECT(clist_field), "select_row",
-                         G_CALLBACK(imp_ldif_field_list_selected), NULL );
-       g_signal_connect( G_OBJECT(clist_field), "button_press_event",
-                         G_CALLBACK(imp_ldif_field_list_toggle), NULL );
-       g_signal_connect( G_OBJECT(btnModify), "clicked",
-                         G_CALLBACK(imp_ldif_modify_pressed), NULL );
-
-       impldif_dlg.clist_field = clist_field;
+       g_signal_connect(G_OBJECT(view_fields), "cursor-changed",
+                       G_CALLBACK(imp_ldif_field_list_cursor_changed), NULL);
+       g_signal_connect(G_OBJECT(view_fields), "row-activated",
+                       G_CALLBACK(imp_ldif_field_list_row_activated), NULL);
+       g_signal_connect(G_OBJECT(btnModify), "clicked",
+                       G_CALLBACK(imp_ldif_modify_pressed), NULL );
+       g_signal_connect(G_OBJECT(entryAttrib), "activate",
+                       G_CALLBACK(imp_ldif_entryattrib_activate), NULL);
+
+       impldif_dlg.view_fields = view_fields;
        impldif_dlg.entryField  = entryField;
        impldif_dlg.entryAttrib = entryAttrib;
        impldif_dlg.checkSelect = checkSelect;
@@ -887,7 +847,7 @@ static void imp_ldif_page_finish( gint pageNum, gchar *pageLbl ) {
 
        /* First row */
        top = 0;
-       label = gtk_label_new( _( "Address Book :" ) );
+       label = gtk_label_new( _( "Address Book:" ) );
        gtk_table_attach(GTK_TABLE(table), label, 0, 1, top, (top + 1), GTK_FILL, 0, 0, 0);
        gtk_misc_set_alignment(GTK_MISC(label), 1, 0.5);
 
@@ -897,7 +857,7 @@ static void imp_ldif_page_finish( gint pageNum, gchar *pageLbl ) {
 
        /* Second row */
        top++;
-       label = gtk_label_new( _( "File Name :" ) );
+       label = gtk_label_new( _( "File Name:" ) );
        gtk_table_attach(GTK_TABLE(table), label, 0, 1, top, (top + 1), GTK_FILL, 0, 0, 0);
        gtk_misc_set_alignment(GTK_MISC(label), 1, 0.5);
 
@@ -907,7 +867,7 @@ static void imp_ldif_page_finish( gint pageNum, gchar *pageLbl ) {
 
        /* Third row */
        top++;
-       label = gtk_label_new( _("Records Imported :") );
+       label = gtk_label_new( _("Records Imported:") );
        gtk_table_attach(GTK_TABLE(table), label, 0, 1, top, (top + 1), GTK_FILL, 0, 0, 0);
        gtk_misc_set_alignment(GTK_MISC(label), 1, 0.5);
 
@@ -941,7 +901,7 @@ static void imp_ldif_dialog_create() {
        gtk_container_set_border_width( GTK_CONTAINER(window), 0 );
        gtk_window_set_title( GTK_WINDOW(window), _("Import LDIF file into Address Book") );
        gtk_window_set_position(GTK_WINDOW(window), GTK_WIN_POS_CENTER);
-       gtk_window_set_modal(GTK_WINDOW(window), TRUE); 
+       gtk_window_set_type_hint(GTK_WINDOW(window), GDK_WINDOW_TYPE_HINT_DIALOG);
        g_signal_connect(G_OBJECT(window), "delete_event",
                         G_CALLBACK(imp_ldif_delete_event),
                         NULL );
@@ -980,7 +940,7 @@ static void imp_ldif_dialog_create() {
        btnProceed = gtk_button_new_with_mnemonic(_("Proceed"));
        gtk_button_set_image(GTK_BUTTON(btnProceed),
                        gtk_image_new_from_stock(GTK_STOCK_OK, GTK_ICON_SIZE_BUTTON));
-       GTK_WIDGET_SET_FLAGS(btnProceed, GTK_CAN_DEFAULT);
+       gtk_widget_set_can_default(btnProceed, TRUE);
        gtk_box_pack_start(GTK_BOX(hbbox), btnProceed, TRUE, TRUE, 0);
        gtk_widget_hide(btnProceed);
 
@@ -1030,12 +990,18 @@ static void imp_ldif_create() {
  *         was cancelled.
  */
 AddressBookFile *addressbook_imp_ldif( AddressIndex *addrIndex ) {
+       GtkWidget *view;
+       GtkTreeModel *model;
+
        _importedBook_ = NULL;
        _imp_addressIndex_ = addrIndex;
 
        if( ! impldif_dlg.window )
                imp_ldif_create();
 
+       view = impldif_dlg.view_fields;
+       model = gtk_tree_view_get_model(GTK_TREE_VIEW(view));
+
        gtk_button_set_label(GTK_BUTTON(impldif_dlg.btnCancel),
                             GTK_STOCK_CANCEL);
        gtk_widget_hide(impldif_dlg.btnProceed);
@@ -1045,21 +1011,20 @@ AddressBookFile *addressbook_imp_ldif( AddressIndex *addrIndex ) {
        gtk_widget_show(impldif_dlg.window);
        manage_window_set_transient(GTK_WINDOW(impldif_dlg.window));
        gtk_widget_grab_default(impldif_dlg.btnNext);
+       gtk_window_set_modal(GTK_WINDOW(impldif_dlg.window), TRUE);
 
        gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryName), IMPORTLDIF_GUESS_NAME );
        gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryFile), "" );
        gtk_label_set_text( GTK_LABEL(impldif_dlg.entryField), "" );
        gtk_entry_set_text( GTK_ENTRY(impldif_dlg.entryAttrib), "" );
-       gtk_cmclist_clear( GTK_CMCLIST(impldif_dlg.clist_field) );
+       gtk_list_store_clear( GTK_LIST_STORE(model) );
        gtk_notebook_set_current_page( GTK_NOTEBOOK(impldif_dlg.notebook), PAGE_FILE_INFO );
        gtk_widget_set_sensitive( impldif_dlg.btnPrev, FALSE );
        gtk_widget_set_sensitive( impldif_dlg.btnNext, TRUE );
-       stock_pixmap_gdk( impldif_dlg.window, STOCK_PIXMAP_MARK,
-                         &markxpm, &markxpmmask );
+       stock_pixbuf_gdk(STOCK_PIXMAP_MARK, &markxpm );
        imp_ldif_message();
        gtk_widget_grab_focus(impldif_dlg.entryFile);
 
-       impldif_dlg.rowIndSelect = -1;
        impldif_dlg.rowCount = 0;
        g_free( impldif_dlg.nameBook );
        g_free( impldif_dlg.fileName );
@@ -1069,6 +1034,7 @@ AddressBookFile *addressbook_imp_ldif( AddressIndex *addrIndex ) {
        _ldifFile_ = ldif_create();
        gtk_main();
        gtk_widget_hide(impldif_dlg.window);
+       gtk_window_set_modal(GTK_WINDOW(impldif_dlg.window), FALSE);
        ldif_free( _ldifFile_ );
        _ldifFile_ = NULL;
        _imp_addressIndex_ = NULL;