Make escape close current view in gtk
[modest] / src / widgets / modest-gtkhtml-msg-view.c
index a73e6fc..416bdf7 100644 (file)
 #include <widgets/modest-ui-constants.h>
 #include <modest-icon-names.h>
 
-/* FIXNE: we should have no maemo-deps in widgets/ */
-#ifndef MODEST_TOOLKIT_GTK
-#include "maemo/modest-hildon-includes.h"
-#endif /*!MODEST_TOOLKIT_GTK*/
-
-
 /* 'private'/'protected' functions */
 static void     modest_gtkhtml_msg_view_class_init   (ModestGtkhtmlMsgViewClass *klass);
 static void     tny_header_view_init (gpointer g, gpointer iface_data);
@@ -270,6 +264,11 @@ struct _ModestGtkhtmlMsgViewPrivate {
        /* id handler for dragged scroll */
        guint idle_motion_id;
 
+       /* idle changes count */
+       gint idle_changes_count;
+       guint idle_readjust_scroll_id;
+       guint idle_resize_children_id;
+
        /* zoom */
        gdouble current_zoom;
 
@@ -908,6 +907,12 @@ size_allocate (GtkWidget *widget,
             allocation->height != widget->allocation.height))
                gdk_window_invalidate_rect (widget->window, NULL, FALSE);
 
+       if (priv->idle_readjust_scroll_id == 0 && priv->idle_changes_count < 5 && widget->allocation.width != allocation->width) {
+               g_object_ref (self);
+               priv->idle_readjust_scroll_id = g_idle_add ((GSourceFunc) idle_readjust_scroll, self);
+               priv->idle_changes_count ++;
+       }
+
        widget->allocation = *allocation;
        set_hadjustment_values (self, &hadj_value_changed);
        set_vadjustment_values (self, &vadj_value_changed);
@@ -925,9 +930,14 @@ size_allocate (GtkWidget *widget,
        html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
 
        html_allocation.x = 0;
-       html_allocation.y = headers_allocation.height;
+       html_allocation.y = MAX (0, headers_allocation.height - priv->vadj->value);
        html_allocation.width = view_allocation.width;
-       html_allocation.height = MAX ((gint) html_vadj->upper, (gint)(priv->vadj->upper - headers_allocation.height));
+       if (html_vadj->upper < allocation->height) {
+               html_allocation.height = MAX (html_vadj->upper, allocation->height - headers_allocation.height);
+       } else {
+               html_allocation.height = allocation->height;
+       }
+       /* html_allocation.height = MAX ((gint) html_vadj->upper, (gint)(priv->vadj->upper - headers_allocation.height)); */
 
        if (GTK_WIDGET_REALIZED (widget)) {
                gdk_window_move_resize (widget->window,
@@ -948,7 +958,7 @@ size_allocate (GtkWidget *widget,
                                        headers_allocation.height);
                gdk_window_move_resize (priv->html_window,
                                        (gint) (- priv->hadj->value),
-                                       (gint) (html_allocation.y - priv->vadj->value),
+                                       (gint) html_allocation.y,
                                        (gint) priv->hadj->upper,
                                        html_allocation.height);
        }
@@ -960,8 +970,11 @@ size_allocate (GtkWidget *widget,
                html_allocation.x = 0;
                html_allocation.y = 0;
                html_allocation.width = (gint) priv->hadj->upper;
-               html_allocation.height = (gint) priv->vadj->upper - headers_allocation.height;
                gtk_widget_size_allocate (priv->html_scroll, &html_allocation);
+               if (html_vadj->page_size != html_allocation.height) {
+                       html_vadj->page_size = html_allocation.height;
+                       gtk_adjustment_changed (html_vadj);
+               }
        }
        gtk_adjustment_changed (priv->hadj);
        gtk_adjustment_changed (priv->vadj);
@@ -1010,9 +1023,15 @@ adjustment_value_changed (GtkAdjustment *adj, gpointer data)
                if (priv->html_scroll && GTK_WIDGET_VISIBLE (priv->html_scroll)) {
                        gint old_x, old_y;
                        gint new_x, new_y;
+                       gint new_internal_vvalue;
                        gdk_window_get_position (priv->html_window, &old_x, &old_y);
                        new_x = -hadj->value;
-                       new_y = headers_offset - vadj->value;
+
+                       new_internal_vvalue = MAX (0, vadj->value - headers_offset);
+                       gtk_adjustment_set_value (gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll)),
+                                                 new_internal_vvalue);
+
+                       new_y = MAX (0, headers_offset - vadj->value);
 
                        if (new_x != old_x || new_y != old_y) {
                                gdk_window_move (priv->html_window, new_x, new_y);
@@ -1023,6 +1042,18 @@ adjustment_value_changed (GtkAdjustment *adj, gpointer data)
        }
 }
 
