i2001-07-02 Larry Ewing <lewing@ximian.com>

	* configure.in: Check for gtkhtml 0.9.99.1.

2001-07-02  Larry Ewing  <lewing@ximian.com>

	* mail-display.c (save_part): move the saving logic out of save_cb
	so that we can reuse it for the image saving code.
	(save_cb): call save_part.
	(save_url): new function to resolve an save an html url.
	(image_save_as): save function for images and.  Add it to the
	link_menu, and add MASK_URL and MASK_SRC to the structure so that
	we cans show the proper menus.
	(html_button_press_event): call get_src as well to find external
	urls references.  Add popup masking.

svn path=/trunk/; revision=10707

2 files changed, 113 insertions, 15 deletions

diff --git a/mail/ChangeLog b/mail/ChangeLog
index f78038640b..0f62287f5e 100644
--- a/mail/ChangeLog
+++ b/mail/ChangeLog
@@ -1,3 +1,15 @@
+2001-07-02  Larry Ewing  <lewing@ximian.com>
+
+	* mail-display.c (save_part): move the saving logic out of save_cb
+	so that we can reuse it for the image saving code.
+	(save_cb): call save_part.
+	(save_url): new function to resolve an save an html url.
+	(image_save_as): save function for images and.  Add it to the
+	link_menu, and add MASK_URL and MASK_SRC to the structure so that
+	we cans show the proper menus.
+	(html_button_press_event): call get_src as well to find external
+	urls references.  Add popup masking.
+
 2001-07-02  Peter Williams  <peterw@ximian.com>
 
 	* README.async: Add a warning about how wrong this document is.
diff --git a/mail/mail-display.c b/mail/mail-display.c
index 78fe4f2022..a2f5d6eaa5 100644
--- a/mail/mail-display.c
+++ b/mail/mail-display.c
@@ -115,12 +115,15 @@ write_data_to_file (CamelMimePart *part, const char *name, gboolean unique)
 }
 
 static char *
-make_safe_filename (const char *prefix, CamelMimePart *part)
+make_safe_filename (const char *prefix,CamelMimePart *part)
 {
 	const char *name = NULL;
 	char *safe, *p;
 
-	name = camel_mime_part_get_filename (part);
+	if (part) {
+		name = camel_mime_part_get_filename (part);
+	}
+		
 	if (!name) {
 		/* This is a filename. Translators take note. */
 		name = _("attachment");
@@ -188,13 +191,12 @@ on_link_clicked (GtkHTML *html, const char *url, MailDisplay *md)
 		gnome_url_show (url);
 }
 
-static void
-save_cb (GtkWidget *widget, gpointer user_data)
+static void 
+save_part (CamelMimePart *part)
 {
-	CamelMimePart *part = gtk_object_get_data (user_data, "CamelMimePart");
 	GtkFileSelection *file_select;
 	char *filename;
-	
+
 	filename = make_safe_filename (g_get_home_dir (), part);
 	file_select = GTK_FILE_SELECTION (
 		gtk_file_selection_new (_("Save Attachment")));
@@ -212,6 +214,14 @@ save_cb (GtkWidget *widget, gpointer user_data)
 }
 
 static void
+save_cb (GtkWidget *widget, gpointer user_data)
+{
+	CamelMimePart *part = gtk_object_get_data (GTK_OBJECT (user_data), "CamelMimePart");
+
+	save_part (part);
+}
+
+static void
 launch_cb (GtkWidget *widget, gpointer user_data)
 {
 	CamelMimePart *part = gtk_object_get_data (user_data, "CamelMimePart");
@@ -641,6 +651,44 @@ get_embedded_for_component (const char *iid, MailDisplay *md)
 	return embedded;
 }
 
+static void *
+save_url (MailDisplay *md, const char *url)
+{
+	GHashTable *urls;
+	GByteArray *ba;
+	CamelMimePart *part;
+
+	urls = g_datalist_get_data (md->data, "part_urls");
+	g_return_val_if_fail (urls != NULL, NULL);
+
+	part = g_hash_table_lookup (urls, url);
+	if (part) {
+		CamelDataWrapper *data;
+
+		g_return_val_if_fail (CAMEL_IS_MIME_PART (part), NULL);
+
+		data = camel_medium_get_content_object ((CamelMedium *)part);
+		if (!mail_content_loaded (data, md)) {
+			return NULL;
+		}
+
+		save_part (part);
+		return NULL;
+	}
+	
+	g_warning ("part not found");
+#if 0
+	urls = g_datalist_get_data (md->data, "data_urls");
+	g_return_val_if_fail (urls != NULL, NULL);
+
+	/* See if it's some piece of cached data */
+	ba = g_hash_table_lookup (urls, url);
+	if (ba) {
+		return ba;
+	}
+#endif
+	return NULL;
+}
 
 static gboolean
 on_object_requested (GtkHTML *html, GtkHTMLEmbedded *eb, gpointer data)
@@ -1260,7 +1308,7 @@ link_open_in_browser (GtkWidget *w, MailDisplay *mail_display)
 static void
 link_save_as (GtkWidget *w, MailDisplay *mail_display)
 {
-	g_print ("FIXME\n");
+	g_print ("FIXME save %s\n", mail_display->html->pointer_url);
 }
 
 static void
@@ -1273,17 +1321,43 @@ link_copy_location (GtkWidget *w, MailDisplay *mail_display)
 		g_warning ("Damn");
 }
 
