2004-11-08 [colin] 0.9.12cvs141.1
[claws.git] / src / textview.c
index e23d2a32621e19b4142e2313db14207e8d81a46b..a6c3381946893d80a517f3934ec22e45a0a3cfef 100644 (file)
@@ -52,8 +52,6 @@
 #include "mimeview.h"
 #include "alertpanel.h"
 
-typedef struct _RemoteURI      RemoteURI;
-
 struct _RemoteURI
 {
        gchar *uri;
@@ -99,6 +97,9 @@ static GdkColor error_color = {
 #endif
 
 
+static GdkCursor *hand_cursor = NULL;
+static GdkCursor *text_cursor = NULL;
+
 #define TEXTVIEW_STATUSBAR_PUSH(textview, str)                                     \
 {                                                                          \
        gtk_statusbar_push(GTK_STATUSBAR(textview->messageview->statusbar), \
@@ -137,12 +138,40 @@ static GPtrArray *textview_scan_header    (TextView       *textview,
 static void textview_show_header       (TextView       *textview,
                                         GPtrArray      *headers);
 
-static gint textview_key_pressed       (GtkWidget      *widget,
-                                        GdkEventKey    *event,
-                                        TextView       *textview);
-static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
-                                           GdkEvent *event, GtkTextIter *iter,
-                                           TextView *textview);
+static gint textview_key_pressed               (GtkWidget      *widget,
+                                                GdkEventKey    *event,
+                                                TextView       *textview);
+static gboolean textview_motion_notify         (GtkWidget      *widget,
+                                                GdkEventMotion *motion,
+                                                TextView       *textview);
+static gboolean textview_leave_notify          (GtkWidget      *widget,
+                                                GdkEventCrossing *event,
+                                                TextView       *textview);
+static gboolean textview_visibility_notify     (GtkWidget      *widget,
+                                                GdkEventVisibility *event,
+                                                TextView       *textview);
+static void textview_uri_update                        (TextView       *textview,
+                                                gint           x,
+                                                gint           y);
+static gboolean textview_get_uri_range         (TextView       *textview,
+                                                GtkTextIter    *iter,
+                                                GtkTextTag     *tag,
+                                                GtkTextIter    *start_iter,
+                                                GtkTextIter    *end_iter);
+static RemoteURI *textview_get_uri_from_range  (TextView       *textview,
+                                                GtkTextIter    *iter,
+                                                GtkTextTag     *tag,
+                                                GtkTextIter    *start_iter,
+                                                GtkTextIter    *end_iter);
+static RemoteURI *textview_get_uri             (TextView       *textview,
+                                                GtkTextIter    *iter,
+                                                GtkTextTag     *tag);
+static gboolean textview_uri_button_pressed    (GtkTextTag     *tag,
+                                                GObject        *obj,
+                                                GdkEvent       *event,
+                                                GtkTextIter    *iter,
+                                                TextView       *textview);
+
 static void textview_smooth_scroll_do          (TextView       *textview,
                                                 gfloat          old_value,
                                                 gfloat          last_value,
@@ -185,9 +214,10 @@ TextView *textview_create(void)
 
        /* create GtkSText widgets for single-byte and multi-byte character */
        text = gtk_text_view_new();
+       gtk_widget_add_events(text, GDK_LEAVE_NOTIFY_MASK);
        gtk_widget_show(text);
        gtk_text_view_set_editable(GTK_TEXT_VIEW(text), FALSE);
-       gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(text), GTK_WRAP_WORD);
+       gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(text), GTK_WRAP_WORD_CHAR);
        gtk_text_view_set_cursor_visible(GTK_TEXT_VIEW(text), FALSE);
        g_signal_connect(G_OBJECT(text), "populate-popup",
                 G_CALLBACK(populate_popup), NULL);
@@ -211,9 +241,23 @@ TextView *textview_create(void)
 
        gtk_container_add(GTK_CONTAINER(scrolledwin), text);
 
+       if (!hand_cursor)
+               hand_cursor = gdk_cursor_new(GDK_HAND2);
+       if (!text_cursor)
+               text_cursor = gdk_cursor_new(GDK_XTERM);
+
        g_signal_connect(G_OBJECT(text), "key_press_event",
                         G_CALLBACK(textview_key_pressed),
                         textview);
+       g_signal_connect(G_OBJECT(text), "motion_notify_event",
+                        G_CALLBACK(textview_motion_notify),
+                        textview);
+       g_signal_connect(G_OBJECT(text), "leave_notify_event",
+                        G_CALLBACK(textview_leave_notify),
+                        textview);
+       g_signal_connect(G_OBJECT(text), "visibility_notify_event",
+                        G_CALLBACK(textview_visibility_notify),
+                        textview);
 
        gtk_widget_show(scrolledwin);
 
@@ -229,7 +273,6 @@ TextView *textview_create(void)
        textview->body_pos         = 0;
        textview->show_all_headers = FALSE;
        textview->last_buttonpress = GDK_NOTHING;
-       textview->show_url_msgid   = 0;
 
        return textview;
 }
@@ -268,6 +311,9 @@ static void textview_create_tags(GtkTextView *text, TextView *textview)
        tag = gtk_text_buffer_create_tag(buffer, "link",
                                         "foreground-gdk", &uri_color,
                                         NULL);
+       gtk_text_buffer_create_tag(buffer, "link-hover",
+                                  "underline", PANGO_UNDERLINE_SINGLE,
+                                  NULL);
 
        g_signal_connect(G_OBJECT(tag), "event",
                          G_CALLBACK(textview_uri_button_pressed), textview);
@@ -332,7 +378,7 @@ void textview_show_part(TextView *textview, MimeInfo *mimeinfo, FILE *fp)
        g_return_if_fail(fp != NULL);
 
        if ((mimeinfo->type == MIMETYPE_MULTIPART) ||
-           ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_strcasecmp(mimeinfo->subtype, "rfc822"))) {
+           ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_ascii_strcasecmp(mimeinfo->subtype, "rfc822"))) {
                textview_clear(textview);
                textview_add_parts(textview, mimeinfo);
                return;
@@ -369,10 +415,10 @@ static void textview_add_part(TextView *textview, MimeInfo *mimeinfo)
 
        if (mimeinfo->type == MIMETYPE_MULTIPART) return;
 
-       if ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_strcasecmp(mimeinfo->subtype, "rfc822")) {
+       if ((mimeinfo->type == MIMETYPE_MESSAGE) && !g_ascii_strcasecmp(mimeinfo->subtype, "rfc822")) {
                FILE *fp;
 
-               fp = fopen(mimeinfo->filename, "rb");
+               fp = fopen(mimeinfo->data.filename, "rb");
                fseek(fp, mimeinfo->offset, SEEK_SET);
                headers = textview_scan_header(textview, fp);
                if (headers) {
@@ -422,7 +468,7 @@ static void recursive_add_parts(TextView *textview, GNode *node)
             (mimeinfo->type != MIMETYPE_MESSAGE))
                 return;
         
-        if (strcasecmp(mimeinfo->subtype, "alternative") == 0) {
+        if (g_ascii_strcasecmp(mimeinfo->subtype, "alternative") == 0) {
                 GNode * prefered_body;
                 int prefered_score;
                 
@@ -445,7 +491,7 @@ static void recursive_add_parts(TextView *textview, GNode *node)
                                 score = 2;
                         
                         if (submime->subtype != NULL) {
-                                if (strcasecmp(submime->subtype, "plain") == 0)
+                                if (g_ascii_strcasecmp(submime->subtype, "plain") == 0)
                                         score = 3;
                         }
                         
@@ -547,7 +593,7 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
 
        procmime_decode_content(mimeinfo);
 
-       if (!g_strcasecmp(mimeinfo->subtype, "html")) {
+       if (!g_ascii_strcasecmp(mimeinfo->subtype, "html")) {
                gchar *filename;
                
                filename = procmime_get_tmp_file_name(mimeinfo);
@@ -558,7 +604,7 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                        unlink(filename);
                }
                g_free(filename);
-       } else if (!g_strcasecmp(mimeinfo->subtype, "enriched")) {
+       } else if (!g_ascii_strcasecmp(mimeinfo->subtype, "enriched")) {
                gchar *filename;
                
                filename = procmime_get_tmp_file_name(mimeinfo);
@@ -570,7 +616,7 @@ static void textview_write_body(TextView *textview, MimeInfo *mimeinfo)
                }
                g_free(filename);
        } else {
-               tmpfp = fopen(mimeinfo->filename, "rb");
+               tmpfp = 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 ((fgets(buf, sizeof(buf), tmpfp) != NULL) && 
@@ -600,7 +646,7 @@ static void textview_show_html(TextView *textview, FILE *fp,
                                 * if still inside an <a>, but already parsed past HREF */
                                str = strtok(str, " ");
                                if (str) { 
-                                       parser->href = strdup(str);
+                                       parser->href = g_strdup(str);
                                        /* the URL may (or not) be followed by the
                                         * referenced text */
                                        str = strtok(NULL, "");
@@ -647,7 +693,7 @@ static gboolean get_uri_part(const gchar *start, const gchar *scanpos,
        /* find end point of URI */
        for (ep_ = scanpos; *ep_ != '\0'; ep_++) {
                if (!isgraph(*(const guchar *)ep_) ||
-                   !isascii(*(const guchar *)ep_) ||
+                   !IS_ASCII(*(const guchar *)ep_) ||
                    strchr("()<>\"", *ep_))
                        break;
        }
@@ -679,14 +725,14 @@ static gchar *make_uri_string(const gchar *bp, const gchar *ep)
 
 /* valid mail address characters */
 #define IS_RFC822_CHAR(ch) \
-       (isascii(ch) && \
+       (IS_ASCII(ch) && \
         (ch) > 32   && \
         (ch) != 127 && \
         !isspace(ch) && \
         !strchr("(),;<>\"", (ch)))
 
 /* alphabet and number within 7bit ASCII */
-#define IS_ASCII_ALNUM(ch)     (isascii(ch) && isalnum(ch))
+#define IS_ASCII_ALNUM(ch)     (IS_ASCII(ch) && isalnum(ch))
 #define IS_QUOTE(ch) ((ch) == '\'' || (ch) == '"')
 
 static GHashTable *create_domain_tab(void)
@@ -983,7 +1029,7 @@ static void textview_make_clickable_parts(TextView *textview,
                const gchar     *bp, *ep;       /* text position */
                gint             pti;           /* index in parse table */
                struct txtpos   *next;          /* next */
-       } head = {NULL, NULL, 0,  NULL}, *last = &head;
+       } head = {NULL, NULL, 0,  NULL}, *last = &head, *next = NULL;
 
        gtk_text_buffer_get_end_iter(buffer, &iter);
 
@@ -1026,7 +1072,15 @@ static void textview_make_clickable_parts(TextView *textview,
                for (last = head.next; last != NULL;
                     normal_text = last->ep, last = last->next) {
                        RemoteURI *uri;
-
+                       next = last->next;
+                       /* fix "colin@colino.net" <colin@colino.net> types of URIs
+                       /* FIXME would be better to fix it in the email parser */
+                       if (next && next->bp == last->ep) {
+                               next->bp = last->bp;
+                               if (*(next->bp -1 )=='"' && strchr(next->bp, '"'))
+                                       next->bp--;
+                               continue;
+                       }
                        uri = g_new(RemoteURI, 1);
                        if (last->bp - normal_text > 0)
                                gtk_text_buffer_insert_with_tags_by_name
@@ -1416,8 +1470,8 @@ gboolean textview_search_string(TextView *textview, const gchar *str,
                gchar *text = NULL;
                int i = 0;
                gtk_text_buffer_get_end_iter(buffer, &real_end);
-               text = strdup(gtk_text_buffer_get_text(buffer, &iter, 
-                                                      &real_end, FALSE));
+               text = gtk_text_buffer_get_text(buffer, &iter, 
+                                               &real_end, FALSE);
                
                while (!found && i++ < strlen(text) - 1) {
                        found = (strncasecmp(text+i, str, strlen(str)) == 0);
@@ -1482,8 +1536,8 @@ gboolean textview_search_string_backward(TextView *textview, const gchar *str,
                i = gtk_text_iter_get_offset(&iter) - strlen(str) - 1;
                gtk_text_buffer_get_start_iter(buffer, &real_start);
                
-               text = strdup(gtk_text_buffer_get_text(buffer, &real_start, 
-                                                      &iter, FALSE));
+               text = gtk_text_buffer_get_text(buffer, &real_start, 
+                                               &iter, FALSE);
 
                while (!found && i-- > 0) {
                        found = (strncasecmp(text+i, str, strlen(str)) == 0);
@@ -1753,15 +1807,6 @@ static gint textview_key_pressed(GtkWidget *widget, GdkEventKey *event,
        return TRUE;
 }
 
-static gint show_url_timeout_cb(gpointer data)
-{
-       TextView *textview = (TextView *)data;
-       
-       TEXTVIEW_STATUSBAR_POP(textview);
-       textview->show_url_timeout_tag = 0;
-       return FALSE;
-}
-
 /*!
  *\brief    Check to see if a web URL has been disguised as a different
  *          URL (possible with HTML email).
@@ -1778,9 +1823,9 @@ static gboolean uri_security_check(RemoteURI *uri, TextView *textview)
        gchar *clicked_str;
        gboolean retval = TRUE;
 
-       if (g_strncasecmp(uri->uri, "http:", 5) &&
-           g_strncasecmp(uri->uri, "https:", 6) &&
-           g_strncasecmp(uri->uri, "www.", 4)) 
+       if (g_ascii_strncasecmp(uri->uri, "http:", 5) &&
+           g_ascii_strncasecmp(uri->uri, "https:", 6) &&
+           g_ascii_strncasecmp(uri->uri, "www.", 4)) 
                return retval;
 
        clicked_str = gtk_editable_get_chars(GTK_EDITABLE(textview->text),
@@ -1790,9 +1835,9 @@ static gboolean uri_security_check(RemoteURI *uri, TextView *textview)
                return TRUE;
 
        if (strcmp(clicked_str, uri->uri) &&
-           (!g_strncasecmp(clicked_str, "http:",  5) ||
-            !g_strncasecmp(clicked_str, "https:", 6) ||
-            !g_strncasecmp(clicked_str, "www.",   4))) {
+           (!g_ascii_strncasecmp(clicked_str, "http:",  5) ||
+            !g_ascii_strncasecmp(clicked_str, "https:", 6) ||
+            !g_ascii_strncasecmp(clicked_str, "www.",   4))) {
                gchar *str;
                retval = FALSE;
 
@@ -1800,7 +1845,7 @@ static gboolean uri_security_check(RemoteURI *uri, TextView *textview)
                   and   clicked_str ==        somewhere.com */
                str = g_strconcat("http://", clicked_str, NULL);
 
-               if (!g_strcasecmp(str, uri->uri))
+               if (!g_ascii_strcasecmp(str, uri->uri))
                        retval = TRUE;
                g_free(str);
        }
@@ -1826,95 +1871,253 @@ static gboolean uri_security_check(RemoteURI *uri, TextView *textview)
        return retval;
 }
 
-static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
-                                           GdkEvent *event, GtkTextIter *iter,
-                                           TextView *textview)
+static gboolean textview_motion_notify(GtkWidget *widget,
+                                      GdkEventMotion *event,
+                                      TextView *textview)
+{
+       textview_uri_update(textview, event->x, event->y);
+       gdk_window_get_pointer(widget->window, NULL, NULL, NULL);
+
+       return FALSE;
+}
+
+static gboolean textview_leave_notify(GtkWidget *widget,
+                                     GdkEventCrossing *event,
+                                     TextView *textview)
 {
+       textview_uri_update(textview, -1, -1);
+
+       return FALSE;
+}
+
+static gboolean textview_visibility_notify(GtkWidget *widget,
+                                          GdkEventVisibility *event,
+                                          TextView *textview)
+{
+       gint wx, wy;
+  
+       gdk_window_get_pointer(widget->window, &wx, &wy, NULL);
+       textview_uri_update(textview, wx, wy);
+
+       return FALSE;
+}
+
+static void textview_uri_update(TextView *textview, gint x, gint y)
+{
+       GtkTextBuffer *buffer;
        GtkTextIter start_iter, end_iter;
-       gint start_pos, end_pos;
-       GdkEventButton *bevent;
-       GSList *cur;
-       gchar *trimmed_uri;
-       if (event->type != GDK_BUTTON_PRESS && event->type != GDK_2BUTTON_PRESS
-               && event->type != GDK_MOTION_NOTIFY)
-               return FALSE;
+       RemoteURI *uri = NULL;
+       
+       buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(textview->text));
+
+       if (x != -1 && y != -1) {
+               gint bx, by;
+               GtkTextIter iter;
+               GSList *tags;
+               GSList *cur;
+           
+               gtk_text_view_window_to_buffer_coords(GTK_TEXT_VIEW(textview->text), 
+                                                     GTK_TEXT_WINDOW_WIDGET,
+                                                     x, y, &bx, &by);
+               gtk_text_view_get_iter_at_location(GTK_TEXT_VIEW(textview->text),
+                                                  &iter, bx, by);
+
+               tags = gtk_text_iter_get_tags(&iter);
+               for (cur = tags; cur != NULL; cur = cur->next) {
+                       GtkTextTag *tag = cur->data;
+                       char *name;
+
+                       g_object_get(G_OBJECT(tag), "name", &name, NULL);
+                       if (!strcmp(name, "link")
+                           && textview_get_uri_range(textview, &iter, tag,
+                                                     &start_iter, &end_iter))
+                               uri = textview_get_uri_from_range(textview,
+                                                                 &iter, tag,
+                                                                 &start_iter,
+                                                                 &end_iter);
+                       g_free(name);
+
+                       if (uri)
+                               break;
+               }
+               g_slist_free(tags);
+       }
+       
+       if (uri != textview->uri_hover) {
+               GdkWindow *window;
+
+               if (textview->uri_hover)
+                       gtk_text_buffer_remove_tag_by_name(buffer,
+                                                          "link-hover",
+                                                          &textview->uri_hover_start_iter,
+                                                          &textview->uri_hover_end_iter);
+                   
+               textview->uri_hover = uri;
+               if (uri) {
+                       textview->uri_hover_start_iter = start_iter;
+                       textview->uri_hover_end_iter = end_iter;
+               }
+               
+               window = gtk_text_view_get_window(GTK_TEXT_VIEW(textview->text),
+                                                 GTK_TEXT_WINDOW_TEXT);
+               gdk_window_set_cursor(window, uri ? hand_cursor : text_cursor);
 
-       bevent = (GdkEventButton *) event;
+               TEXTVIEW_STATUSBAR_POP(textview);
+
+               if (uri) {
+                       char *trimmed_uri;
 
-       /* get start and end positions */
-        start_iter = *iter;
-        if(!gtk_text_iter_backward_to_tag_toggle(&start_iter, tag)) {
+                       gtk_text_buffer_apply_tag_by_name(buffer,
+                                                         "link-hover",
+                                                         &start_iter,
+                                                         &end_iter);
+
+                       trimmed_uri = trim_string(uri->uri, 60);
+                       TEXTVIEW_STATUSBAR_PUSH(textview, trimmed_uri);
+                       g_free(trimmed_uri);
+               }
+       }
+}
+
+static gboolean textview_get_uri_range(TextView *textview,
+                                      GtkTextIter *iter,
+                                      GtkTextTag *tag,
+                                      GtkTextIter *start_iter,
+                                      GtkTextIter *end_iter)
+{
+       GtkTextIter _start_iter, _end_iter;
+
+       _start_iter = *iter;
+       if(!gtk_text_iter_backward_to_tag_toggle(&_start_iter, tag)) {
                debug_print("Can't find start.");
                return FALSE;
        }
-       start_pos = gtk_text_iter_get_offset(&start_iter);
 
-       end_iter = *iter;
-       if(!gtk_text_iter_forward_to_tag_toggle(&end_iter, tag)) {
+       _end_iter = *iter;
+       if(!gtk_text_iter_forward_to_tag_toggle(&_end_iter, tag)) {
                debug_print("Can't find end");
                return FALSE;
        }
-       end_pos = gtk_text_iter_get_offset(&end_iter);
 
-       /* search current uri */
-       for (cur = textview->uri_list; cur != NULL; cur = cur->next) {
-               RemoteURI *uri = (RemoteURI *)cur->data;
+       *start_iter = _start_iter;
+       *end_iter = _end_iter;
 
-               if (start_pos != uri->start || end_pos !=  uri->end)
-                       continue;
+       return TRUE;
+}
+
+static RemoteURI *textview_get_uri_from_range(TextView *textview,
+                                             GtkTextIter *iter,
+                                             GtkTextTag *tag,
+                                             GtkTextIter *start_iter,
+                                             GtkTextIter *end_iter)
+{
+       gint start_pos, end_pos, cur_pos;
+       RemoteURI *uri = NULL;
+       GSList *cur;
 
-               trimmed_uri = trim_string(uri->uri, 60);
-               /* hover or single click: display url in statusbar */
+       start_pos = gtk_text_iter_get_offset(start_iter);
+       end_pos = gtk_text_iter_get_offset(end_iter);
+       cur_pos = gtk_text_iter_get_offset(iter);
 
-               if (event->type == GDK_MOTION_NOTIFY
-                   || (event->type == GDK_BUTTON_PRESS && bevent->button == 1)) {
-                       if (textview->messageview->mainwin) {
-                               TEXTVIEW_STATUSBAR_PUSH(textview, trimmed_uri);
-                               textview->show_url_timeout_tag = gtk_timeout_add
-                                       (4000, show_url_timeout_cb, textview);
+       for (cur = textview->uri_list; cur != NULL; cur = cur->next) {
+               RemoteURI *uri_ = (RemoteURI *)cur->data;
+               if (start_pos == uri_->start &&
+                   end_pos ==  uri_->end) {
+                       uri = uri_;
+                       break;
+               } else if (start_pos == uri_->start ||
+                          end_pos == uri_->end) {
+                       /* in case of contiguous links, textview_get_uri_range
+                        * returns a broader range (start of 1st link to end
+                        * of last link).
+                        * In that case, correct link is the one covering
+                        * current iter.
+                        */
+                       if (uri_->start <= cur_pos && cur_pos <= uri_->end) {
+                               uri = uri_;
+                               break;
                        }
-                       return FALSE;
-               }
-               /* doubleclick: open compose / add address / browser */
-               if ((event->type == GDK_2BUTTON_PRESS && bevent->button == 1) ||
-                       bevent->button == 2 || bevent->button == 3) {
-                       if (!g_strncasecmp(uri->uri, "mailto:", 7)) {
-                               if (bevent->button == 3) {
-                                       gchar *fromname, *fromaddress;
+               } 
+       }
+
+       return uri;
+}
+
+static RemoteURI *textview_get_uri(TextView *textview,
+                                  GtkTextIter *iter,
+                                  GtkTextTag *tag)
+{
+       GtkTextIter start_iter, end_iter;
+       RemoteURI *uri = NULL;
+
+       if (textview_get_uri_range(textview, iter, tag, &start_iter,
+                                  &end_iter))
+               uri = textview_get_uri_from_range(textview, iter, tag,
+                                                 &start_iter, &end_iter);
+
+       return uri;
+}
+
+static gboolean textview_uri_button_pressed(GtkTextTag *tag, GObject *obj,
+                                           GdkEvent *event, GtkTextIter *iter,
+                                           TextView *textview)
+{
+       GdkEventButton *bevent;
+       RemoteURI *uri = NULL;
+       GSList *cur;
+       gchar *trimmed_uri;
+
+       if (!event)
+               return FALSE;
+
+       if (event->type != GDK_BUTTON_PRESS && event->type != GDK_2BUTTON_PRESS
+               && event->type != GDK_MOTION_NOTIFY)
+               return FALSE;
+
+       uri = textview_get_uri(textview, iter, tag);
+       if (!uri)
+               return FALSE;
+
+       bevent = (GdkEventButton *) event;
+       
+       /* doubleclick: open compose / add address / browser */
+       if ((event->type == GDK_BUTTON_PRESS && bevent->button == 1) ||
+               bevent->button == 2 || bevent->button == 3) {
+               if (!g_ascii_strncasecmp(uri->uri, "mailto:", 7)) {
+                       if (bevent->button == 3) {
+                               gchar *fromname, *fromaddress;
                                                
-                                       /* extract url */
-                                       fromaddress = g_strdup(uri->uri + 7);
-                                       /* Hiroyuki: please put this function in utils.c! */
-                                       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 );
+                               /* extract url */
+                               fromaddress = g_strdup(uri->uri + 7);
+                               /* Hiroyuki: please put this function in utils.c! */
+                               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 );
                                                
-                                       g_free(fromaddress);
-                                       g_free(fromname);
-                               } else {
-                                       PrefsAccount *account = 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);
-                                       }
-                                       compose_new(account, uri->uri + 7, NULL);
-                               }
-                               return TRUE;
+                               g_free(fromaddress);
+                               g_free(fromname);
                        } else {
-                               if (textview_uri_security_check(textview, uri) == TRUE) 
-                                       open_uri(uri->uri,
-                                                prefs_common.uri_cmd);
-                               return TRUE;
+                               PrefsAccount *account = 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);
+                               }
+                               compose_new(account, uri->uri + 7, NULL);
                        }
+                       return TRUE;
+               } else {
+                       if (textview_uri_security_check(textview, uri) == TRUE) 
+                               open_uri(uri->uri,
+                                        prefs_common.uri_cmd);
+                       return TRUE;
                }
-               g_free(trimmed_uri);
        }
 
        return FALSE;