+static gboolean
+resize_children_idle (GtkContainer *cont)
+{
+       ModestGtkhtmlMsgViewPrivate *priv = MODEST_GTKHTML_MSG_VIEW_GET_PRIVATE (cont);
+       if (GTK_WIDGET_DRAWABLE (cont)) {
+               gtk_container_resize_children (cont);
+       }
+       priv->idle_resize_children_id = 0;
+
+       return FALSE;
+}
+
 static void
 html_adjustment_changed (GtkAdjustment *adj,
                         gpointer userdata)
@@ -1036,19 +1067,25 @@ html_adjustment_changed (GtkAdjustment *adj,
        g_signal_stop_emission_by_name (G_OBJECT (adj), "changed");
 
        priv->html_scroll->requisition.height = html_vadj->upper;
-       if (html_vadj->upper == priv->html_scroll->allocation.height)
+       if ((priv->vadj != NULL && (html_vadj->upper == (priv->vadj->upper - GTK_WIDGET (priv->headers_box)->allocation.height)))
+           && html_vadj->page_size == priv->html_scroll->allocation.height)
                return;
-       priv->html_scroll->allocation.height = html_vadj->upper;
 
        set_vadjustment_values (self, &vadj_changed);
 
-       new_height = MAX ((gint) html_vadj->upper, (gint) (priv->vadj->upper - priv->headers_box->allocation.height));
+       if (html_vadj->upper < GTK_WIDGET (self)->allocation.height) {
+               new_height = MAX (html_vadj->upper, 
+                                       GTK_WIDGET (self)->allocation.height - GTK_WIDGET(priv->headers_box)->allocation.height);
+       } else {
+               new_height = GTK_WIDGET (self)->allocation.height;
+       }
        
        gtk_adjustment_changed (priv->vadj);
        if (GTK_WIDGET_DRAWABLE (priv->html_scroll)) {
                gdk_window_resize (priv->html_window, (gint) priv->hadj->upper, (gint) new_height);
                gdk_window_process_updates (priv->view_window, TRUE);
-               gtk_container_resize_children (GTK_CONTAINER (self));
+               if (priv->idle_resize_children_id == 0)
+                       priv->idle_resize_children_id = gdk_threads_add_idle ((GSourceFunc) resize_children_idle, self);
        }
        
 }
