X-Git-Url: https://www.fi.muni.cz/~kas/git//home/kas/public_html/git/?a=blobdiff_plain;ds=sidebyside;f=shell%2Fev-view.c;h=8998a7f161fb1050df1086836526d6c00b244243;hb=2bc43f51aad785bd5bddeb4e743756894f3e0b1c;hp=41bf94b60e96550d83fe15040bf92760d57f0ebe;hpb=88ec7e43a81072ea4d48ce91e6bf018a14c311a6;p=evince.git

diff --git a/shell/ev-view.c b/shell/ev-view.c
index 41bf94b6..8998a7f1 100644
--- a/shell/ev-view.c
+++ b/shell/ev-view.c
@@ -29,6 +29,8 @@
 #include "ev-marshal.h"
 #include "ev-view.h"
 #include "ev-document-find.h"
+#include "ev-document-misc.h"
+#include "ev-debug.h"
 
 #define EV_VIEW_CLASS(klass)    (G_TYPE_CHECK_CLASS_CAST ((klass), EV_TYPE_VIEW, EvViewClass))
 #define EV_IS_VIEW_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE ((klass), EV_TYPE_VIEW))
@@ -55,11 +57,29 @@ static const GtkTargetEntry targets[] = {
 	{ "UTF8_STRING", 0, TARGET_UTF8_STRING },
 };
 
