static void textview_write_line (TextView *textview,
const gchar *str,
CodeConverter *conv);
+static void textview_write_link (TextView *textview,
+ const gchar *url,
+ const gchar *str,
+ CodeConverter *conv);
static GPtrArray *textview_scan_header (TextView *textview,
FILE *fp);
static void textview_show_header (TextView *textview,
} else {
if (mimeinfo->mime_type == MIME_TEXT && mimeinfo->parent) {
glong fpos;
+ MimeInfo *parent = mimeinfo->parent;
+
+ while (parent->parent) {
+ if (parent->main &&
+ parent->main->mime_type ==
+ MIME_MESSAGE_RFC822)
+ break;
+ parent = parent->parent;
+ }
if ((fpos = ftell(fp)) < 0)
perror("ftell");
- else if (fseek(fp, mimeinfo->parent->fpos, SEEK_SET)
- < 0)
+ else if (fseek(fp, parent->fpos, SEEK_SET) < 0)
perror("fseek");
else {
headers = textview_scan_header(textview, fp);
perror("fseek");
}
}
+ /* skip MIME part headers */
while (fgets(buf, sizeof(buf), fp) != NULL)
if (buf[0] == '\r' || buf[0] == '\n') break;
}
{
HTMLParser *parser;
gchar *str;
+ gchar* url = NULL;
parser = html_parser_new(fp, conv);
g_return_if_fail(parser != NULL);
while ((str = html_parse(parser)) != NULL) {
- textview_write_line(textview, str, NULL);
+ if (parser->state == HTML_HREF) {
+ /* first time : get and copy the URL */
+ if (url == NULL) {
+ /* ALF - the sylpheed html parser returns an empty string,
+ * if still inside an <a>, but already parsed past HREF */
+ str = strtok(str, " ");
+ if (str) {
+ url = strdup(str);
+ /* the URL may (or not) be followed by the
+ * referenced text */
+ str = strtok(NULL, "");
+ }
+ }
+ if (str != NULL) {
+ textview_write_link(textview, url, str, NULL);
+ }
+ } else {
+ if (url != NULL) {
+ free(url);
+ url = NULL;
+ }
+ textview_write_line(textview, str, NULL);
+ }
}
html_parser_destroy(parser);
}
* should pass some URI type to this function and decide on that whether
* to perform punctuation stripping */
-#define IS_REAL_PUNCT(ch) \
- (ispunct(ch) && ((ch) != '/'))
+#define IS_REAL_PUNCT(ch) (ispunct(ch) && ((ch) != '/'))
for (; ep_ - 1 > scanpos + 1 && IS_REAL_PUNCT(*(ep_ - 1)); ep_--)
;
!isspace(ch) && \
!strchr("()<>\"", (ch)))
+/* alphabet and number within 7bit ASCII */
+#define IS_ASCII_ALNUM(ch) (isascii(ch) && isalnum(ch))
#define IS_QUOTE(ch) ((ch) == '\'' || (ch) == '"')
/* get_email_part() - retrieves an email address. Returns TRUE if succesful */
/* TODO: should start with an alnum? */
bp_++;
- for (; bp_ < scanpos && !isalnum(*bp_); bp_++)
+ for (; bp_ < scanpos && !IS_ASCII_ALNUM(*bp_); bp_++)
;
if (bp_ != scanpos) {
;
/* TODO: really should terminate with an alnum? */
- for (; ep_ > scanpos && !isalnum(*ep_); --ep_)
+ for (; ep_ > scanpos && !IS_ASCII_ALNUM(*ep_); --ep_)
;
ep_++;
#undef ADD_TXT_POS
+/* This function writes str as a double-clickable link with the given url. */
+static void textview_write_link(TextView *textview, const gchar *url,
+ const gchar *str, CodeConverter *conv)
+{
+ GdkColor *link_color = NULL;
+ RemoteURI* uri;
+ GtkText *text = GTK_TEXT(textview->text);
+ gchar buf[BUFFSIZE];
+
+ /* this part is taken from textview_write_line. Right now the only place
+ * that calls this function passes NULL for conv, but you never know. */
+ if (!conv)
+ strncpy2(buf, str, sizeof(buf));
+ else if (conv_convert(conv, buf, sizeof(buf), str) < 0) {
+ gtk_text_insert(text, textview->msgfont,
+ prefs_common.enable_color
+ ? &error_color : NULL, NULL,
+ "*** Warning: code conversion failed ***\n",
+ -1);
+ return;
+ }
+
+ /* this part is based on the code in make_clickable_parts */
+ if (prefs_common.enable_color) {
+ link_color = &uri_color;
+ }
+ uri = g_new(RemoteURI, 1);
+ uri->uri = g_strdup(url);
+ uri->start = gtk_text_get_point(text);
+ gtk_text_insert(text, textview->msgfont, link_color, NULL, buf,
+ strlen(buf));
+ uri->end = gtk_text_get_point(text);
+ textview->uri_list = g_slist_append(textview->uri_list, uri);
+}
+
static void textview_write_line(TextView *textview, const gchar *str,
CodeConverter *conv)
{
GtkText *text = GTK_TEXT(textview->text);
gchar buf[BUFFSIZE];
- size_t len;
GdkColor *fg_color;
gint quotelevel = -1;
return;
}
- len = strlen(buf);
- if (len > 1 && buf[len - 1] == '\n' && buf[len - 2] == '\r') {
- buf[len - 2] = '\n';
- buf[len - 1] = '\0';
- }
+ strcrchomp(buf);
if (prefs_common.conv_mb_alnum) conv_mb_alnum(buf);
fg_color = NULL;
if (MB_CUR_MAX > 1) {
if (codeset) {
if (!g_strncasecmp(codeset, "ISO-8859-", 9) ||
- !g_strncasecmp(codeset, "KOI8-", 5) ||
- !g_strncasecmp(codeset, "CP", 2) ||
- !g_strncasecmp(codeset, "WINDOWS-", 8) ||
!g_strcasecmp(codeset, "BALTIC"))
use_fontset = FALSE;
+ else if (conv_get_current_charset() != C_EUC_JP &&
+ (!g_strncasecmp(codeset, "KOI8-", 5) ||
+ !g_strncasecmp(codeset, "CP", 2) ||
+ !g_strncasecmp(codeset, "WINDOWS-", 8)))
+ use_fontset = FALSE;
}
} else
use_fontset = FALSE;
textview_scroll_page(textview, FALSE);
break;
case GDK_BackSpace:
- case GDK_Delete:
textview_scroll_page(textview, TRUE);
break;
case GDK_Return:
|| event->button == 2 || event->button == 3)) {
GSList *cur;
+ /* double click seems to set the cursor after the current
+ * word. The cursor position needs fixing, otherwise the
+ * last word of a clickable zone will not work */
+ if (event->button == 1 && event->type == GDK_2BUTTON_PRESS) {
+ textview->cur_pos--;
+ }
+
for (cur = textview->uri_list; cur != NULL; cur = cur->next) {
RemoteURI *uri = (RemoteURI *)cur->data;