@@ -1056,27 +1093,42 @@ html_adjustment_changed (GtkAdjustment *adj,
 gboolean
 idle_readjust_scroll (ModestGtkhtmlMsgView *self)
 {
+       ModestGtkhtmlMsgViewPrivate *priv = MODEST_GTKHTML_MSG_VIEW_GET_PRIVATE (self);
 
        /* We're out the main lock */
        gdk_threads_enter ();
 
        if (GTK_WIDGET_DRAWABLE (self)) {
-               ModestGtkhtmlMsgViewPrivate *priv = MODEST_GTKHTML_MSG_VIEW_GET_PRIVATE (self);
                GtkAdjustment *html_vadj;
+               GtkAdjustment *html_hadj;
+
                html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
-               html_vadj->page_size = html_vadj->upper;
-               gtk_adjustment_changed (html_vadj);
-               gtk_widget_queue_resize (GTK_WIDGET (self));
-               gtk_widget_queue_draw (GTK_WIDGET (self));
+               html_vadj->upper = 0;
+               html_vadj->page_size = 0;
+               g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
+
+               html_hadj = gtk_scrolled_window_get_hadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+               html_hadj->upper = 0;
+               html_hadj->page_size = 0;
+               g_signal_emit_by_name (G_OBJECT (html_hadj), "changed");
+
+               gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
+               gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
+
+               gtk_widget_queue_resize (GTK_WIDGET (priv->body_view));
+               gtk_widget_queue_draw (GTK_WIDGET (priv->body_view));
 
                /* Just another hack for making readjust really work. This forces an update
                 * of the scroll, and then, make the scroll really update properly the
                 * the size and not corrupt scrollable area */
                gtk_adjustment_set_value (priv->vadj, 1.0);
                gtk_adjustment_set_value (priv->vadj, 0.0);
+
        }
+       priv->idle_readjust_scroll_id = 0;
        g_object_unref (G_OBJECT (self));
 
+
        gdk_threads_leave ();
 
        return FALSE;
@@ -1087,6 +1139,9 @@ modest_gtkhtml_msg_view_init (ModestGtkhtmlMsgView *obj)
 {
        ModestGtkhtmlMsgViewPrivate *priv;
        GtkAdjustment *html_vadj;
+#ifndef MODEST_TOOLKIT_HILDON2
+       GtkWidget *separator;
+#endif
 
        GTK_WIDGET_UNSET_FLAGS (obj, GTK_NO_WINDOW);
        gtk_widget_set_redraw_on_allocate (GTK_WIDGET (obj), TRUE);
@@ -1095,6 +1150,9 @@ modest_gtkhtml_msg_view_init (ModestGtkhtmlMsgView *obj)
        
        priv = MODEST_GTKHTML_MSG_VIEW_GET_PRIVATE(obj);
 
+       priv->idle_changes_count = 0;
+       priv->idle_readjust_scroll_id = 0;
+       priv->idle_resize_children_id = 0;
        priv->current_zoom = 1.0;
 
        priv->hadj = NULL;
@@ -1228,6 +1286,16 @@ modest_gtkhtml_msg_view_finalize (GObject *obj)
                priv->msg = NULL;
        }
 
+       if (priv->idle_resize_children_id > 0) {
+               g_source_remove (priv->idle_resize_children_id);
+               priv->idle_resize_children_id = 0;
+       }
+
+       if (priv->idle_readjust_scroll_id > 0) {
+               g_source_remove (priv->idle_readjust_scroll_id);
+               priv->idle_readjust_scroll_id = 0;
+       }
+
        if (priv->idle_motion_id > 0) {
                g_source_remove (priv->idle_motion_id);
                priv->idle_motion_id = 0;
@@ -1701,7 +1769,7 @@ set_message (ModestGtkhtmlMsgView *self, TnyMsg *msg, TnyMimePart *other_body)
        TnyMimePart *body;
        ModestGtkhtmlMsgViewPrivate *priv;
        TnyHeader *header;
-       GtkAdjustment *html_vadj;
+       GtkAdjustment *html_vadj, *html_hadj;
 
        g_return_if_fail (self);
 
@@ -1710,15 +1778,6 @@ set_message (ModestGtkhtmlMsgView *self, TnyMsg *msg, TnyMimePart *other_body)
        gtk_widget_set_no_show_all (priv->mail_header_view, FALSE);
        modest_mime_part_view_set_view_images (MODEST_MIME_PART_VIEW (priv->body_view), FALSE);
 
-       html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
-       html_vadj->upper = 0;
-       html_vadj->page_size = 0;
-       g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
-
-       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
-       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
-
-
        if (msg != priv->msg) {
                if (priv->msg)
                        g_object_unref (G_OBJECT(priv->msg));
@@ -1737,6 +1796,23 @@ set_message (ModestGtkhtmlMsgView *self, TnyMsg *msg, TnyMimePart *other_body)
 #endif
                gtk_widget_set_no_show_all (priv->mail_header_view, TRUE);
                tny_mime_part_view_clear (TNY_MIME_PART_VIEW (priv->body_view));
+
+               html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+               html_vadj->upper = 0;
+               html_vadj->page_size = 0;
+               g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
+
+               html_hadj = gtk_scrolled_window_get_hadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+               html_hadj->upper = 0;
+               html_hadj->page_size = 0;
+               g_signal_emit_by_name (G_OBJECT (html_hadj), "changed");
+
+               gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
+               gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
+
+               priv->idle_changes_count = 0;
+               gtk_widget_queue_resize (GTK_WIDGET (priv->body_view));
+
                gtk_widget_queue_resize (GTK_WIDGET(self));
                gtk_widget_queue_draw (GTK_WIDGET(self));
                return;
@@ -1804,6 +1880,23 @@ set_message (ModestGtkhtmlMsgView *self, TnyMsg *msg, TnyMimePart *other_body)
        gtk_widget_set_no_show_all (priv->priority_box, FALSE);
 #endif
        gtk_widget_set_no_show_all (priv->mail_header_view, TRUE);
+
+       html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+       html_vadj->upper = 0;
+       html_vadj->page_size = 0;
+       g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
+
+       html_hadj = gtk_scrolled_window_get_hadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+       html_hadj->upper = 0;
+       html_hadj->page_size = 0;
+       g_signal_emit_by_name (G_OBJECT (html_hadj), "changed");
+
+       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
+       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
+
+       priv->idle_changes_count = 0;
+       gtk_widget_queue_resize (GTK_WIDGET (priv->body_view));
+
        gtk_widget_queue_resize (GTK_WIDGET(self));
        gtk_widget_queue_draw (GTK_WIDGET(self));
 
@@ -1817,15 +1910,17 @@ set_message (ModestGtkhtmlMsgView *self, TnyMsg *msg, TnyMimePart *other_body)
        /* This is a hack to force reallocation of scroll after drawing all the stuff. This
         * makes the html view get the proper and expected size and prevent being able to scroll
         * the buffer when it shouldn't be scrollable */
-       g_object_ref (self);
-       g_timeout_add (250, (GSourceFunc) idle_readjust_scroll, self);
+       if (priv->idle_readjust_scroll_id == 0) {
+               g_object_ref (self);
+               priv->idle_readjust_scroll_id = g_idle_add ((GSourceFunc) idle_readjust_scroll, self);
+       }
 }
 
 static void
 set_header (ModestGtkhtmlMsgView *self, TnyHeader *header)
 {
        ModestGtkhtmlMsgViewPrivate *priv;
-       GtkAdjustment *html_vadj;
+       GtkAdjustment *html_vadj, *html_hadj;
        
        g_return_if_fail (self);
 
@@ -1844,9 +1939,15 @@ set_header (ModestGtkhtmlMsgView *self, TnyHeader *header)
        html_vadj->page_size = 0;
        g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
 
+       html_hadj = gtk_scrolled_window_get_hadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+       html_hadj->upper = 0;
+       html_hadj->page_size = 0;
+       g_signal_emit_by_name (G_OBJECT (html_hadj), "changed");
+
+       priv->idle_changes_count = 0;
        gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
        gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
-
+       gtk_widget_queue_resize (GTK_WIDGET (priv->body_view));
 
        if (priv->msg) {
                g_object_unref (G_OBJECT(priv->msg));
@@ -1862,6 +1963,7 @@ set_header (ModestGtkhtmlMsgView *self, TnyHeader *header)
 #endif
        gtk_widget_set_no_show_all (priv->mail_header_view, TRUE);
        tny_mime_part_view_clear (TNY_MIME_PART_VIEW (priv->body_view));
+       priv->idle_changes_count = 0;
        gtk_widget_queue_resize (GTK_WIDGET(self));
        gtk_widget_queue_draw (GTK_WIDGET(self));
 }
@@ -1894,12 +1996,27 @@ static void
 set_zoom (ModestGtkhtmlMsgView *self, gdouble zoom)
 {
        ModestGtkhtmlMsgViewPrivate *priv;
+       GtkAdjustment *html_vadj, *html_hadj;
 
        g_return_if_fail (MODEST_IS_GTKHTML_MSG_VIEW (self));
        priv = MODEST_GTKHTML_MSG_VIEW_GET_PRIVATE (self);
 
        modest_zoomable_set_zoom (MODEST_ZOOMABLE(priv->body_view), zoom);
 
+       html_vadj = gtk_scrolled_window_get_vadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+       html_vadj->upper = 0;
+       html_vadj->page_size = 0;
+       g_signal_emit_by_name (G_OBJECT (html_vadj), "changed");
+
+       html_hadj = gtk_scrolled_window_get_hadjustment (GTK_SCROLLED_WINDOW (priv->html_scroll));
+       html_hadj->upper = 0;
+       html_hadj->page_size = 0;
+       g_signal_emit_by_name (G_OBJECT (html_hadj), "changed");
+
+       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), 1, 1);
+       gtk_widget_set_size_request (GTK_WIDGET (priv->body_view), -1, -1);
+
+       priv->idle_changes_count = 0;
        gtk_widget_queue_resize (priv->body_view);
 }