+typedef enum {
+	EV_VIEW_CURSOR_NORMAL,
+	EV_VIEW_CURSOR_LINK,
+	EV_VIEW_CURSOR_WAIT,
+	EV_VIEW_CURSOR_HIDDEN
+} EvViewCursor;
+
+#define ZOOM_IN_FACTOR  1.2
+#define ZOOM_OUT_FACTOR (1.0/ZOOM_IN_FACTOR)
+
+#define MIN_SCALE 0.05409
+#define MAX_SCALE 18.4884
+
+/* FIXME: temporarily setting the epsilon very high until we figure out how to
+ * constrain the size of the window to a pixel width, instead of to the zoom
+ * level */
+#define ZOOM_EPSILON 1e-2
+
+
 struct _EvView {
 	GtkWidget parent_instance;
 
 	EvDocument *document;
-	EvHistory  *history;
 	
 	GdkWindow *bin_window;
 
@@ -71,17 +91,19 @@ struct _EvView {
 
 	gboolean pressed_button;
 	gboolean has_selection;
+	GdkPoint selection_start;
 	GdkRectangle selection;
+	EvViewCursor cursor;
 
 	GtkAdjustment *hadjustment;
 	GtkAdjustment *vadjustment;
 
-        GArray *find_results;
-	int results_on_this_page;
-	int next_page_with_result;
-	double find_percent_complete;
+	int find_page;
+	int find_result;
+	int spacing;
 
 	double scale;
+	EvSizingMode sizing_mode;
 };
 
 struct _EvViewClass {
@@ -177,17 +199,13 @@ ev_view_finalize (GObject *object)
 {
 	EvView *view = EV_VIEW (object);
 
+	LOG ("Finalize");
+
 	if (view->document)
 		g_object_unref (view->document);
 
-	if (view->history)
-		g_object_unref (view->history);
-
 	ev_view_set_scroll_adjustments (view, NULL, NULL);
 
-        g_array_free (view->find_results, TRUE);
-        view->find_results = NULL;
-        
 	G_OBJECT_CLASS (ev_view_parent_class)->finalize (object);
 }
 
@@ -206,18 +224,39 @@ ev_view_size_request (GtkWidget      *widget,
 		      GtkRequisition *requisition)
 {
 	EvView *view = EV_VIEW (widget);
+	GtkBorder border;
+	gint width, height;
 
-	if (GTK_WIDGET_REALIZED (widget)) {
-		if (view->document) {
-			ev_document_get_page_size (view->document,
-						   &requisition->width,
-						   &requisition->height);
-		} else {
-			requisition->width = 10;
-			requisition->height = 10;
-		}
+	if (! GTK_WIDGET_REALIZED (widget))
+		return;
+
+	if (! view->document) {
+		requisition->width = 1;
+		requisition->height = 1;
+		return;
+	}
+
+	ev_document_get_page_size (view->document, -1,
+				   &width, &height);
+	ev_document_misc_get_page_border_size (width, height, &border);
+
+	switch (view->sizing_mode) {
+	case EV_SIZING_BEST_FIT:
+		requisition->width = MIN_SCALE * ((float) width) / view->scale;
+		requisition->height = MIN_SCALE * ((float) height) / view->scale;
+		break;
+	case EV_SIZING_FIT_WIDTH:
+		requisition->width = MIN_SCALE * ((float) width) / view->scale;
+		requisition->height = height + border.top + border.bottom;
+		requisition->height += view->spacing * 2;
+		break;
+	case EV_SIZING_FREE:
+		requisition->width = width + border.left + border.right;
+		requisition->height = height + border.top + border.bottom;
+		requisition->width += view->spacing * 2;
+		requisition->height += view->spacing * 2;
+		break;
 	}
-  
 }
 
 static void
@@ -238,17 +277,6 @@ ev_view_size_allocate (GtkWidget      *widget,
 	}
 }
 
-static void
-update_window_backgrounds (EvView *view)
-{
-	GtkWidget *widget = GTK_WIDGET (view);
-  
-	if (GTK_WIDGET_REALIZED (view)) {
-		gdk_window_set_background (view->bin_window,
-					   &widget->style->base[GTK_WIDGET_STATE (widget)]);
-	}
-}
-
 static void
 ev_view_realize (GtkWidget *widget)
 {
@@ -257,6 +285,7 @@ ev_view_realize (GtkWidget *widget)
 
 	GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
   
+
 	attributes.window_type = GDK_WINDOW_CHILD;
 	attributes.wclass = GDK_INPUT_OUTPUT;
 	attributes.visual = gtk_widget_get_visual (widget);
@@ -275,6 +304,7 @@ ev_view_realize (GtkWidget *widget)
 					 GDK_WA_VISUAL);
 	gdk_window_set_user_data (widget->window, widget);
 	widget->style = gtk_style_attach (widget->style, widget->window);
+	gdk_window_set_background (widget->window, &widget->style->mid[widget->state]);
   
 	attributes.x = 0;
 	attributes.y = 0;
@@ -285,7 +315,8 @@ ev_view_realize (GtkWidget *widget)
 				GDK_BUTTON_RELEASE_MASK |
 				GDK_SCROLL_MASK |
 				GDK_KEY_PRESS_MASK |
-				GDK_POINTER_MOTION_MASK;
+				GDK_POINTER_MOTION_MASK |
+		                GDK_LEAVE_NOTIFY_MASK;
   
 	view->bin_window = gdk_window_new (widget->window,
 					   &attributes,
@@ -295,6 +326,9 @@ ev_view_realize (GtkWidget *widget)
 	gdk_window_set_user_data (view->bin_window, widget);
 	gdk_window_show (view->bin_window);
 
+	widget->style = gtk_style_attach (widget->style, view->bin_window);
+	gdk_window_set_background (view->bin_window, &widget->style->mid[widget->state]);
+
 	if (view->document) {
 		ev_document_set_target (view->document, view->bin_window);
 
@@ -305,8 +339,6 @@ ev_view_realize (GtkWidget *widget)
 		 */
 		gtk_widget_queue_resize (widget);
 	}
-
-	update_window_backgrounds (view);
 }
 
 static void
@@ -324,20 +356,6 @@ ev_view_unrealize (GtkWidget *widget)
 	GTK_WIDGET_CLASS (ev_view_parent_class)->unrealize (widget);
 }
 
-static void
-ev_view_style_set (GtkWidget      *widget,
-		   GtkStyle       *previous_style)
-{
-	update_window_backgrounds (EV_VIEW (widget));
-}
-
-static void
-ev_view_state_changed (GtkWidget    *widget,
-		       GtkStateType  previous_state)
-{
-	update_window_backgrounds (EV_VIEW (widget));
-}
-
 static guint32
 ev_gdk_color_to_rgb (const GdkColor *color)
 {
@@ -349,7 +367,8 @@ ev_gdk_color_to_rgb (const GdkColor *color)
 }
 
 static void
-draw_rubberband (GtkWidget *widget, GdkWindow *window, const GdkRectangle *rect)
+draw_rubberband (GtkWidget *widget, GdkWindow *window,
+		 const GdkRectangle *rect, guchar alpha)
 {
 	GdkGC *gc;
 	GdkPixbuf *pixbuf;
@@ -357,7 +376,7 @@ draw_rubberband (GtkWidget *widget, GdkWindow *window, const GdkRectangle *rect)
 	guint fill_color;
 
 	fill_color_gdk = gdk_color_copy (&GTK_WIDGET (widget)->style->base[GTK_STATE_SELECTED]);
-	fill_color = ev_gdk_color_to_rgb (fill_color_gdk) << 8 | 0x40;
+	fill_color = ev_gdk_color_to_rgb (fill_color_gdk) << 8 | alpha;
 
 	pixbuf = gdk_pixbuf_new (GDK_COLORSPACE_RGB, TRUE, 8,
 				 rect->width, rect->height);
@@ -383,42 +402,82 @@ draw_rubberband (GtkWidget *widget, GdkWindow *window, const GdkRectangle *rect)
 	gdk_color_free (fill_color_gdk);
 }
 
+static void
+highlight_find_results (EvView *view)
+{
+	EvDocumentFind *find;
+	int i, results;
+
+	g_return_if_fail (EV_IS_DOCUMENT_FIND (view->document));
+
+	find = EV_DOCUMENT_FIND (view->document);
+
+	results = ev_document_find_get_n_results (find);
+
+	for (i = 0; i < results; i++) {
+		GdkRectangle rectangle;
+		guchar alpha;
+
+		alpha = (i == view->find_result) ? 0x90 : 0x20;
+		ev_document_find_get_result (find, i, &rectangle);
+		draw_rubberband (GTK_WIDGET (view), view->bin_window,
+				 &rectangle, alpha);
+        }
+}
+
+
 static void
 expose_bin_window (GtkWidget      *widget,
 		   GdkEventExpose *event)
 {
 	EvView *view = EV_VIEW (widget);
-        int i;
-	int current_page;
-        const EvFindResult *results;
-
+	int x_offset, y_offset;
+	GtkBorder border;
+	gint width, height;
+	GdkRectangle area;
+	int target_width, target_height;
+			
 	if (view->document == NULL)
 		return;
+
+	ev_document_get_page_size (view->document, -1,
+				   &width, &height);
+	ev_document_misc_get_page_border_size (width, height, &border);
 	
+	x_offset = view->spacing;
+	y_offset = view->spacing;
+	target_width = width + border.left + border.right + view->spacing * 2;
+	target_height = height + border.top + border.bottom + view->spacing * 2;
+
+	x_offset += MAX (0, (widget->allocation.width - target_width) / 2);
+	y_offset += MAX (0, (widget->allocation.height - target_height) / 2);
+
+	/* Paint the frame */
+	area.x = x_offset;
+	area.y = y_offset;
+	area.width = width + border.left + border.right;
+	area.height = height + border.top + border.bottom;
+	ev_document_misc_paint_one_page (view->bin_window, widget, &area, &border);
+
+	/* Render the document itself */
+	ev_document_set_page_offset (view->document,
+				     x_offset + border.left,
+				     y_offset + border.top);
+
+	LOG ("Render area %d %d %d %d", event->area.x, event->area.y,
+             event->area.width, event->area.height);
+
 	ev_document_render (view->document,
 			    event->area.x, event->area.y,
 			    event->area.width, event->area.height);
 
-        results = (EvFindResult*) view->find_results->data;
-	current_page = ev_document_get_page (view->document);
-        i = 0;
-        while (i < view->find_results->len) {
-#if 0
-                g_printerr ("highlighting result %d page %d at %d,%d %dx%d\n",
-                            i, results[i].page_num,
-                            results[i].highlight_area.x,
-                            results[i].highlight_area.y,
-                            results[i].highlight_area.width,
-                            results[i].highlight_area.height);
-#endif
-		if (results[i].page_num == current_page)
-			draw_rubberband (widget, view->bin_window,
-					 &results[i].highlight_area);
-                ++i;
-        }
+	if (EV_IS_DOCUMENT_FIND (view->document)) {
+		highlight_find_results (view);
+	}
 
 	if (view->has_selection) {
-		draw_rubberband (widget, view->bin_window, &view->selection);
+		draw_rubberband (widget, view->bin_window,
+				 &view->selection, 0x40);
 	}
 }
 
@@ -527,10 +586,8 @@ ev_view_button_press_event (GtkWidget      *widget,
 				gtk_widget_queue_draw (widget);
 			}
 
-			view->selection.x = event->x;
-			view->selection.y = event->y;
-			view->selection.width = 0;
-			view->selection.height = 0;
+			view->selection_start.x = event->x;
+			view->selection_start.y = event->y;
 			break;
 	}
 
