[gtk+/wip/baedert/gtkimageview: 58/160] GtkImageView: Fix drawing in scrollable case
- From: Timm Bäder <baedert src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/wip/baedert/gtkimageview: 58/160] GtkImageView: Fix drawing in scrollable case
- Date: Tue, 16 Feb 2016 12:58:57 +0000 (UTC)
commit 1d24f59a59b9846ba307abfe7fa7b67749caee1c
Author: Timm Bäder <mail baedert org>
Date: Thu Jul 23 17:35:09 2015 +0200
GtkImageView: Fix drawing in scrollable case
and a thousand other (frustrating) things
demos/gtk-demo/image_view.c | 6 ++-
gtk/gtkimageview.c | 164 ++++++++++++++++++++++++------------------
2 files changed, 99 insertions(+), 71 deletions(-)
---
diff --git a/demos/gtk-demo/image_view.c b/demos/gtk-demo/image_view.c
index 3c827c8..cb8a7ae 100644
--- a/demos/gtk-demo/image_view.c
+++ b/demos/gtk-demo/image_view.c
@@ -82,9 +82,12 @@ scrolled_check_button_active_cb (GObject *source)
{
GtkWidget *grandparent = gtk_widget_get_parent (parent);
g_assert (grandparent != NULL);
+ g_object_ref (G_OBJECT (image_view));
+ gtk_container_remove (GTK_CONTAINER (parent), image_view);
gtk_container_remove (GTK_CONTAINER (grandparent), parent);
gtk_container_add (GTK_CONTAINER (grandparent), image_view);
gtk_widget_show (image_view);
+ g_object_unref (G_OBJECT (image_view));
}
else
{
@@ -92,6 +95,7 @@ scrolled_check_button_active_cb (GObject *source)
gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (scroller),
GTK_POLICY_ALWAYS,
GTK_POLICY_ALWAYS);
+ gtk_scrolled_window_set_overlay_scrolling (GTK_SCROLLED_WINDOW (scroller), FALSE);
gtk_widget_show (scroller);
gtk_container_remove (GTK_CONTAINER (parent), image_view);
gtk_container_add (GTK_CONTAINER (scroller), image_view);
@@ -172,7 +176,7 @@ do_image_view (GtkWidget *do_widget)
GtkAdjustment *angle_adjustment = GTK_ADJUSTMENT (gtk_builder_get_object (builder, "angle_adjustment"));
g_object_bind_property (scale_adjustment, "value", image_view, "scale",
- G_BINDING_BIDIRECTIONAL | G_BINDING_SYNC_CREATE);
+ G_BINDING_BIDIRECTIONAL);
g_object_bind_property (image_view, "angle", angle_adjustment, "value",
G_BINDING_BIDIRECTIONAL | G_BINDING_SYNC_CREATE);
g_object_bind_property (image_view, "snap-angle", snap_angle_button, "active",
diff --git a/gtk/gtkimageview.c b/gtk/gtkimageview.c
index bfb9ca1..4786712 100644
--- a/gtk/gtkimageview.c
+++ b/gtk/gtkimageview.c
@@ -144,7 +144,7 @@ gtk_image_view_init (GtkImageView *image_view)
priv->rotate_gesture = gtk_gesture_rotate_new ((GtkWidget *)image_view);
g_signal_connect (priv->rotate_gesture, "angle-changed", (GCallback)gesture_angle_changed_cb, NULL);
priv->zoom_gesture = gtk_gesture_zoom_new ((GtkWidget *)image_view);
- g_signal_connect (priv->zoom_gesture, "zoom-changed", (GCallback)gesture_zoom_changed_cb, NULL);
+ g_signal_connect (priv->zoom_gesture, "scale-changed", (GCallback)gesture_zoom_changed_cb, NULL);
gtk_style_context_add_class (sc, GTK_STYLE_CLASS_BACKGROUND);
}
@@ -330,16 +330,25 @@ gtk_image_view_compute_bounding_box (GtkImageView *image_view,
bb_height = MAX (fabs (upper_right_y), fabs (upper_left_y)) * 2;
- if (!priv->scale_set)
- {
- double scale_x = (double)alloc.width / (double)bb_width;
- double scale_y = (double)alloc.height / (double)bb_height;
- scale = MIN (MIN (scale_x, scale_y), 1.0);
+
+ if (priv->scale_set)
+ {
+ scale = priv->scale;
}
else
{
- scale = priv->scale;
+ if (priv->fit_allocation)
+ {
+ double scale_x = (double)alloc.width / (double)bb_width;
+ double scale_y = (double)alloc.height / (double)bb_height;
+
+ scale = MIN (MIN (scale_x, scale_y), 1.0);
+ }
+ else
+ {
+ scale = 1.0;
+ }
}
if (scale_out)
@@ -363,18 +372,39 @@ gtk_image_view_compute_bounding_box (GtkImageView *image_view,
}
}
+
+static inline void
+gtk_image_view_restrict_adjustment (GtkAdjustment *adjustment)
+{
+ double value = gtk_adjustment_get_value (adjustment);
+ double upper = gtk_adjustment_get_upper (adjustment);
+ double page_size = gtk_adjustment_get_page_size (adjustment);
+
+ value = gtk_adjustment_get_value (adjustment);
+ upper = gtk_adjustment_get_upper (adjustment);
+
+ if (value > upper - page_size)
+ gtk_adjustment_set_value (adjustment, upper - page_size);
+ else if (value < 0)
+ gtk_adjustment_set_value (adjustment, 0);
+
+
+}
+
static void
gtk_image_view_update_adjustments (GtkImageView *image_view)
{
GtkImageViewPrivate *priv = gtk_image_view_get_instance_private (image_view);
+ int widget_width = gtk_widget_get_allocated_width ((GtkWidget *)image_view);
+ int widget_height = gtk_widget_get_allocated_height ((GtkWidget *)image_view);
if (!priv->hadjustment && !priv->vadjustment)
return;
if (!priv->image_surface)
{
- gtk_adjustment_configure (priv->vadjustment, 0, 0, 0, 0, 0, 0);
gtk_adjustment_configure (priv->hadjustment, 0, 0, 0, 0, 0, 0);
+ gtk_adjustment_configure (priv->vadjustment, 0, 0, 0, 0, 0, 0);
return;
}
@@ -382,8 +412,8 @@ gtk_image_view_update_adjustments (GtkImageView *image_view)
if (priv->fit_allocation)
{
- gtk_adjustment_set_upper (priv->vadjustment, 0);
- gtk_adjustment_set_upper (priv->hadjustment, 0);
+ gtk_adjustment_set_upper (priv->hadjustment, widget_width);
+ gtk_adjustment_set_upper (priv->vadjustment, widget_height);
}
else
{
@@ -392,17 +422,16 @@ gtk_image_view_update_adjustments (GtkImageView *image_view)
&width,
&height,
NULL);
- gtk_adjustment_set_upper (priv->hadjustment, width);
- gtk_adjustment_set_upper (priv->vadjustment, height);
-
+ gtk_adjustment_set_upper (priv->hadjustment, MAX (width, widget_width));
+ gtk_adjustment_set_upper (priv->vadjustment, MAX (height, widget_height));
}
- gtk_adjustment_set_page_size (priv->hadjustment,
- gtk_widget_get_allocated_width ((GtkWidget *)image_view));
- gtk_adjustment_set_page_size (priv->vadjustment,
- gtk_widget_get_allocated_height ((GtkWidget *)image_view));
+ gtk_adjustment_set_page_size (priv->hadjustment, widget_width);
+ gtk_adjustment_set_page_size (priv->vadjustment, widget_height);
+ gtk_image_view_restrict_adjustment (priv->hadjustment);
+ gtk_image_view_restrict_adjustment (priv->vadjustment);
}
static gboolean
@@ -411,91 +440,75 @@ gtk_image_view_draw (GtkWidget *widget, cairo_t *ct)
GtkImageView *image_view = (GtkImageView *)widget;
GtkImageViewPrivate *priv = gtk_image_view_get_instance_private (image_view);
GtkStyleContext *sc = gtk_widget_get_style_context (widget);
- GtkAllocation alloc;
int draw_x;
int draw_y;
int draw_width;
int draw_height;
- int image_width;
- int image_height;
double scale = 0.0;
+ int widget_width = gtk_widget_get_allocated_width (widget);
+ int widget_height = gtk_widget_get_allocated_height (widget);
- gtk_widget_get_allocation (widget, &alloc);
-
- gtk_render_background (sc, ct, 0, 0, alloc.width, alloc.height);
- gtk_render_frame (sc, ct, 0, 0, alloc.width, alloc.height);
+ gtk_render_background (sc, ct, 0, 0, widget_width, widget_height);
+ gtk_render_frame (sc, ct, 0, 0, widget_width, widget_height);
if (!priv->image_surface)
return GDK_EVENT_PROPAGATE;
gtk_image_view_compute_bounding_box (image_view, &draw_width, &draw_height, &scale);
- image_width = priv->surface_width * scale;
- image_height = priv->surface_height * scale;
- if (image_width == 0 || image_height == 0)
+ if (draw_width == 0 || draw_height == 0)
return GDK_EVENT_PROPAGATE;
- draw_x = (alloc.width - draw_width) / 2;
- draw_y = (alloc.height - draw_height) / 2;
-
+ int image_width = priv->surface_width * scale;
+ int image_height = priv->surface_height * scale;
#if 0
- {
- cairo_save (ct);
- cairo_set_source_rgba (ct, 0.7, 0.7, 0.7, 1);
- cairo_rectangle (ct, (alloc.width - draw_width) / 2, (alloc.height - draw_height) / 2, draw_width,
draw_height);
- cairo_fill (ct);
- cairo_set_source_rgba (ct, 0, 0, 0, 1);
- cairo_rectangle (ct, (alloc.width - draw_width) / 2, (alloc.height - draw_height) / 2, draw_width,
draw_height);
- cairo_stroke (ct);
- cairo_restore (ct);
- }
-
- {
- /*cairo_save (ct);*/
- /*cairo_set_source_rgba (ct, 1, 0, 0, 1);*/
- /*cairo_rectangle (ct, ((alloc.width - image_width) / 2) / scale,*/
- /*((alloc.height - image_height) / 2) / scale,*/
- /*cairo_image_surface_get_width (priv->image_surface),*/
- /*cairo_image_surface_get_height (priv->image_surface));*/
- /*cairo_stroke (ct);*/
- /*cairo_restore (ct);*/
- }
+ cairo_save (ct);
+ cairo_rectangle (ct, draw_x, draw_y, draw_width, draw_height);
+ cairo_set_source_rgba (ct, 0, 1, 0, 1);
+ cairo_stroke (ct);
+ cairo_restore (ct);
#endif
-
+ if (priv->vadjustment == NULL)
+ {
+ draw_x = (widget_width - image_width) / 2;
+ draw_y = (widget_height - image_height) / 2;
+ }
+ else
+ {
+ draw_x = (gtk_adjustment_get_page_size (priv->hadjustment) - image_width) / 2;
+ draw_y = (gtk_adjustment_get_page_size (priv->vadjustment) - image_height) / 2;
+ }
cairo_save (ct);
- cairo_rectangle (ct, draw_x, draw_y, draw_width, draw_height);
+ /*cairo_rectangle (ct, draw_x, draw_y, draw_width, draw_height);*/
+ cairo_rectangle (ct, 0, 0, widget_width, widget_height);
- /* Handle the h/vadjustments, but keep the image centered in all cases */
-
- /*if (priv->hadjustment &&*/
- /*gtk_adjustment_get_page_size (priv->hadjustment) < draw_width)*/
- /*draw_x = -gtk_adjustment_get_value (priv->hadjustment);*/
- /*else*/
- draw_x = (alloc.width - image_width) / 2;
+ if (priv->hadjustment && draw_width >= widget_width)
+ {
+ draw_x = (draw_width - image_width) / 2;
+ draw_x -= gtk_adjustment_get_value (priv->hadjustment);
+ }
- /*if (priv->vadjustment &&*/
- /*gtk_widget_get_allocated_height (widget) < draw_height)*/
- /*draw_y = -gtk_adjustment_get_value (priv->vadjustment);*/
- /*else*/
- draw_y = (alloc.height - image_height) / 2;
- if (priv->vadjustment)
+ if (priv->vadjustment && draw_height >= widget_height)
+ {
+ draw_y = (draw_height - image_height) / 2;
draw_y -= gtk_adjustment_get_value (priv->vadjustment);
+ }
/* Rotate around the center */
- cairo_translate (ct, draw_x + (image_width / 2.0), draw_y + (image_height / 2.0));
+ cairo_translate (ct, draw_x + (image_width / 2.0), draw_y + (image_height / 2.0));
cairo_rotate (ct, DEG_TO_RAD (priv->angle));
- cairo_translate (ct, -draw_x - (image_width / 2.0), - draw_y - (image_height / 2.0));
+ cairo_translate (ct, -draw_x - (image_width / 2.0), -draw_y - (image_height / 2.0));
cairo_scale (ct, scale, scale);
@@ -535,6 +548,11 @@ gtk_image_view_set_hadjustment (GtkImageView *image_view,
priv->hadjustment = hadjustment;
}
+ if (priv->fit_allocation)
+ gtk_widget_queue_draw ((GtkWidget *)image_view);
+ else
+ gtk_widget_queue_resize ((GtkWidget *)image_view);
+
}
static void
@@ -562,6 +580,11 @@ gtk_image_view_set_vadjustment (GtkImageView *image_view,
{
priv->vadjustment = vadjustment;
}
+
+ if (priv->fit_allocation)
+ gtk_widget_queue_draw ((GtkWidget *)image_view);
+ else
+ gtk_widget_queue_resize ((GtkWidget *)image_view);
}
@@ -627,8 +650,6 @@ gtk_image_view_set_angle (GtkImageView *image_view,
g_return_if_fail (GTK_IS_IMAGE_VIEW (image_view));
- g_message (__FUNCTION__);
-
if (priv->snap_angle)
gtk_image_view_do_snapping (image_view, angle);
else
@@ -743,6 +764,7 @@ gtk_image_view_set_fit_allocation (GtkImageView *image_view,
gtk_image_view_update_adjustments (image_view);
gtk_widget_queue_resize ((GtkWidget *)image_view);
+ gtk_image_view_update_adjustments (image_view);
}
gboolean
@@ -884,6 +906,8 @@ adjustment_value_changed_cb (GtkAdjustment *adjustment,
{
GtkImageView *image_view = user_data;
+ gtk_image_view_update_adjustments (image_view);
+
gtk_widget_queue_draw ((GtkWidget *)image_view);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]