2007-10-10 [claws] 3.0.2cvs49
[claws.git] / src / textview.c
index 8c37ca89affd4a9962a8f00931941c6a2bacfa18..60169413f81fca250a84a9c4208d85832aa62c90 100644 (file)
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
+ * Copyright (C) 1999-2007 Hiroyuki Yamamoto 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
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,8 +13,8 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #ifdef HAVE_CONFIG_H
@@ -35,7 +35,9 @@
 #include <string.h>
 #include <stdlib.h>
 #include <errno.h>
+#if HAVE_SYS_WAIT_H
 #include <sys/wait.h>
+#endif
 #if HAVE_LIBCOMPFACE
 #  include <compface.h>
 #endif
 #include "base64.h"
 #include "inputdialog.h"
 #include "timing.h"
-
-struct _ClickableText
-{
-       gchar *uri;
-
-       gchar *filename;
-
-       gpointer data;
-
-       guint start;
-       guint end;
-       
-       gboolean is_quote;
-       gint quote_level;
-       gboolean q_expanded;
-       gchar *fg_color;
-};
-
-gint previousquotelevel = -1;
+#include "tags.h"
+#include "addrindex.h"
 
 static GdkColor quote_colors[3] = {
        {(gulong)0, (gushort)0, (gushort)0, (gushort)0},
@@ -115,7 +100,7 @@ static GdkColor emphasis_color = {
        (gulong)0,
        (gushort)0,
        (gushort)0,
-       (gushort)0xcfff
+       (gushort)0
 };
 
 static GdkCursor *hand_cursor = NULL;
@@ -149,7 +134,8 @@ static void textview_show_html              (TextView       *textview,
 
 static void textview_write_line                (TextView       *textview,
                                         const gchar    *str,
-                                        CodeConverter  *conv);
+                                        CodeConverter  *conv,
+                                        gboolean        do_quote_folding);
 static void textview_write_link                (TextView       *textview,
                                         const gchar    *str,
                                         const gchar    *uri,
@@ -194,11 +180,10 @@ static gboolean textview_uri_button_pressed       (GtkTextTag     *tag,
                                                 GtkTextIter    *iter,
                                                 TextView       *textview);
 
-static gboolean textview_uri_security_check    (TextView       *textview,
-                                                ClickableText  *uri);
 static void textview_uri_list_remove_all       (GSList         *uri_list);
 
 static void textview_toggle_quote              (TextView       *textview, 
+                                                GSList         *start_list,
                                                 ClickableText  *uri,
                                                 gboolean        expand_only);
 
@@ -223,7 +208,6 @@ static void save_file_cb                    (TextView       *textview,
 static void open_image_cb                      (TextView       *textview,
                                                 guint           action,
                                                 void           *data);
-static void textview_show_icon(TextView *textview, const gchar *stock_id);
 
 static GtkItemFactoryEntry textview_link_popup_entries[] = 
 {
@@ -376,8 +360,19 @@ static void textview_create_tags(GtkTextView *text, TextView *textview)
 {
        GtkTextBuffer *buffer;
        GtkTextTag *tag, *qtag;
+       static GdkColor yellow, black;
+       static gboolean color_init = FALSE;
        static PangoFontDescription *font_desc, *bold_font_desc;
        
+       if (!color_init) {
+               gdk_color_parse("#f5f6be", &yellow);
+               gdk_color_parse("#000000", &black);
+               color_init = gdk_colormap_alloc_color(
+                       gdk_colormap_get_system(), &yellow, FALSE, TRUE);
+               color_init &= gdk_colormap_alloc_color(
+                       gdk_colormap_get_system(), &black, FALSE, TRUE);
+       }
+
        if (!font_desc)
                font_desc = pango_font_description_from_string
                        (NORMAL_FONT);
@@ -403,6 +398,18 @@ static void textview_create_tags(GtkTextView *text, TextView *textview)
        gtk_text_buffer_create_tag(buffer, "header_title",
                                   "font-desc", bold_font_desc,
                                   NULL);
+       tag = gtk_text_buffer_create_tag(buffer, "hlink",
+                                  "pixels-above-lines", 0,
+                                  "pixels-above-lines-set", TRUE,
+                                  "pixels-below-lines", 0,
+                                  "pixels-below-lines-set", TRUE,
+                                  "font-desc", font_desc,
+                                  "left-margin", 3,
+                                  "left-margin-set", TRUE,
+                                  "foreground-gdk", &uri_color,
+                                  NULL);
+       g_signal_connect(G_OBJECT(tag), "event",
+                         G_CALLBACK(textview_uri_button_pressed), textview);
        if (prefs_common.enable_bgcolor) {
                gtk_text_buffer_create_tag(buffer, "quote0",
                                "foreground-gdk", &quote_colors[0],
@@ -427,6 +434,16 @@ static void textview_create_tags(GtkTextView *text, TextView *textview)
                                "foreground-gdk", &quote_colors[2],
                                NULL);
        }
+#if GTK_CHECK_VERSION(2, 8, 0)
+       gtk_text_buffer_create_tag(buffer, "tags",
+                       "foreground-gdk", &black,
+                       "paragraph-background-gdk", &yellow,
+                       NULL);
+#else
+       gtk_text_buffer_create_tag(buffer, "tags",
+                       "foreground-gdk", &emphasis_color,
+                       NULL);
+#endif
        gtk_text_buffer_create_tag(buffer, "emphasis",
                        "foreground-gdk", &emphasis_color,
                        NULL);
@@ -479,7 +496,6 @@ void textview_init(TextView *textview)
 static void textview_update_message_colors(TextView *textview)
 {
        GdkColor black = {0, 0, 0, 0};
-       GdkColor colored_emphasis = {0, 0, 0, 0xcfff};
        GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
 
        GtkTextTagTable *tags = gtk_text_buffer_get_tag_table(buffer);
@@ -501,7 +517,8 @@ static void textview_update_message_colors(TextView *textview)
                                               &uri_color);
                gtkut_convert_int_to_gdk_color(prefs_common.signature_col,
                                               &signature_color);
-               emphasis_color = colored_emphasis;
+               gtkut_convert_int_to_gdk_color(prefs_common.emphasis_col,
+                                              &emphasis_color);
        }
        if (prefs_common.enable_color && prefs_common.enable_bgcolor) {
                gtkut_convert_int_to_gdk_color(prefs_common.quote_level1_bgcol,
@@ -537,26 +554,40 @@ void textview_reflect_prefs(TextView *textview)
 void textview_show_message(TextView *textview, MimeInfo *mimeinfo,
                           const gchar *file)
 {
+       textview->loading = TRUE;
+       textview->stop_loading = FALSE;
+
        textview_clear(textview);
 
        textview_add_parts(textview, mimeinfo);
 
        textview_set_position(textview, 0);
+
+       textview->loading = FALSE;
+       textview->stop_loading = FALSE;
 }
 
 void textview_show_part(TextView *textview, MimeInfo *mimeinfo, FILE *fp)
 {
-       START_TIMING("textview_show_part");
+       START_TIMING("");
        g_return_if_fail(mimeinfo != NULL);
        g_return_if_fail(fp != NULL);
 
        if ((mimeinfo->type == MIMETYPE_MULTIPART) ||
            ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_ascii_strcasecmp(mimeinfo->subtype, "rfc822"))) {
+               textview->loading = TRUE;
+               textview->stop_loading = FALSE;
+               
                textview_clear(textview);
                textview_add_parts(textview, mimeinfo);
+
+               textview->loading = FALSE;
+               textview->stop_loading = FALSE;
                END_TIMING();
                return;
        }
+       textview->loading = TRUE;
+       textview->stop_loading = FALSE;
 
        if (fseek(fp, mimeinfo->offset, SEEK_SET) < 0)
                perror("fseek");
@@ -567,28 +598,10 @@ void textview_show_part(TextView *textview, MimeInfo *mimeinfo, FILE *fp)
                textview_add_parts(textview, mimeinfo);
        else
                textview_write_body(textview, mimeinfo);
-       END_TIMING();
-}
-
-#define TEXT_INSERT(str) \
-       gtk_text_buffer_insert_with_tags_by_name \
-                               (buffer, &iter, str, -1,\
-                                "header", NULL)
 
-#define TEXT_INSERT_LINK(str, fname, udata) {                          \
-       ClickableText *uri;                                                     \
-       uri = g_new0(ClickableText, 1);                                 \
-       uri->uri = g_strdup("");                                        \
-       uri->start = gtk_text_iter_get_offset(&iter);                   \
-       gtk_text_buffer_insert_with_tags_by_name                        \
-                               (buffer, &iter, str, -1,                \
-                                "link", "header_title", "header",      \
-                                NULL);                                 \
-       uri->end = gtk_text_iter_get_offset(&iter);                     \
-       uri->filename = fname?g_strdup(fname):NULL;                     \
-       uri->data = udata;                                              \
-       textview->uri_list =                                            \
-               g_slist_append(textview->uri_list, uri);                \
+       textview->loading = FALSE;
+       textview->stop_loading = FALSE;
+       END_TIMING();
 }
 
 static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
@@ -601,19 +614,24 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
        const gchar *name;
        gchar *content_type;
        gint charcount;
-       START_TIMING("textview_add_part");
+       START_TIMING("");
 
        g_return_if_fail(mimeinfo != NULL);
        text = GTK_TEXT_VIEW(textview->text);
        buffer = gtk_text_view_get_buffer(text);
        charcount = gtk_text_buffer_get_char_count(buffer);
        gtk_text_buffer_get_end_iter(buffer, &iter);
-
+       
+       if (textview->stop_loading) {
+               return;
+       }
        if (mimeinfo->type == MIMETYPE_MULTIPART) {
                END_TIMING();
                return;
        }
 
+       textview->prev_quote_level = -1;
+
        if ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_ascii_strcasecmp(mimeinfo->subtype, "rfc822")) {
                FILE *fp;
 
@@ -649,7 +667,7 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
        || (mimeinfo->disposition == DISPOSITIONTYPE_INLINE && 
            mimeinfo->type != MIMETYPE_TEXT)) {
                gtk_text_buffer_insert(buffer, &iter, "\n", 1);
-               TEXT_INSERT_LINK(buf, "sc://select_attachment", mimeinfo);
+               TEXTVIEW_INSERT_LINK(buf, "sc://select_attachment", mimeinfo);
                gtk_text_buffer_insert(buffer, &iter, " \n", -1);
                if (mimeinfo->type == MIMETYPE_IMAGE  &&
                    prefs_common.inline_img ) {
@@ -658,24 +676,25 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
                        gchar *filename;
                        ClickableText *uri;
                        gchar *uri_str;
+                       gint err;
                        START_TIMING("inserting image");
+
                        filename = procmime_get_tmp_file_name(mimeinfo);
 
-                       if (procmime_get_part(filename, mimeinfo) < 0) {
-                               g_warning("Can't get the image file.");
+                       if ((err = procmime_get_part(filename, mimeinfo)) < 0) {
+                               g_warning("Can't get the image file.(%s)", strerror(-err));
                                g_free(filename);
                                END_TIMING();
                                return;
                        }
 
-                       printf("1-get_part: "); END_TIMING();
-
                        if (!prefs_common.resize_img) {
                                pixbuf = gdk_pixbuf_new_from_file(filename, &error);
                        } else {
                                gint w, h;
                                gdk_pixbuf_get_file_info(filename, &w, &h);
-                               if (w > textview->scrolledwin->allocation.width - 100)
+                               if (textview->scrolledwin->allocation.width - 100 > 0 &&
+                                   w > textview->scrolledwin->allocation.width - 100)
                                        pixbuf = gdk_pixbuf_new_from_file_at_scale(filename, 
                                                textview->scrolledwin->allocation.width - 100, 
                                                -1, TRUE, &error);
@@ -692,7 +711,6 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
                                END_TIMING();
                                return;
                        }
-                       printf("2-gdk_pixbuf: "); END_TIMING();
 
                        uri_str = g_filename_to_uri(filename, NULL, NULL);
                        if (uri_str) {
@@ -705,7 +723,7 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
                                uri->end = uri->start + 1;
                                uri->filename = procmime_get_part_file_name(mimeinfo);
                                textview->uri_list =
-                                       g_slist_append(textview->uri_list, uri);
+                                       g_slist_prepend(textview->uri_list, uri);
                                
                                gtk_text_buffer_insert(buffer, &iter, " ", 1);
                                gtk_text_buffer_get_iter_at_offset(buffer, &start_iter, uri->start);    
@@ -719,6 +737,7 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
                        g_object_unref(pixbuf);
                        g_free(filename);
                        END_TIMING();
+                       GTK_EVENTS_FLUSH();
                }
        } else if (mimeinfo->type == MIMETYPE_TEXT) {
                if (prefs_common.display_header && (charcount > 0))
@@ -733,7 +752,7 @@ static void recursive_add_parts(TextView *textview, GNode *node)
 {
         GNode * iter;
        MimeInfo *mimeinfo;
-        START_TIMING("recursive_add_parts");
+        START_TIMING("");
 
         mimeinfo = (MimeInfo *) node->data;
         
@@ -810,13 +829,13 @@ void textview_show_error(TextView *textview)
        buffer = gtk_text_view_get_buffer(text);
        gtk_text_buffer_get_start_iter(buffer, &iter);
 
-       TEXT_INSERT(_("\n"
+       TEXTVIEW_INSERT(_("\n"
                      "  This message can't be displayed.\n"
                      "  This is probably due to a network error.\n"
                      "\n"
                      "  Use "));
-       TEXT_INSERT_LINK(_("'View Log'"), "sc://view_log", NULL);
-       TEXT_INSERT(_(" in the Tools menu for more information."));
+       TEXTVIEW_INSERT_LINK(_("'View Log'"), "sc://view_log", NULL);
+       TEXTVIEW_INSERT(_(" in the Tools menu for more information."));
        textview_show_icon(textview, GTK_STOCK_DIALOG_ERROR);
 
 }
@@ -826,6 +845,8 @@ void textview_show_mime_part(TextView *textview, MimeInfo *partinfo)
        GtkTextView *text;
        GtkTextBuffer *buffer;
        GtkTextIter iter;
+       const gchar *name;
+       gchar *content_type;
 
        if (!partinfo) return;
 
@@ -836,30 +857,61 @@ void textview_show_mime_part(TextView *textview, MimeInfo *partinfo)
        buffer = gtk_text_view_get_buffer(text);
        gtk_text_buffer_get_start_iter(buffer, &iter);
 
-       TEXT_INSERT("\n");
-       TEXT_INSERT(_("  The following can be performed on this part by\n"));
-       TEXT_INSERT(_("  right-clicking the icon or list item:\n"));
-
-       TEXT_INSERT(_("     - To save, select "));
-       TEXT_INSERT_LINK(_("'Save as...'"), "sc://save_as", NULL);
-       TEXT_INSERT(_(" (Shortcut key: 'y')\n"));
-       TEXT_INSERT(_("     - To display as text, select "));
-       TEXT_INSERT_LINK(_("'Display as text'"), "sc://display_as_text", NULL);
-       TEXT_INSERT(_(" (Shortcut key: 't')\n"));
-       TEXT_INSERT(_("     - To open with an external program, select "));
-       TEXT_INSERT_LINK(_("'Open'"), "sc://open", NULL);
-       TEXT_INSERT(_(" (Shortcut key: 'l')\n"));
-       TEXT_INSERT(_("       (alternately double-click, or click the middle "));
-       TEXT_INSERT(_("mouse button)\n"));
-       TEXT_INSERT(_("     - Or use "));
-       TEXT_INSERT_LINK(_("'Open with...'"), "sc://open_with", NULL);
-       TEXT_INSERT(_(" (Shortcut key: 'o')\n"));
+       TEXTVIEW_INSERT("\n");
+
+       name = procmime_mimeinfo_get_parameter(partinfo, "filename");
+       if (name == NULL)
+               name = procmime_mimeinfo_get_parameter(partinfo, "name");
+       if (name != NULL) {
+               content_type = procmime_get_content_type_str(partinfo->type,
+                                                    partinfo->subtype);
+               TEXTVIEW_INSERT("  ");
+               TEXTVIEW_INSERT_BOLD(name);
+               TEXTVIEW_INSERT(" (");
+               TEXTVIEW_INSERT(content_type);
+               TEXTVIEW_INSERT(", ");
+               TEXTVIEW_INSERT(to_human_readable(partinfo->length));
+               TEXTVIEW_INSERT("):\n\n");
+               
+               g_free(content_type);
+       }
+       TEXTVIEW_INSERT(_("  The following can be performed on this part\n"));
+#ifndef MAEMO
+       TEXTVIEW_INSERT(_("  by right-clicking the icon or list item:"));
+#endif
+       TEXTVIEW_INSERT("\n");
+
+       TEXTVIEW_INSERT(_("     - To save, select "));
+       TEXTVIEW_INSERT_LINK(_("'Save as...'"), "sc://save_as", NULL);
+#ifndef MAEMO
+       TEXTVIEW_INSERT(_(" (Shortcut key: 'y')"));
+#endif
+       TEXTVIEW_INSERT("\n");
+
+       TEXTVIEW_INSERT(_("     - To display as text, select "));
+       TEXTVIEW_INSERT_LINK(_("'Display as text'"), "sc://display_as_text", NULL);
+
+#ifndef MAEMO
+       TEXTVIEW_INSERT(_(" (Shortcut key: 't')"));
+#endif
+       TEXTVIEW_INSERT("\n");
+
+       TEXTVIEW_INSERT(_("     - To open with an external program, select "));
+       TEXTVIEW_INSERT_LINK(_("'Open'"), "sc://open", NULL);
+
+#ifndef MAEMO
+       TEXTVIEW_INSERT(_(" (Shortcut key: 'l')\n"));
+       TEXTVIEW_INSERT(_("       (alternately double-click, or click the middle "));
+       TEXTVIEW_INSERT(_("mouse button)\n"));
+       TEXTVIEW_INSERT(_("     - Or use "));
+       TEXTVIEW_INSERT_LINK(_("'Open with...'"), "sc://open_with", NULL);
+       TEXTVIEW_INSERT(_(" (Shortcut key: 'o')"));
+#endif
+       TEXTVIEW_INSERT("\n");
+
        textview_show_icon(textview, GTK_STOCK_DIALOG_INFO);
 }
 
-#undef TEXT_INSERT
-#undef TEXT_INSERT_LINK
-
 static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
 {
        FILE *tmpfp;
@@ -906,6 +958,7 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                        g_unlink(filename);
                }
                g_free(filename);
+#ifndef G_OS_WIN32
        } else if ( g_ascii_strcasecmp(mimeinfo->subtype, "plain") &&
                   (cmd = prefs_common.mime_textviewer) && *cmd &&
                   (p = strchr(cmd, '%')) && *(p + 1) == 's') {
@@ -922,14 +975,14 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                if (pipe(pfd) < 0) {
                        g_snprintf(buf, sizeof(buf),
                                "pipe failed for textview\n\n%s\n", strerror(errno));
-                       textview_write_line(textview, buf, conv);
+                       textview_write_line(textview, buf, conv, TRUE);
                        goto textview_default;
                }
                pid = fork();
                if (pid < 0) {
                        g_snprintf(buf, sizeof(buf),
                                "fork failed for textview\n\n%s\n", strerror(errno));
-                       textview_write_line(textview, buf, conv);
+                       textview_write_line(textview, buf, conv, TRUE);
                        close(pfd[0]);
                        close(pfd[1]);
                        goto textview_default;
@@ -943,7 +996,7 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                        dup(pfd[1]);
                        rc = execvp(argv[0], argv);
                        close(pfd[1]);
-                       printf (_("The command to view attachment "
+                       g_print(_("The command to view attachment "
                                "as text failed:\n"
                                "    %s\n"
                                "Exit code %d\n"), buf, rc);
@@ -951,34 +1004,55 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                }
                close(pfd[1]);
                tmpfp = fdopen(pfd[0], "rb");
-               while (fgets(buf, sizeof(buf), tmpfp))
-                       textview_write_line(textview, buf, conv);
+               while (fgets(buf, sizeof(buf), tmpfp)) {
+                       textview_write_line(textview, buf, conv, TRUE);
+                       
+                       if (textview->stop_loading) {
+                               fclose(tmpfp);
+                               waitpid(pid, pfd, 0);
+                               unlink(fname);
+                               return;
+                       }
+               }
+
                fclose(tmpfp);
                waitpid(pid, pfd, 0);
                unlink(fname);
+#endif
        } else {
 textview_default:
                tmpfp = g_fopen(mimeinfo->data.filename, "rb");
                fseek(tmpfp, mimeinfo->offset, SEEK_SET);
                debug_print("Viewing text content of type: %s (length: %d)\n", mimeinfo->subtype, mimeinfo->length);
                while ((ftell(tmpfp) < mimeinfo->offset + mimeinfo->length) &&
-                      (fgets(buf, sizeof(buf), tmpfp) != NULL))
-                       textview_write_line(textview, buf, conv);
+                      (fgets(buf, sizeof(buf), tmpfp) != NULL)) {
+                       textview_write_line(textview, buf, conv, TRUE);
+                       if (textview->stop_loading) {
+                               fclose(tmpfp);
+                               return;
+                       }
+               }
                fclose(tmpfp);
        }
 
        conv_code_converter_destroy(conv);
        procmime_force_encoding(0);
 
+       textview->uri_list = g_slist_reverse(textview->uri_list);
        for (cur = textview->uri_list; cur; cur = cur->next) {
                ClickableText *uri = (ClickableText *)cur->data;
                if (!uri->is_quote)
                        continue;
                if (!prefs_common.hide_quotes ||
                    uri->quote_level+1 < prefs_common.hide_quotes) {
-                       textview_toggle_quote(textview, uri, TRUE);
+                       textview_toggle_quote(textview, cur, uri, TRUE);
+                       if (textview->stop_loading) {
+                               return;
+                       }
                }
        }
+       
+       GTK_EVENTS_FLUSH();
 }
 
 static void textview_show_html(TextView *textview, FILE *fp,
@@ -986,6 +1060,7 @@ static void textview_show_html(TextView *textview, FILE *fp,
 {
        SC_HTMLParser *parser;
        gchar *str;
+       gint lines = 0;
 
        parser = sc_html_parser_new(fp, conv);
        g_return_if_fail(parser != NULL);
@@ -994,7 +1069,7 @@ static void textview_show_html(TextView *textview, FILE *fp,
                if (parser->state == SC_HTML_HREF) {
                        /* first time : get and copy the URL */
                        if (parser->href == NULL) {
-                               /* ALF - the sylpheed html parser returns an empty string,
+                               /* ALF - the claws html parser returns an empty string,
                                 * if still inside an <a>, but already parsed past HREF */
                                str = strtok(str, " ");
                                if (str) {
@@ -1009,9 +1084,15 @@ static void textview_show_html(TextView *textview, FILE *fp,
                        if (str != NULL)
                                textview_write_link(textview, str, parser->href, NULL);
                } else
-                       textview_write_line(textview, str, NULL);
+                       textview_write_line(textview, str, NULL, FALSE);
+               lines++;
+               if (lines % 500 == 0)
+                       GTK_EVENTS_FLUSH();
+               if (textview->stop_loading) {
+                       return;
+               }
        }
-       textview_write_line(textview, "\n", NULL);
+       textview_write_line(textview, "\n", NULL, FALSE);
        sc_html_parser_destroy(parser);
 }
 
@@ -1020,12 +1101,19 @@ static void textview_show_ertf(TextView *textview, FILE *fp,
 {
        ERTFParser *parser;
        gchar *str;
+       gint lines = 0;
 
        parser = ertf_parser_new(fp, conv);
        g_return_if_fail(parser != NULL);
 
        while ((str = ertf_parse(parser)) != NULL) {
-               textview_write_line(textview, str, NULL);
+               textview_write_line(textview, str, NULL, FALSE);
+               lines++;
+               if (lines % 500 == 0)
+                       GTK_EVENTS_FLUSH();
+               if (textview->stop_loading) {
+                       return;
+               }
        }
        
        ertf_parser_destroy(parser);
@@ -1104,6 +1192,7 @@ static void textview_make_clickable_parts(TextView *textview,
        } head = {NULL, NULL, 0,  NULL}, *last = &head;
 
        if (!g_utf8_validate(linebuf, -1, NULL)) {
+               g_free(mybuf);
                mybuf = g_malloc(strlen(linebuf)*2 +1);
                conv_localetodisp(mybuf, strlen(linebuf)*2 +1, linebuf);
        }
@@ -1165,7 +1254,7 @@ static void textview_make_clickable_parts(TextView *textview,
                        uri->end = gtk_text_iter_get_offset(&iter);
                        uri->filename = NULL;
                        textview->uri_list =
-                               g_slist_append(textview->uri_list, uri);
+                               g_slist_prepend(textview->uri_list, uri);
                }
 
                if (*normal_text)
@@ -1294,7 +1383,7 @@ static void textview_make_clickable_parts_later(TextView *textview,
                        uri->end = gtk_text_iter_get_offset(&end_iter);
                        uri->filename = NULL;
                        textview->uri_list =
-                               g_slist_append(textview->uri_list, uri);
+                               g_slist_prepend(textview->uri_list, uri);
                }
        } 
 
@@ -1304,7 +1393,7 @@ static void textview_make_clickable_parts_later(TextView *textview,
 #undef ADD_TXT_POS
 
 static void textview_write_line(TextView *textview, const gchar *str,
-                               CodeConverter *conv)
+                               CodeConverter *conv, gboolean do_quote_folding)
 {
        GtkTextView *text;
        GtkTextBuffer *buffer;
@@ -1352,41 +1441,66 @@ static void textview_write_line(TextView *textview, const gchar *str,
                fg_color = quote_tag_str;
        }
 
-       if (prefs_common.enable_color && (strcmp(buf,"-- \n") == 0 || textview->is_in_signature)) {
+       if (prefs_common.enable_color && 
+           (strcmp(buf,"-- \n") == 0 || strcmp(buf, "- -- \n") == 0 || textview->is_in_signature)) {
                fg_color = "signature";
                textview->is_in_signature = TRUE;
        }
 
-       if (real_quotelevel > -1) {
-               if ( previousquotelevel != real_quotelevel ) {
+       if (real_quotelevel > -1 && do_quote_folding) {
+               if (!g_utf8_validate(buf, -1, NULL)) {
+                       gchar *utf8buf = NULL;
+                       utf8buf = g_malloc(BUFFSIZE);
+                       conv_localetodisp(utf8buf, BUFFSIZE, buf);
+                       strncpy2(buf, utf8buf, BUFFSIZE-1);
+                       g_free(utf8buf);
+               }
+do_quote:
+               if ( textview->prev_quote_level != real_quotelevel ) {
                        ClickableText *uri;
                        uri = g_new0(ClickableText, 1);
                        uri->uri = g_strdup("");
                        uri->data = g_strdup(buf);
                        uri->start = gtk_text_iter_get_offset(&iter);
+                       uri->is_quote = TRUE;
+                       uri->quote_level = real_quotelevel;
+                       uri->fg_color = g_strdup(fg_color);
+
                        gtk_text_buffer_insert_with_tags_by_name
-                                               (buffer, &iter, " [...]", -1,
-                                                "qlink", fg_color, NULL);
+                                       (buffer, &iter, " [...]", -1,
+                                        "qlink", fg_color, NULL);
                        uri->end = gtk_text_iter_get_offset(&iter);
+                       gtk_text_buffer_insert(buffer, &iter, "  \n", -1);
+                       
                        uri->filename = NULL;
-                       uri->fg_color = g_strdup(fg_color);
-                       uri->is_quote = TRUE;
-                       uri->quote_level = real_quotelevel;
                        textview->uri_list =
-                               g_slist_append(textview->uri_list, uri);
-                       gtk_text_buffer_insert(buffer, &iter, "  \n", -1);
+                               g_slist_prepend(textview->uri_list, uri);
                
-                       previousquotelevel = real_quotelevel;
+                       textview->prev_quote_level = real_quotelevel;
                } else {
-                       GSList *last = g_slist_last(textview->uri_list);
-                       ClickableText *lasturi = (ClickableText *)last->data;
-                       gchar *tmp = g_strdup_printf("%s%s", (gchar *)lasturi->data, buf);
-                       g_free(lasturi->data);
-                       lasturi->data = tmp;
+                       GSList *last = textview->uri_list;
+                       ClickableText *lasturi = NULL;
+                       gint e_len = 0, n_len = 0;
+                       
+                       if (textview->uri_list) {
+                               lasturi = (ClickableText *)last->data;
+                       } else {
+                               g_print("oops (%d %d)\n",
+                                       real_quotelevel, textview->prev_quote_level);
+                       }               
+                       if (lasturi->is_quote == FALSE) {
+                               textview->prev_quote_level = -1;
+                               goto do_quote;
+                       }
+                       e_len = lasturi->data ? strlen(lasturi->data):0;
+                       n_len = strlen(buf);
+                       lasturi->data = g_realloc((gchar *)lasturi->data, e_len + n_len + 1);
+                       strcpy((gchar *)lasturi->data + e_len, buf);
+                       *((gchar *)lasturi->data + e_len + n_len) = '\0';
                }
        } else {
                textview_make_clickable_parts(textview, fg_color, "link", buf, FALSE);
-               previousquotelevel = -1;
+               textview->prev_quote_level = -1;
        }
 }
 
@@ -1444,7 +1558,7 @@ void textview_write_link(TextView *textview, const gchar *str,
                (buffer, &iter, bufp, -1, "link", NULL);
        r_uri->end = gtk_text_iter_get_offset(&iter);
        r_uri->filename = NULL;
-       textview->uri_list = g_slist_append(textview->uri_list, r_uri);
+       textview->uri_list = g_slist_prepend(textview->uri_list, r_uri);
 }
 
 static void textview_set_cursor(GdkWindow *window, GdkCursor *cursor)
@@ -1466,6 +1580,7 @@ void textview_clear(TextView *textview)
        textview_uri_list_remove_all(textview->uri_list);
        textview->uri_list = NULL;
        textview->uri_hover = NULL;
+       textview->prev_quote_level = -1;
 
        textview->body_pos = 0;
        if (textview->image) 
@@ -1481,8 +1596,16 @@ void textview_clear(TextView *textview)
 
 void textview_destroy(TextView *textview)
 {
+       GtkTextBuffer *buffer;
+       GtkClipboard *clipboard;
+
+       buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
+       clipboard = gtk_clipboard_get(GDK_SELECTION_PRIMARY);
+       gtk_text_buffer_remove_selection_clipboard(buffer, clipboard);
+
        textview_uri_list_remove_all(textview->uri_list);
        textview->uri_list = NULL;
+       textview->prev_quote_level = -1;
 
        g_free(textview);
 }
@@ -1513,6 +1636,7 @@ void textview_set_font(TextView *textview, const gchar *codeset)
                if (font_desc) {
                        gtk_widget_modify_font(textview->text, font_desc);
                        CHANGE_TAG_FONT("header", font_desc);
+                       CHANGE_TAG_FONT("hlink", font_desc);
                        pango_font_description_free(font_desc);
                }
                if (bold_font_desc) {
@@ -1577,6 +1701,26 @@ void textview_set_position(TextView *textview, gint pos)
        gtkut_text_view_set_position(text, pos);
 }
 
+static gboolean header_is_internal(Header *header)
+{
+       const gchar *internal_hdrs[] = 
+               {"AF:", "NF:", "PS:", "SRH:", "SFN:", "DSR:", "MID:", 
+                "CFG:", "PT:", "S:", "RQ:", "SSV:", "NSV:", "SSH:", 
+                "R:", "MAID:", "SCF:", "RMID:", "FMID:", "NAID:", 
+                "X-Claws-Account-Id:", "X-Claws-Sign:", "X-Claws-Encrypt:", 
+                "X-Claws-Privacy-System:", "X-Claws-End-Special-Headers:",
+                "X-Sylpheed-Account-Id:", "X-Sylpheed-Sign:", "X-Sylpheed-Encrypt:", 
+                "X-Sylpheed-Privacy-System:", "X-Sylpheed-End-Special-Headers:",
+                NULL};
+       int i;
+       
+       for (i = 0; internal_hdrs[i]; i++) {
+               if (!strcmp(header->name, internal_hdrs[i]))
+                       return TRUE;
+       }
+       return FALSE;
+}
+
 static GPtrArray *textview_scan_header(TextView *textview, FILE *fp)
 {
        gchar buf[BUFFSIZE];
@@ -1587,8 +1731,19 @@ static GPtrArray *textview_scan_header(TextView *textview, FILE *fp)
 
        g_return_val_if_fail(fp != NULL, NULL);
 
-       if (textview->show_all_headers)
-               return procheader_get_header_array_asis(fp);
+       if (textview->show_all_headers) {
+               headers = procheader_get_header_array_asis(fp);
+               sorted_headers = g_ptr_array_new();
+               for (i = 0; i < headers->len; i++) {
+                       header = g_ptr_array_index(headers, i);
+                       if (!header_is_internal(header))
+                               g_ptr_array_add(sorted_headers, header);
+                       else
+                               procheader_header_free(header);
+               }
+               g_ptr_array_free(headers, TRUE);
+               return sorted_headers;
+       }
 
        if (!prefs_common.display_header) {
                while (fgets(buf, sizeof(buf), fp) != NULL)
@@ -1624,7 +1779,11 @@ static GPtrArray *textview_scan_header(TextView *textview, FILE *fp)
        if (prefs_common.show_other_header) {
                for (i = 0; i < headers->len; i++) {
                        header = g_ptr_array_index(headers, i);
-                       g_ptr_array_add(sorted_headers, header);
+                       if (!header_is_internal(header)) {
+                               g_ptr_array_add(sorted_headers, header);
+                       } else {
+                               procheader_header_free(header);
+                       }
                }
                g_ptr_array_free(headers, TRUE);
        } else
@@ -1671,7 +1830,7 @@ bail:
        textview->image = NULL; 
 }
 
-static void textview_show_icon(TextView *textview, const gchar *stock_id)
+void textview_show_icon(TextView *textview, const gchar *stock_id)
 {
        GtkTextView *text = GTK_TEXT_VIEW(textview->text);
        int x = 0;
@@ -1701,7 +1860,8 @@ static void textview_show_xface(TextView *textview)
        MsgInfo *msginfo = textview->messageview->msginfo;
        GtkTextView *text = GTK_TEXT_VIEW(textview->text);
        int x = 0;
-
+       GdkWindow *window = NULL;
+       
        if (prefs_common.display_header_pane
        ||  !prefs_common.display_xface)
                goto bail;
@@ -1718,10 +1878,13 @@ static void textview_show_xface(TextView *textview)
 
        if (textview->image) 
                gtk_widget_destroy(textview->image);
-       
+
+       window = mainwindow_get_mainwindow() ?
+                       mainwindow_get_mainwindow()->window->window :
+                       textview->text->window;
        textview->image = xface_get_from_header(msginfo->extradata->xface,
                                &textview->text->style->white,
-                               textview->text->window);
+                               window);
        g_return_if_fail(textview->image != NULL);
 
        gtk_widget_show(textview->image);
@@ -1742,6 +1905,146 @@ bail:
 }
 #endif
 
+static void textview_save_contact_pic(TextView *textview)
+{
+       MsgInfo *msginfo = textview->messageview->msginfo;
+       gchar *filename = NULL;
+       GError *error = NULL;
+       GdkPixbuf *picture = NULL;
+                               
+       if (!msginfo->extradata || (!msginfo->extradata->face && !msginfo->extradata->xface))
+               return;
+
+       if (textview->image) 
+               picture = gtk_image_get_pixbuf(GTK_IMAGE(textview->image));
+
+       filename = addrindex_get_picture_file(msginfo->from);
+       if (!filename)
+               return;
+       if (!is_file_exist(filename))
+               gdk_pixbuf_save(picture, filename, "png", &error, NULL);
+       g_free(filename);
+}
+
+static void textview_show_contact_pic(TextView *textview)
+{
+       MsgInfo *msginfo = textview->messageview->msginfo;
+       GtkTextView *text = GTK_TEXT_VIEW(textview->text);
+       int x = 0;
+       gchar *filename = NULL;
+       GError *error = NULL;
+       GdkPixbuf *picture = NULL;
+       gint w, h;
+                               
+       if (prefs_common.display_header_pane
+       ||  !prefs_common.display_xface)
+               goto bail;
+       
+       if (msginfo->extradata && (msginfo->extradata->face || msginfo->extradata->xface))
+               return;
+
+       if (textview->image) 
+               gtk_widget_destroy(textview->image);
+
+       filename = addrindex_get_picture_file(msginfo->from);
+       
+       if (!filename)
+               goto bail;
+       if (!is_file_exist(filename)) {
+               g_free(filename);
+               goto bail;
+       }
+
+       gdk_pixbuf_get_file_info(filename, &w, &h);
+       
+       if (w > 48 || h > 48)
+               picture = gdk_pixbuf_new_from_file_at_scale(filename, 
+                                               48, 48, TRUE, &error);
+       else
+               picture = gdk_pixbuf_new_from_file(filename, &error);
+
+       if (error) {
+               debug_print("Failed to import image: \n%s",
+                               error->message);
+               g_error_free(error);
+               goto bail;
+       }
+       g_free(filename);
+
+       if (picture) {
+               textview->image = gtk_image_new_from_pixbuf(picture);
+               g_object_unref(picture);
+       }
+       g_return_if_fail(textview->image != NULL);
+
+       gtk_widget_show(textview->image);
+       
+       x = textview->text->allocation.width - WIDTH -5;
+
+       gtk_text_view_add_child_in_window(text, textview->image, 
+               GTK_TEXT_WINDOW_TEXT, x, 5);
+
+       gtk_widget_show_all(textview->text);
+       
+       return;
+bail:
+       if (textview->image) 
+               gtk_widget_destroy(textview->image);
+       textview->image = NULL;
+       
+}
+
+static void textview_show_tags(TextView *textview)
+{
+       MsgInfo *msginfo = textview->messageview->msginfo;
+       GtkTextView *text = GTK_TEXT_VIEW(textview->text);
+       GtkTextBuffer *buffer = gtk_text_view_get_buffer(text);
+       GtkTextIter iter;
+       ClickableText *uri;
+       GSList *cur;
+       gboolean found_tag = FALSE;
+       
+       if (!msginfo->tags)
+               return;
+       
+       for (cur = msginfo->tags; cur; cur = cur->next) {
+               if (tags_get_tag(GPOINTER_TO_INT(cur->data)) != NULL) {
+                       found_tag = TRUE;
+                       break;
+               }
+       }
+       if (!found_tag) 
+               return;
+
+       gtk_text_buffer_get_end_iter (buffer, &iter);
+       gtk_text_buffer_insert_with_tags_by_name(buffer,
+               &iter, _("Tags: "), -1,
+               "header_title", "header", "tags", NULL);
+
+       for (cur = msginfo->tags; cur; cur = cur->next) {
+               uri = g_new0(ClickableText, 1);
+               uri->uri = g_strdup("");
+               uri->start = gtk_text_iter_get_offset(&iter);
+               gtk_text_buffer_insert_with_tags_by_name(buffer, &iter, 
+                       tags_get_tag(GPOINTER_TO_INT(cur->data)), -1,
+                       "link", "header", "tags", NULL);
+               uri->end = gtk_text_iter_get_offset(&iter);
+               uri->filename = g_strdup_printf("sc://search_tags:%s", tags_get_tag(GPOINTER_TO_INT(cur->data)));
+               uri->data = NULL;
+               textview->uri_list =
+                       g_slist_prepend(textview->uri_list, uri);
+               if (cur->next)
+                       gtk_text_buffer_insert_with_tags_by_name(buffer, &iter, ", ", 2,
+                               "header", "tags", NULL);
+               else
+                       gtk_text_buffer_insert_with_tags_by_name(buffer, &iter, " ", 1,
+                               "header", "tags", NULL);
+       }
+
+       gtk_text_buffer_insert_with_tags_by_name(buffer, &iter, "\n", 1,
+               "header", "tags", NULL);
+}
+
 static void textview_show_header(TextView *textview, GPtrArray *headers)
 {
        GtkTextView *text = GTK_TEXT_VIEW(textview->text);
@@ -1752,6 +2055,8 @@ static void textview_show_header(TextView *textview, GPtrArray *headers)
 
        g_return_if_fail(headers != NULL);
 
+       textview_show_tags(textview);
+
        for (i = 0; i < headers->len; i++) {
                header = g_ptr_array_index(headers, i);
                g_return_if_fail(header->name != NULL);
@@ -1785,7 +2090,8 @@ static void textview_show_header(TextView *textview, GPtrArray *headers)
                if ((procheader_headername_equal(header->name, "X-Mailer") ||
                     procheader_headername_equal(header->name,
                                                 "X-Newsreader")) &&
-                   strstr(header->body, "Sylpheed-Claws") != NULL) {
+                   (strstr(header->body, "Claws Mail") != NULL ||
+                    strstr(header->body, "Sylpheed-Claws") != NULL)) {
                        gtk_text_buffer_get_end_iter (buffer, &iter);
                        gtk_text_buffer_insert_with_tags_by_name
                                (buffer, &iter, header->body, -1,
@@ -1799,7 +2105,7 @@ static void textview_show_header(TextView *textview, GPtrArray *headers)
                          procheader_headername_equal(header->name, "Reply-To") ||
                          procheader_headername_equal(header->name, "Sender");
                        textview_make_clickable_parts(textview, "header", 
-                                                     "link", header->body, 
+                                                     "hlink", header->body, 
                                                      hdr);
                }
                gtk_text_buffer_get_end_iter (buffer, &iter);
@@ -1811,6 +2117,8 @@ static void textview_show_header(TextView *textview, GPtrArray *headers)
 #if HAVE_LIBCOMPFACE
        textview_show_xface(textview);
 #endif
+       textview_save_contact_pic(textview);
+       textview_show_contact_pic(textview);
 }
 
 gboolean textview_search_string(TextView *textview, const gchar *str,
@@ -1834,7 +2142,7 @@ void textview_scroll_one_line(TextView *textview, gboolean up)
        GtkTextView *text = GTK_TEXT_VIEW(textview->text);
        GtkAdjustment *vadj = text->vadjustment;
 
-       gtkutils_scroll_one_line(text, vadj, up);
+       gtkutils_scroll_one_line(GTK_WIDGET(text), vadj, up);
 }
 
 gboolean textview_scroll_page(TextView *textview, gboolean up)
@@ -1842,7 +2150,7 @@ gboolean textview_scroll_page(TextView *textview, gboolean up)
        GtkTextView *text = GTK_TEXT_VIEW(textview->text);
        GtkAdjustment *vadj = text->vadjustment;
 
-       return gtkutils_scroll_page(text, vadj, up);
+       return gtkutils_scroll_page(GTK_WIDGET(text), vadj, up);
 }
 
 #define KEY_PRESS_EVENT_STOP() \
@@ -1885,6 +2193,7 @@ static gint textview_key_pressed(GtkWidget *widget, GdkEventKey *event,
                mimeview_scroll_page(messageview->mimeview, TRUE);
                break;
        case GDK_Return:
+       case GDK_KP_Enter:
                mimeview_scroll_one_line
                        (messageview->mimeview, (event->state &
                                    (GDK_SHIFT_MASK|GDK_MOD1_MASK)) != 0);
@@ -1896,7 +2205,9 @@ static gint textview_key_pressed(GtkWidget *widget, GdkEventKey *event,
        case GDK_y:
        case GDK_t:
        case GDK_l:
+       case GDK_o:
        case GDK_c:
+       case GDK_a:
                if ((event->state & (GDK_MOD1_MASK|GDK_CONTROL_MASK)) == 0) {
                        KEY_PRESS_EVENT_STOP();
                        mimeview_pass_key_press_event(messageview->mimeview,
@@ -1924,6 +2235,8 @@ static gboolean textview_motion_notify(GtkWidget *widget,
                                       GdkEventMotion *event,
                                       TextView *textview)
 {
+       if (textview->loading)
+               return FALSE;
        textview_uri_update(textview, event->x, event->y);
        gdk_window_get_pointer(widget->window, NULL, NULL, NULL);
 
@@ -1934,6 +2247,8 @@ static gboolean textview_leave_notify(GtkWidget *widget,
                                      GdkEventCrossing *event,
                                      TextView *textview)
 {
+       if (textview->loading)
+               return FALSE;
        textview_uri_update(textview, -1, -1);
 
        return FALSE;
@@ -1946,6 +2261,9 @@ static gboolean textview_visibility_notify(GtkWidget *widget,
        gint wx, wy;
        GdkWindow *window;
 
+       if (textview->loading)
+               return FALSE;
+
        window = gtk_text_view_get_window(GTK_TEXT_VIEW(widget),
                                          GTK_TEXT_WINDOW_TEXT);
 
@@ -2002,7 +2320,7 @@ static void textview_uri_update(TextView *textview, gint x, gint y)
 
                        g_object_get(G_OBJECT(tag), "name", &name, NULL);
 
-                       if ((!strcmp(name, "link"))
+                       if ((!strcmp(name, "link") || !strcmp(name, "hlink"))
                            && textview_get_uri_range(textview, &iter, tag,
                                                      &start_iter, &end_iter)) {
 
@@ -2125,10 +2443,13 @@ static ClickableText *textview_get_uri(TextView *textview,
        return uri;
 }
 
-static void textview_shift_uris_after(TextView *textview, gint start, gint shift)
+static void textview_shift_uris_after(TextView *textview, GSList *start_list, gint start, gint shift)
 {
        GSList *cur;
-       for (cur = textview->uri_list; cur; cur = cur->next) {
+       if (!start_list)
+               start_list = textview->uri_list;
+
+       for (cur = start_list; cur; cur = cur->next) {
                ClickableText *uri = (ClickableText *)cur->data;
                if (uri->start <= start)
                        continue;
@@ -2160,7 +2481,7 @@ static void textview_remove_uris_in(TextView *textview, gint start, gint end)
        }
 }
 
-static void textview_toggle_quote(TextView *textview, ClickableText *uri, gboolean expand_only)
+static void textview_toggle_quote(TextView *textview, GSList *start_list, ClickableText *uri, gboolean expand_only)
 {
        GtkTextIter start, end;
        GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
@@ -2190,7 +2511,7 @@ static void textview_toggle_quote(TextView *textview, ClickableText *uri, gboole
        if (!uri->q_expanded) {
                gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
                gtk_text_buffer_get_iter_at_offset(buffer, &end,   uri->end);
-               textview_shift_uris_after(textview, uri->start, 
+               textview_shift_uris_after(textview, start_list, uri->start, 
                        g_utf8_strlen((gchar *)uri->data, -1)-strlen(" [...]\n"));
                gtk_text_buffer_delete(buffer, &start, &end);
                gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
@@ -2206,7 +2527,7 @@ static void textview_toggle_quote(TextView *textview, ClickableText *uri, gboole
                gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
                gtk_text_buffer_get_iter_at_offset(buffer, &end,   uri->end);
                textview_remove_uris_in(textview, uri->start, uri->end);
-               textview_shift_uris_after(textview, uri->start, 
+               textview_shift_uris_after(textview, start_list, uri->start, 
                        strlen(" [...]\n")-g_utf8_strlen((gchar *)uri->data, -1));
                gtk_text_buffer_delete(buffer, &start, &end);
                gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
@@ -2272,7 +2593,7 @@ static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
                        } 
                        return TRUE;
                } else if (qlink && bevent->button == 1) {
-                       textview_toggle_quote(textview, uri, FALSE);
+                       textview_toggle_quote(textview, NULL, uri, FALSE);
                        return TRUE;
                } else if (!g_ascii_strncasecmp(uri->uri, "mailto:", 7)) {
                        if (bevent->button == 3) {
@@ -2284,16 +2605,18 @@ static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
                                               bevent->button, bevent->time);
                        } else {
                                PrefsAccount *account = NULL;
-
+                               FolderItem   *folder_item = NULL;
                                if (textview->messageview && textview->messageview->msginfo &&
                                    textview->messageview->msginfo->folder) {
-                                       FolderItem   *folder_item;
+                                       
 
                                        folder_item = textview->messageview->msginfo->folder;
                                        if (folder_item->prefs && folder_item->prefs->enable_default_account)
                                                account = account_find_from_id(folder_item->prefs->default_account);
+                                       if (!account)
+                                               account = account_find_from_item(folder_item);
                                }
-                               compose_new(account, uri->uri + 7, NULL);
+                               compose_new_with_folderitem(account, folder_item, uri->uri + 7);
                        }
                        return TRUE;
                } else if (g_ascii_strncasecmp(uri->uri, "file:", 5)) {
@@ -2326,6 +2649,20 @@ static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
        return FALSE;
 }
 
+gchar *textview_get_visible_uri                (TextView       *textview, 
+                                        ClickableText  *uri)
+{
+       GtkTextBuffer *buffer;
+       GtkTextIter start, end;
+
+       buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
+
+       gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
+       gtk_text_buffer_get_iter_at_offset(buffer, &end,   uri->end);
+
+       return gtk_text_buffer_get_text(buffer, &start, &end, FALSE);
+}
+
 /*!
  *\brief    Check to see if a web URL has been disguised as a different
  *          URL (possible with HTML email).
@@ -2337,23 +2674,15 @@ static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
  *\return   gboolean TRUE if the URL is ok, or if the user chose to open
  *          it anyway, otherwise FALSE          
  */
-static gboolean textview_uri_security_check(TextView *textview, ClickableText *uri)
+gboolean textview_uri_security_check(TextView *textview, ClickableText *uri)
 {
        gchar *visible_str;
        gboolean retval = TRUE;
-       GtkTextBuffer *buffer;
-       GtkTextIter start, end;
 
        if (is_uri_string(uri->uri) == FALSE)
                return TRUE;
 
-       buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
-
-       gtk_text_buffer_get_iter_at_offset(buffer, &start, uri->start);
-       gtk_text_buffer_get_iter_at_offset(buffer, &end,   uri->end);
-
-       visible_str = gtk_text_buffer_get_text(buffer, &start, &end, FALSE);
-
+       visible_str = textview_get_visible_uri(textview, uri);
        if (visible_str == NULL)
                return TRUE;
 
@@ -2547,7 +2876,7 @@ static void save_file_cb (TextView *textview, guint action, void *data)
        filedir = g_path_get_dirname(filename);
        if (filedir && strcmp(filedir, ".")) {
                g_free(prefs_common.attach_save_dir);
-               prefs_common.attach_save_dir = g_strdup(filedir);
+               prefs_common.attach_save_dir = g_filename_to_utf8(filedir, -1, NULL, NULL, NULL);
        }
 
        g_free(filedir);
@@ -2575,17 +2904,42 @@ static void add_uri_to_addrbook_cb (TextView *textview, guint action, void *data
        gchar *fromname, *fromaddress;
        ClickableText *uri = g_object_get_data(G_OBJECT(textview->mail_popup_menu),
                                           "menu_button");
+       GtkWidget *image = NULL;
+       GdkPixbuf *picture = NULL;
+       gboolean use_picture = FALSE;
        if (uri == NULL)
                return;
 
        /* extract url */
        fromaddress = g_strdup(uri->uri + 7);
-       /* Hiroyuki: please put this function in utils.c! */
+       
+       if (textview->messageview->msginfo &&
+          !strcmp2(fromaddress, textview->messageview->msginfo->from))
+               use_picture = TRUE;
+
        fromname = procheader_get_fromname(fromaddress);
        extract_address(fromaddress);
-       g_message("adding from textview %s <%s>", fromname, fromaddress);
-       /* Add to address book - Match */
-       addressbook_add_contact( fromname, fromaddress, NULL );
+
+       if (use_picture && 
+           textview->messageview->msginfo &&
+           textview->messageview->msginfo->extradata &&
+           textview->messageview->msginfo->extradata->face) {
+               image = face_get_from_header(textview->messageview->msginfo->extradata->face);
+       }
+#if HAVE_LIBCOMPFACE 
+       else if (use_picture && 
+                textview->messageview->msginfo &&
+                textview->messageview->msginfo->extradata &&
+                textview->messageview->msginfo->extradata->xface) {
+               image = xface_get_from_header(textview->messageview->msginfo->extradata->xface,
+                               &textview->text->style->white,
+                               mainwindow_get_mainwindow()->window->window);   
+       }
+#endif
+       if (image)
+               picture = gtk_image_get_pixbuf(GTK_IMAGE(image));
+
+       addressbook_add_contact( fromname, fromaddress, NULL, picture);
 
        g_free(fromaddress);
        g_free(fromname);
@@ -2623,3 +2977,20 @@ static void copy_mail_to_uri_cb  (TextView *textview, guint action, void *data)
                          NULL);
 }
 
+void textview_get_selection_offsets(TextView *textview, gint *sel_start, gint *sel_end)
+{
+               GtkTextView *text = GTK_TEXT_VIEW(textview->text);
+               GtkTextBuffer *buffer = gtk_text_view_get_buffer(text);
+               GtkTextIter start, end;
+               if (gtk_text_buffer_get_selection_bounds(buffer, &start, &end)) {
+                       if (sel_start)
+                               *sel_start = gtk_text_iter_get_offset(&start);
+                       if (sel_end)
+                               *sel_end = gtk_text_iter_get_offset(&end);
+               } else {
+                       if (sel_start)
+                               *sel_start = -1;
+                       if (sel_end)
+                               *sel_end = -1;
+               }
+}