@@ -552,7 +609,7 @@ status_message_from_link (EvLink *link)
 			break;
 		case EV_LINK_TYPE_PAGE:
 			page = ev_link_get_page (link);
-			msg = g_strdup_printf (_("Page %d"), page);
+			msg = g_strdup_printf (_("Go to page %d"), page);
 			break;
 		case EV_LINK_TYPE_EXTERNAL_URI:
 			msg = g_strdup (ev_link_get_uri (link));
@@ -569,9 +626,11 @@ ev_view_set_status (EvView *view, const char *message)
 {
 	g_return_if_fail (EV_IS_VIEW (view));
 
-	g_free (view->status);
-	view->status = g_strdup (message);
-	g_object_notify (G_OBJECT (view), "status");
+	if (message != view->status) {
+		g_free (view->status);
+		view->status = g_strdup (message);
+		g_object_notify (G_OBJECT (view), "status");
+	}
 }
 
 static void
@@ -584,6 +643,56 @@ ev_view_set_find_status (EvView *view, const char *message)
 	g_object_notify (G_OBJECT (view), "find-status");
 }
 
+static GdkCursor *
+ev_view_create_invisible_cursor(void)
+{
+       GdkBitmap *empty;
+       GdkColor black = { 0, 0, 0, 0 };
+       static unsigned char bits[] = { 0x00 };
+
+       empty = gdk_bitmap_create_from_data (NULL, bits, 1, 1);
+
+       return gdk_cursor_new_from_pixmap (empty, empty, &black, &black, 0, 0);
+}
+
+static void
+ev_view_set_cursor (EvView *view, EvViewCursor new_cursor)
+{
+	GdkCursor *cursor = NULL;
+	GdkDisplay *display;
+	GtkWidget *widget;
+
+	if (view->cursor == new_cursor) {
+		return;
+	}
+
+	widget = gtk_widget_get_toplevel (GTK_WIDGET (view));
+	display = gtk_widget_get_display (widget);
+	view->cursor = new_cursor;
+
+	switch (new_cursor) {
+		case EV_VIEW_CURSOR_NORMAL:
+			gdk_window_set_cursor (widget->window, NULL);
+			break;
+		case EV_VIEW_CURSOR_LINK:
+			cursor = gdk_cursor_new_for_display (display, GDK_HAND2);
+			break;
+		case EV_VIEW_CURSOR_WAIT:
+			cursor = gdk_cursor_new_for_display (display, GDK_WATCH);
+			break;
+                case EV_VIEW_CURSOR_HIDDEN:
+                        cursor = ev_view_create_invisible_cursor ();
+                        break;
+
+	}
+
+	if (cursor) {
+		gdk_window_set_cursor (widget->window, cursor);
+		gdk_cursor_unref (cursor);
+		gdk_flush();
+	}
+}
+
 static gboolean
 ev_view_motion_notify_event (GtkWidget      *widget,
 			     GdkEventMotion *event)
