X-Git-Url: https://git.stderr.nl/gitweb?a=blobdiff_plain;f=libchimara%2Fpager.c;h=c51dbd75e0c43574fd52d35a3a4af9125b0767b2;hb=490f0a12cc09054e98d99ddc195859099223e597;hp=294a6587aef1f5a84378f23b49772ca0924bffeb;hpb=1a93d1eeab1585367a5c53eeb2de98f923822db1;p=rodin%2Fchimara.git

diff --git a/libchimara/pager.c b/libchimara/pager.c
index 294a658..c51dbd7 100644
--- a/libchimara/pager.c
+++ b/libchimara/pager.c
@@ -32,8 +32,10 @@ move_pager_and_get_scroll_distance(GtkTextView *textview, gint *view_height, gin
 	gtk_text_view_get_iter_location(textview, &newpager, &pagerpos);
 	gtk_text_view_get_iter_location(textview, &end, &endpos);
 
+	/*
 	g_printerr("View height = %d\n", visiblerect.height);
 	g_printerr("End - Pager = %d\n", endpos.y - pagerpos.y);
+	*/
 	
 	*view_height = visiblerect.height;
 	*scroll_distance = endpos.y - pagerpos.y;
@@ -43,7 +45,6 @@ move_pager_and_get_scroll_distance(GtkTextView *textview, gint *view_height, gin
 static void
 start_paging(winid_t win)
 {
-	printf("Start paging\n");
 	win->currently_paging = TRUE;
 	g_signal_handler_unblock(win->widget, win->pager_expose_handler);
 	g_signal_handler_unblock(win->widget, win->pager_keypress_handler);
@@ -53,58 +54,11 @@ start_paging(winid_t win)
 static void
 stop_paging(winid_t win)
 {
-	printf("Stop paging\n");
 	win->currently_paging = FALSE;
 	g_signal_handler_block(win->widget, win->pager_expose_handler);
 	g_signal_handler_block(win->widget, win->pager_keypress_handler);
 }
 
-/* Check whether paging should be done. This function is called inside the 
- * idle handler, after the textview has finished updating. */
-gboolean
-pager_check(gpointer data)
-{
-
-	printf("pager check (idle)...\n");
-	winid_t win = (winid_t) data;
-
-	/* Move the pager to the last visible character in the buffer */
-	gint view_height, scroll_distance;
-	move_pager_and_get_scroll_distance( GTK_TEXT_VIEW(win->widget), &view_height, &scroll_distance, FALSE );
-
-	gdk_threads_enter();
-
-	if(view_height <= 1)
-		/* Paging is unusable when window is too small */
-		return FALSE;
-	
-	/* Scroll past text already read by user. This is automatic scrolling, so disable the pager_ajustment_handler
-	 * first, that acts on the belief the scolling is performed by the user. */
-	GtkAdjustment *adj = gtk_scrolled_window_get_vadjustment(GTK_SCROLLED_WINDOW(win->frame));
-	g_signal_handler_block(adj, win->pager_adjustment_handler);
-	GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(win->widget));
-	GtkTextMark *pager_position = gtk_text_buffer_get_mark(buffer, "pager_position");
-	gtk_text_view_scroll_to_mark(GTK_TEXT_VIEW(win->widget), pager_position, 0.0, TRUE, 0.0, 0.0);
-	g_signal_handler_unblock(adj, win->pager_adjustment_handler);
-	
-	if(!win->currently_paging) {
-		if(scroll_distance > view_height) {
-			start_paging(win);
-			/* Seriously... */
-			/* COMPAT: */
-#if GTK_CHECK_VERSION(2,14,0)
-			gdk_window_invalidate_rect(gtk_widget_get_window(win->widget), NULL, TRUE);
-#else
-			gdk_window_invalidate_rect(win->widget->window, NULL, TRUE);
-#endif
-		}
-	}
-	gdk_threads_leave();
-
-	/* Returning FALSE to prevent this function from being called multiple times */
-	return FALSE;
-}
-
 /* When the user scrolls up in a textbuffer, start paging. */
 void
 pager_after_adjustment_changed(GtkAdjustment *adj, winid_t win)
@@ -174,19 +128,40 @@ pager_on_expose(GtkTextView *textview, GdkEventExpose *event, winid_t win)
 	return FALSE; /* Propagate event further */
 }
 
-gboolean
-pager_after_expose_event(GtkTextView *textview, GdkEventExpose *event, winid_t win)
-{
-	printf("pager check (expose)...\n");
-	g_idle_add(pager_check, win);
-	return FALSE;
-}
+/* Check whether paging should be done. This function is called after the
+ * textview has finished validating text positions. */
+void pager_after_size_request(GtkTextView *textview, GtkRequisition
+		*requisition, winid_t win) {
+	/* Move the pager to the last visible character in the buffer */
+	gint view_height, scroll_distance; move_pager_and_get_scroll_distance(
+			GTK_TEXT_VIEW(win->widget), &view_height, &scroll_distance, FALSE
+			);
 
-void
-pager_after_size_request(GtkTextView *textview, GtkRequisition *requisition, winid_t win)
-{
-	printf("pager check (size request)...\n");
-	g_idle_add(pager_check, win);
+	if(view_height <= 1)
+		/* Paging is unusable when window is too small */
+		return;
+	
+	/* Scroll past text already read by user. This is automatic scrolling, so disable the pager_ajustment_handler
+	 * first, that acts on the belief the scolling is performed by the user. */
+	GtkAdjustment *adj = gtk_scrolled_window_get_vadjustment(GTK_SCROLLED_WINDOW(win->frame));
+	g_signal_handler_block(adj, win->pager_adjustment_handler);
+	GtkTextBuffer *buffer = gtk_text_view_get_buffer(GTK_TEXT_VIEW(win->widget));
+	GtkTextMark *pager_position = gtk_text_buffer_get_mark(buffer, "pager_position");
+	gtk_text_view_scroll_to_mark(GTK_TEXT_VIEW(win->widget), pager_position, 0.0, TRUE, 0.0, 0.0);
+	g_signal_handler_unblock(adj, win->pager_adjustment_handler);
+	
+	if(!win->currently_paging) {
+		if(scroll_distance > view_height) {
+			start_paging(win);
+			/* Seriously... */
+			/* COMPAT: */
+#if GTK_CHECK_VERSION(2,14,0)
+			gdk_window_invalidate_rect(gtk_widget_get_window(win->widget), NULL, TRUE);
+#else
+			gdk_window_invalidate_rect(win->widget->window, NULL, TRUE);
+#endif
+		}
+	}
 }
 
 void