+static void
+image_save_as (GtkWidget *w, MailDisplay *mail_display)
+{
+	const char *src;
+
+	src = gtk_object_get_data (GTK_OBJECT (mail_display), "current_src_uri");
+
+	g_warning ("loading uri=%s", src);
+	
+	save_url (mail_display, src);
+}
+
+enum {
+	/* 
+	 * This is used to mask the link specific menu items.
+	 */
+	MASK_URL = 1,
+
+	/*
+	 * This is used to mask src specific menu items.
+	 */
+	MASK_SRC = 2
+};
+
 #define SEPARATOR  { "", NULL, (NULL), NULL,  0 }
 #define TERMINATOR { NULL, NULL, (NULL), NULL,  0 }
 
 static EPopupMenu link_menu [] = {
 	{ N_("Open Link in Browser"), NULL,
-	  GTK_SIGNAL_FUNC (link_open_in_browser),  NULL,  0 },
-	{ N_("Save as (FIXME)"), NULL,
-	  GTK_SIGNAL_FUNC (link_save_as), NULL,  0 },
+	  GTK_SIGNAL_FUNC (link_open_in_browser),  NULL,  MASK_URL },
 	{ N_("Copy Link Location"), NULL,
-	  GTK_SIGNAL_FUNC (link_copy_location), NULL,  0 },
-	
+	  GTK_SIGNAL_FUNC (link_copy_location), NULL,  MASK_URL },
+	{ N_("Save Link as (FIXME)"), NULL,
+	  GTK_SIGNAL_FUNC (link_save_as), NULL,  MASK_URL },
+	{ N_("Save Image as"), NULL,
+	  GTK_SIGNAL_FUNC (image_save_as), NULL, MASK_SRC }, 
+     
 	TERMINATOR
 };
 
@@ -1494,8 +1568,10 @@ html_button_press_event (GtkWidget *widget, GdkEventButton *event, MailDisplay *
 			
 			if (point) {
 				const gchar *url;
-				
+				const gchar *src;
+
 				url = html_object_get_url (point->object);
+				src = html_object_get_src (point->object);
 
 				if (url && !g_strncasecmp (url, "mailto:", 7)) {
 
@@ -1517,9 +1593,19 @@ html_button_press_event (GtkWidget *widget, GdkEventButton *event, MailDisplay *
 
 					
 
-				} else if (url) {
+				} else if (url || src) {
+				        gint hide_mask = 0;
 
-					e_popup_menu_run (link_menu, (GdkEvent *) event, 0, 0, mail_display);
+					if (!url)
+						hide_mask |= MASK_URL;
+
+					if (!src)
+						hide_mask |= MASK_SRC;
+
+					g_free (gtk_object_get_data (GTK_OBJECT (mail_display), "current_src_uri"));
+					gtk_object_set_data (GTK_OBJECT (mail_display), "current_src_uri", g_strdup (src));
+					
+					e_popup_menu_run (link_menu, (GdkEvent *) event, 0, hide_mask, mail_display);
 
 				}