@@ -592,10 +701,12 @@ ev_view_motion_notify_event (GtkWidget      *widget,
 
 	if (view->pressed_button > 0) {
 		view->has_selection = TRUE;
-		view->selection.x = MIN (view->selection.x, event->x);
-		view->selection.y = MIN (view->selection.y, event->y);
-		view->selection.width = ABS (view->selection.x - event->x) + 1;
-		view->selection.height = ABS (view->selection.y - event->y) + 1;
+		view->selection.x = MIN (view->selection_start.x, event->x);
+		view->selection.y = MIN (view->selection_start.y, event->y);
+		view->selection.width = ABS (view->selection_start.x - event->x) + 1;
+		view->selection.height = ABS (view->selection_start.y - event->y) + 1;
+
+		gtk_widget_queue_draw (widget);
 	} else if (view->document) {
 		EvLink *link;
 
@@ -605,14 +716,18 @@ ev_view_motion_notify_event (GtkWidget      *widget,
 
 			msg = status_message_from_link (link);
 			ev_view_set_status (view, msg);
+			ev_view_set_cursor (view, EV_VIEW_CURSOR_LINK);
 			g_free (msg);
 
                         g_object_unref (link);
-                }
+		} else {
+			ev_view_set_status (view, NULL);
+			if (view->cursor == EV_VIEW_CURSOR_LINK) {
+				ev_view_set_cursor (view, EV_VIEW_CURSOR_NORMAL);
+			}
+		}
 	}
 
-	gtk_widget_queue_draw (widget);
-
 	return TRUE;
 }
 
@@ -801,8 +916,6 @@ ev_view_class_init (EvViewClass *class)
 	widget_class->size_allocate = ev_view_size_allocate;
 	widget_class->realize = ev_view_realize;
 	widget_class->unrealize = ev_view_unrealize;
-	widget_class->style_set = ev_view_style_set;
-	widget_class->state_changed = ev_view_state_changed;
 	gtk_object_class->destroy = ev_view_destroy;
 
 	class->set_scroll_adjustments = ev_view_set_scroll_adjustments;
@@ -865,147 +978,184 @@ ev_view_class_init (EvViewClass *class)
 static void
 ev_view_init (EvView *view)
 {
-	static const GdkColor white = { 0, 0xffff, 0xffff, 0xffff };
-
 	GTK_WIDGET_SET_FLAGS (view, GTK_CAN_FOCUS);
 
+	view->spacing = 10;
 	view->scale = 1.0;
 	view->pressed_button = -1;
-	
-	gtk_widget_modify_bg (GTK_WIDGET (view), GTK_STATE_NORMAL, &white);
-
-        view->find_results = g_array_new (FALSE,
-                                          FALSE,
-                                          sizeof (EvFindResult));
-	view->results_on_this_page = 0;
-	view->next_page_with_result = 0;
+	view->cursor = EV_VIEW_CURSOR_NORMAL;
+	view->sizing_mode = EV_SIZING_BEST_FIT;
 }
 
-static char *
-ev_view_get_find_status_message (EvView *view)
+static void
+update_find_status_message (EvView *view)
 {
-	if (view->find_results->len == 0) {
-		if (view->find_percent_complete >= (1.0 - 1e-10)) {
-			return g_strdup (_("Not found"));
+	char *message;
+
+	if (ev_document_get_page (view->document) == view->find_page) {
+		int results;
+
+		results = ev_document_find_get_n_results
+				(EV_DOCUMENT_FIND (view->document));
+
+		message = g_strdup_printf (_("%d found on this page"),
+					   results);
+	} else {
+		double percent;
+		
+		percent = ev_document_find_get_progress
+				(EV_DOCUMENT_FIND (view->document));
+
+		if (percent >= (1.0 - 1e-10)) {
+			message = g_strdup (_("Not found"));
 		} else {
-			return g_strdup_printf (_("%3d%% remaining to search"),
-						(int) ((1.0 - view->find_percent_complete) * 100));
+			message = g_strdup_printf (_("%3d%% remaining to search"),
+						   (int) ((1.0 - percent) * 100));
 		}
-	} else if (view->results_on_this_page == 0) {
-		g_assert (view->next_page_with_result != 0);
-		return g_strdup_printf (_("Found on page %d"),
-					view->next_page_with_result);
-	} else {
-		return g_strdup_printf (_("%d found on this page"),
-					view->results_on_this_page);
+		
 	}
+
+	ev_view_set_find_status (view, message);
+	g_free (message);
 }
 
 static void
-update_find_results (EvView *view)
+set_document_page (EvView *view, int new_page)
 {
-	const EvFindResult *results;
-	int i;
-	int on_this_page;
-	int next_page_with_result;
-	int earliest_page_with_result;
-	int current_page;
-	gboolean counts_changed;
-	
-	results = (EvFindResult*) view->find_results->data;
-	current_page = ev_document_get_page (view->document);
-	next_page_with_result = 0;
-	on_this_page = 0;
-	earliest_page_with_result = 0;
-	
-	i = 0;
-	while (i < view->find_results->len) {
-		if (results[i].page_num == current_page) {
-			++on_this_page;
-		} else {
-			int delta = results[i].page_num - current_page;
+	int page;
+	int pages;
+
+	pages = ev_document_get_n_pages (view->document);
+	page = CLAMP (new_page, 1, pages);
+
+	if (view->document) {
+		int old_page = ev_document_get_page (view->document);
+		int old_width, old_height;
+
+		ev_document_get_page_size (view->document,
+					   -1, 
+					   &old_width, &old_height);
+
+		if (old_page != page) {
+			if (view->cursor != EV_VIEW_CURSOR_HIDDEN) {
+				ev_view_set_cursor (view, EV_VIEW_CURSOR_WAIT);
+			}
+			ev_document_set_page (view->document, page);
+		}
+
+		if (old_page != ev_document_get_page (view->document)) {
+			int width, height;
 			
-			if (delta > 0 && /* result on later page */
-			    (next_page_with_result == 0 ||
-			     results[i].page_num < next_page_with_result))
-				next_page_with_result = results[i].page_num;
-
-			if (delta < 0 && /* result on a previous page */
-			    (earliest_page_with_result == 0 ||
-			     results[i].page_num < earliest_page_with_result))
-				earliest_page_with_result = results[i].page_num;
+			g_signal_emit (view, page_changed_signal, 0);
+
+			view->has_selection = FALSE;
+			ev_document_get_page_size (view->document,
+						   -1, 
+						   &width, &height);
+			if (width != old_width || height != old_height)
+				gtk_widget_queue_resize (GTK_WIDGET (view));
+
+			gtk_adjustment_set_value (view->vadjustment,
+						  view->vadjustment->lower);
+		}
+
+		if (EV_IS_DOCUMENT_FIND (view->document)) {
+			view->find_page = page;
+			view->find_result = 0;
+			update_find_status_message (view);
 		}
-		++i;
 	}
+}
 
-	/* If earliest page is just the current page, there is no earliest page */
-	if (earliest_page_with_result == current_page)
-		earliest_page_with_result = 0;
-	
-	/* If no next page, then wrap and the wrapped page is the next page */
-	if (next_page_with_result == 0)
-		next_page_with_result = earliest_page_with_result;
-
-	counts_changed = FALSE;
-	if (on_this_page != view->results_on_this_page ||
-	    next_page_with_result != view->next_page_with_result) {
-		view->results_on_this_page = on_this_page;
-		view->next_page_with_result = next_page_with_result;
-		counts_changed = TRUE;
+#define MARGIN 5
+
+static void
+ensure_rectangle_is_visible (EvView *view, GdkRectangle *rect)
+{
+	GtkWidget *widget = GTK_WIDGET (view);
+	GtkAdjustment *adjustment;
+	int value;
+
+	adjustment = view->vadjustment;
+
+	if (rect->y < adjustment->value) {
+		value = MAX (adjustment->lower, rect->y - MARGIN);
+		gtk_adjustment_set_value (view->vadjustment, value);
+	} else if (rect->y + rect->height >
+		   adjustment->value + widget->allocation.height) {
+		value = MIN (adjustment->upper, rect->y + rect->height -
+			     widget->allocation.height + MARGIN);
+		gtk_adjustment_set_value (view->vadjustment, value);
 	}
 
-	if (counts_changed ||
-	    view->find_results->len == 0) {
-		char *message;
+	adjustment = view->hadjustment;
 
-		message = ev_view_get_find_status_message (view);
-		ev_view_set_find_status (view, message);
-		g_free (message);
+	if (rect->x < adjustment->value) {
+		value = MAX (adjustment->lower, rect->x - MARGIN);
+		gtk_adjustment_set_value (view->hadjustment, value);
+	} else if (rect->x + rect->height >
+		   adjustment->value + widget->allocation.width) {
+		value = MIN (adjustment->upper, rect->x + rect->width -
+			     widget->allocation.width + MARGIN);
+		gtk_adjustment_set_value (view->hadjustment, value);
 	}
 }
 
 static void
-found_results_callback (EvDocument         *document,
-                        const EvFindResult *results,
-                        int                 n_results,
-                        double              percent_complete,
-                        void               *data)
+jump_to_find_result (EvView *view)
 {
-  EvView *view = EV_VIEW (data);
-  
-  g_array_set_size (view->find_results, 0);
-
-  if (n_results > 0)
-          g_array_append_vals (view->find_results,
-                               results, n_results);
-
-#if 0
-  {
-	  int i;
-
-	  g_printerr ("%d results %d%%: ", n_results,
-		      (int) (percent_complete * 100));
-	  i = 0;
-	  while (i < n_results) {
-		  g_printerr ("%d ", results[i].page_num);
-		  ++i;
-	  }
-	  g_printerr ("\n");
-  }
-#endif
-
-  view->find_percent_complete = percent_complete;
-  update_find_results (view);
-  
-  gtk_widget_queue_draw (GTK_WIDGET (view));
+	EvDocumentFind *find = EV_DOCUMENT_FIND (view->document);
+	GdkRectangle rect;
+	int n_results;
+
+	n_results = ev_document_find_get_n_results (find);
+
+	if (n_results > view->find_result) {
+		ev_document_find_get_result
+			(find, view->find_result, &rect);
+		ensure_rectangle_is_visible (view, &rect);
+	}
 }
 
-/*** Public API ***/       
-     
-GtkWidget*
-ev_view_new (void)
+static void
+jump_to_find_page (EvView *view)
 {
-	return g_object_new (EV_TYPE_VIEW, NULL);
+	int n_pages, i;
+
+	n_pages = ev_document_get_n_pages (view->document);
+
+	for (i = 0; i <= n_pages; i++) {
+		int has_results;
+		int page;
+
+		page = i + view->find_page;
+		if (page > n_pages) {
+			page = page - n_pages;
+		}
+
+		has_results = ev_document_find_page_has_results
+				(EV_DOCUMENT_FIND (view->document), page);
+		if (has_results == -1) {
+			view->find_page = page;
+			break;
+		} else if (has_results == 1) {
+			set_document_page (view, page);
+			jump_to_find_result (view);
+			break;
+		}
+	}
+}
+
+static void
+find_changed_cb (EvDocument *document, int page, EvView *view)
+{
+	jump_to_find_page (view);
+	jump_to_find_result (view);
+	update_find_status_message (view);
+
+	if (ev_document_get_page (document) == page) {
+		gtk_widget_queue_draw (GTK_WIDGET (view));
+	}
 }
 
 static void
@@ -1013,6 +1163,18 @@ document_changed_callback (EvDocument *document,
 			   EvView     *view)
 {
 	gtk_widget_queue_draw (GTK_WIDGET (view));
+
+	if (view->cursor != EV_VIEW_CURSOR_HIDDEN) {
+		ev_view_set_cursor (view, EV_VIEW_CURSOR_NORMAL);
+	}
+}
+
+/*** Public API ***/       
+     
+GtkWidget*
+ev_view_new (void)
+{
+	return g_object_new (EV_TYPE_VIEW, NULL);
 }
 
 void
@@ -1022,28 +1184,25 @@ ev_view_set_document (EvView     *view,
 	g_return_if_fail (EV_IS_VIEW (view));
 
 	if (document != view->document) {
-		int old_page = ev_view_get_page (view);
-		
 		if (view->document) {
                         g_signal_handlers_disconnect_by_func (view->document,
-                                                              found_results_callback,
+                                                              find_changed_cb,
                                                               view);
-                        g_array_set_size (view->find_results, 0);
-			view->results_on_this_page = 0;
-			view->next_page_with_result = 0;
-
 			g_object_unref (view->document);
                 }
 
 		view->document = document;
+		view->find_page = 1;
+		view->find_result = 0;
 
 		if (view->document) {
 			g_object_ref (view->document);
-			if (EV_IS_DOCUMENT_FIND (view->document))
+			if (EV_IS_DOCUMENT_FIND (view->document)) {
 				g_signal_connect (view->document,
-						  "found",
-						  G_CALLBACK (found_results_callback),
+						  "find_changed",
+						  G_CALLBACK (find_changed_cb),
 						  view);
+			}
 			g_signal_connect (view->document,
 					  "changed",
 					  G_CALLBACK (document_changed_callback),
@@ -1055,25 +1214,19 @@ ev_view_set_document (EvView     *view,
 		
 		gtk_widget_queue_resize (GTK_WIDGET (view));
 		
-		if (old_page != ev_view_get_page (view))
-			g_signal_emit (view, page_changed_signal, 0);
+		g_signal_emit (view, page_changed_signal, 0);
 	}
 }
 
-static void
-set_document_page (EvView *view, int page)
+void
+ev_view_set_mode (EvView       *view,
+		  EvSizingMode  sizing_mode)
 {
-	if (view->document) {
-		int old_page = ev_document_get_page (view->document);
-		if (old_page != page)
-			ev_document_set_page (view->document, page);
-		if (old_page != ev_document_get_page (view->document)) {
-			g_signal_emit (view, page_changed_signal, 0);
+	if (view->sizing_mode == sizing_mode)
+		return;
 
-			view->find_percent_complete = 0.0;
-			update_find_results (view);	
-		}
-	}
+	view->sizing_mode = sizing_mode;
+	gtk_widget_queue_resize (GTK_WIDGET (view));
 }
 
 static void
@@ -1102,69 +1255,16 @@ go_to_link (EvView *view, EvLink *link)
 void
 ev_view_go_to_link (EvView *view, EvLink *link)
 {
-	EvLinkType type;
-
 	go_to_link (view, link);
-
-	type = ev_link_get_link_type (link);
-	if (type == EV_LINK_TYPE_PAGE) {
-		ev_history_add_link (view->history, link);
-	}
-}
-
-static void
-go_to_index (EvView *view, int index)
-{
-	EvLink *link;
-	
-	link = ev_history_get_link_nth (view->history, index);
-	g_return_if_fail (link != NULL);
-
-	go_to_link (view, link);
-}
-
-void
-ev_view_go_back	(EvView	*view)
-{
-	int index, n;
-
-	g_return_if_fail (EV_IS_HISTORY (view->history));
-
-	index = ev_history_get_current_index (view->history);
-	n = ev_history_get_n_links (view->history);
-
-	if (n > 0) {
-		index = MAX (0, index - 1);
-		ev_history_set_current_index (view->history, index);
-	}
 }
 
-void
-ev_view_go_forward (EvView *view)
-{
-	int index, n;
-
-	g_return_if_fail (EV_IS_HISTORY (view->history));
-
-	index = ev_history_get_current_index (view->history);
-	n = ev_history_get_n_links (view->history);
-
-	if (n > 0) {
-		index = MIN (n - 1, index + 1);
-		ev_history_set_current_index (view->history, index);
-	}
-}
-
-
 void
 ev_view_set_page (EvView *view,
 		  int     page)
 {
 	g_return_if_fail (EV_IS_VIEW (view));
-	g_return_if_fail (EV_IS_HISTORY (view->history));
 
 	set_document_page (view, page);
-	ev_history_add_page (view->history, page);
 }
 
 int
@@ -1176,12 +1276,6 @@ ev_view_get_page (EvView *view)
 		return 1;
 }
 
-#define ZOOM_IN_FACTOR  1.2
-#define ZOOM_OUT_FACTOR (1.0/ZOOM_IN_FACTOR)
-
-#define MIN_SCALE 0.05409
-#define MAX_SCALE 18.4884
-
 static void
 ev_view_zoom (EvView   *view,
 	      double    factor,
@@ -1194,7 +1288,12 @@ ev_view_zoom (EvView   *view,
 	else
 		scale = factor;
 
-	view->scale = CLAMP (scale, MIN_SCALE, MAX_SCALE);
+	scale = CLAMP (scale, MIN_SCALE, MAX_SCALE);
+
+	if (ABS (scale - view->scale) < ZOOM_EPSILON)
+		return;
+
+	view->scale = scale;
 
 	ev_document_set_scale (view->document, view->scale);
 
@@ -1219,69 +1318,73 @@ ev_view_normal_size (EvView *view)
 	ev_view_zoom (view, 1.0, FALSE);
 }
 
+/* Unfortunately this is not idempotent (!) (numerical stability
+ * issues because width and height are rounded) */
 void
-ev_view_best_fit (EvView *view)
+ev_view_best_fit (EvView *view, int allocation_width, int allocation_height)
 {
-	double scale;
+	int target_width, target_height;
 	int width, height;
+	GtkBorder border;
+
+	if (!GTK_WIDGET_REALIZED (view) || view->document == NULL)
+		return;
 
 	width = height = 0;
-	ev_document_get_page_size (view->document, &width, &height);
+	/* This is the bad part. You could make it stable by doing
+	 * ev_document_set_scale 1.0. But at least with pdf this means
+	 * redrawing the whole page */
+	ev_document_get_page_size (view->document, -1, &width, &height);
+	/* FIXME: The border size isn't constant.  Ugh.  Still, if we have extra
+	 * space, we just cut it from the border */
+	ev_document_misc_get_page_border_size (width, height, &border);
+
+	target_width = allocation_width - (view->spacing * 2 + border.left + border.right);
+	target_height = allocation_height - (view->spacing * 2 + border.top + border.bottom);
+
+	LOG ("Best fit %d %d", allocation_width, allocation_height);
 
-	scale = 1.0;
 	if (width != 0 && height != 0) {
+		double scale;
 		double scale_w, scale_h;
 
-		scale_w = (double)GTK_WIDGET (view)->allocation.width * view->scale / width;
-		scale_h = (double)GTK_WIDGET (view)->allocation.height * view->scale / height;
+		scale_w = (double)target_width * view->scale / width;
+		scale_h = (double)target_height * view->scale / height;
 
 		scale = (scale_w < scale_h) ? scale_w : scale_h;
+
+		ev_view_zoom (view, scale, FALSE);
 	}
 
-	ev_view_zoom (view, scale, FALSE);
 }
 
 void
-ev_view_fit_width (EvView *view)
+ev_view_fit_width (EvView *view, int allocation_width, int allocation_height,
+		   int vsb_width)
 {
-	double scale = 1.0;
-	int width;
+	int target_width, target_height;
+	int width, height;
+	GtkBorder border;
 
-	width = 0;
-	ev_document_get_page_size (view->document, &width, NULL);
+	if (view->document == NULL)
+		return;
 
-	scale = 1.0;
-	if (width != 0)
-		scale = (double)GTK_WIDGET (view)->allocation.width * view->scale / width;
+	width = height = 0;
+	ev_document_get_page_size (view->document, -1, &width, &height);
+	ev_document_misc_get_page_border_size (width, height, &border);
 
-	ev_view_zoom (view, scale, FALSE);
-}
+	target_width = allocation_width - (view->spacing * 2 + border.left + border.right);
+	target_height = allocation_height - (view->spacing * 2 + border.top + border.bottom);
 
-static void
-history_index_changed_cb (EvHistory  *history,
-			  GParamSpec *pspec,
-			  EvView     *view)
-{
-	int index;
+	if (width) {
+		double scale;
+		scale = (double)target_width * view->scale / width;
 
-	index = ev_history_get_current_index (history);
-	go_to_index (view, index);
-}
+		if (height * scale / view->scale > target_height)
+			scale = ((double)(target_width - vsb_width) * view->scale / width);
 
-void
-ev_view_set_history (EvView     *view,
-		     EvHistory  *history)
-{
-	if (view->history) {
-		g_object_unref (view->history);
+		ev_view_zoom (view, scale, FALSE);
 	}
-
-	view->history = g_object_ref (history);
-	ev_history_add_page (view->history, ev_view_get_page (view));
-
-	g_signal_connect (view->history, "notify::index",
-			  G_CALLBACK (history_index_changed_cb),
-			  view);
 }
 
 const char *
@@ -1300,4 +1403,65 @@ ev_view_get_find_status (EvView *view)
 	return view->find_status;
 }
 
+void
+ev_view_find_next (EvView *view)
+{
+	int n_results, n_pages;
+	EvDocumentFind *find = EV_DOCUMENT_FIND (view->document);
+
+	n_results = ev_document_find_get_n_results (find);
+	n_pages = ev_document_get_n_pages (view->document);
+
+	view->find_result++;
+
+	if (view->find_result >= n_results) {
+		view->find_result = 0;
+		view->find_page++;
 
+		if (view->find_page > n_pages) {
+			view->find_page = 1;
+		}
+
+		jump_to_find_page (view);
+	} else {
+		jump_to_find_result (view);
+		gtk_widget_queue_draw (GTK_WIDGET (view));
+	}
+}
+
+void
+ev_view_find_previous (EvView *view)
+{
+	int n_results, n_pages;
+	EvDocumentFind *find = EV_DOCUMENT_FIND (view->document);
+
+	n_results = ev_document_find_get_n_results (find);
+	n_pages = ev_document_get_n_pages (view->document);
+
+	view->find_result--;
+
+	if (view->find_result < 0) {
+		view->find_result = 0;
+		view->find_page--;
+
+		if (view->find_page < 1) {
+			view->find_page = n_pages;
+		}
+
+		jump_to_find_page (view);
+	} else {
+		jump_to_find_result (view);
+		gtk_widget_queue_draw (GTK_WIDGET (view));
+	}
+}
+void
+ev_view_hide_cursor (EvView *view)
+{
+       ev_view_set_cursor (view, EV_VIEW_CURSOR_HIDDEN);
+}
+
+void
+ev_view_show_cursor (EvView *view)
+{
+       ev_view_set_cursor (view, EV_VIEW_CURSOR_LINK);
+}