[gtk+/wip/baedert/drawing: 136/355] range: Remove contents gadget



commit e8075f7c804d28ddf585eca63a40bf21062d768d
Author: Timm Bäder <mail baedert org>
Date:   Sun May 7 16:44:13 2017 +0200

    range: Remove contents gadget
    
    It's not needed anymore since the range only has one child gadget: the
    trough.

 gtk/gtkrange.c |   41 ++++++++---------------------------------
 gtk/gtkscale.c |   14 +++++---------
 2 files changed, 13 insertions(+), 42 deletions(-)
---
diff --git a/gtk/gtkrange.c b/gtk/gtkrange.c
index eca256f..d3dd8cd 100644
--- a/gtk/gtkrange.c
+++ b/gtk/gtkrange.c
@@ -84,7 +84,6 @@ struct _GtkRangePrivate
 
   GtkAdjustment     *adjustment;
 
-  GtkCssGadget *contents_gadget;
   GtkCssGadget *trough_gadget;
   GtkCssGadget *fill_gadget;
   GtkCssGadget *highlight_gadget;
@@ -463,17 +462,6 @@ gtk_range_class_init (GtkRangeClass *class)
 }
 
 static void
-gtk_range_sync_orientation (GtkRange *range)
-{
-  GtkRangePrivate *priv = range->priv;
-  GtkOrientation orientation;
-
-  orientation = gtk_orientable_get_orientation (GTK_ORIENTABLE (range));
-  _gtk_orientable_set_style_classes (GTK_ORIENTABLE (range));
-  gtk_box_gadget_set_orientation (GTK_BOX_GADGET (priv->contents_gadget), orientation);
-}
-
-static void
 gtk_range_set_property (GObject      *object,
                        guint         prop_id,
                        const GValue *value,
@@ -488,7 +476,7 @@ gtk_range_set_property (GObject      *object,
       if (priv->orientation != g_value_get_enum (value))
         {
           priv->orientation = g_value_get_enum (value);
-          gtk_range_sync_orientation (range);
+          _gtk_orientable_set_style_classes (GTK_ORIENTABLE (range));
           gtk_widget_queue_resize (GTK_WIDGET (range));
           g_object_notify_by_pspec (object, pspec);
         }
@@ -582,12 +570,6 @@ gtk_range_init (GtkRange *range)
   _gtk_orientable_set_style_classes (GTK_ORIENTABLE (range));
 
   widget_node = gtk_widget_get_css_node (GTK_WIDGET (range));
-  priv->contents_gadget = gtk_box_gadget_new ("contents",
-                                              GTK_WIDGET (range),
-                                              NULL, NULL);
-  gtk_css_node_set_parent (gtk_css_gadget_get_node (priv->contents_gadget),
-                           widget_node);
-
   priv->trough_gadget = gtk_css_custom_gadget_new ("trough",
                                                    GTK_WIDGET (range),
                                                    NULL, NULL,
@@ -597,8 +579,8 @@ gtk_range_init (GtkRange *range)
                                                    NULL, NULL);
   gtk_css_gadget_set_state (priv->trough_gadget,
                             gtk_css_node_get_state (widget_node));
-  gtk_box_gadget_insert_gadget (GTK_BOX_GADGET (priv->contents_gadget), -1, priv->trough_gadget,
-                                TRUE, GTK_ALIGN_CENTER);
+  gtk_css_node_set_parent (gtk_css_gadget_get_node (priv->trough_gadget),
+                           widget_node);
 
   priv->slider_gadget = gtk_builtin_icon_new ("slider",
                                               GTK_WIDGET (range),
@@ -955,14 +937,10 @@ void
 gtk_range_get_range_rect (GtkRange     *range,
                           GdkRectangle *range_rect)
 {
-  GtkRangePrivate *priv;
-
   g_return_if_fail (GTK_IS_RANGE (range));
   g_return_if_fail (range_rect != NULL);
 
-  priv = range->priv;
-
-  gtk_css_gadget_get_margin_box (priv->contents_gadget, range_rect);
+  gtk_widget_get_margin_allocation (GTK_WIDGET (range), range_rect);
 }
 
 /**
@@ -1349,7 +1327,6 @@ gtk_range_finalize (GObject *object)
   g_clear_object (&priv->multipress_gesture);
   g_clear_object (&priv->long_press_gesture);
 
-  g_clear_object (&priv->contents_gadget);
   g_clear_object (&priv->trough_gadget);
   g_clear_object (&priv->fill_gadget);
   g_clear_object (&priv->highlight_gadget);
@@ -1412,7 +1389,7 @@ static void gtk_range_measure (GtkWidget     *widget,
   GtkBorder border = { 0 };
 
   /* Measure the main box */
-  gtk_css_gadget_get_preferred_size (priv->contents_gadget,
+  gtk_css_gadget_get_preferred_size (priv->trough_gadget,
                                      orientation,
                                      -1,
                                      minimum, natural,
@@ -1666,7 +1643,7 @@ gtk_range_size_allocate (GtkWidget     *widget,
   if (GTK_RANGE_GET_CLASS (range)->get_range_border)
     GTK_RANGE_GET_CLASS (range)->get_range_border (range, &border);
 
-  measure_one_gadget (priv->contents_gadget, &box_min_width, &box_min_height);
+  measure_one_gadget (priv->trough_gadget, &box_min_width, &box_min_height);
 
   if (priv->orientation == GTK_ORIENTATION_VERTICAL)
     clamp_dimensions (allocation, &box_min_width, &box_min_height, &border, TRUE);
@@ -1678,7 +1655,7 @@ gtk_range_size_allocate (GtkWidget     *widget,
   box_alloc.width = box_min_width;
   box_alloc.height = box_min_height;
 
-  gtk_css_gadget_allocate (priv->contents_gadget,
+  gtk_css_gadget_allocate (priv->trough_gadget,
                            &box_alloc,
                            gtk_widget_get_allocated_baseline (widget),
                            &child_clip);
@@ -1730,8 +1707,6 @@ update_trough_state (GtkRange *range)
 
   state &= ~(GTK_STATE_FLAG_PRELIGHT | GTK_STATE_FLAG_ACTIVE);
 
-  gtk_css_gadget_set_state (priv->contents_gadget, state);
-
   if (priv->mouse_location == priv->trough_gadget &&
       !(state & GTK_STATE_FLAG_INSENSITIVE))
     state |= GTK_STATE_FLAG_PRELIGHT;
@@ -1807,7 +1782,7 @@ gtk_range_snapshot (GtkWidget   *widget,
   GtkRange *range = GTK_RANGE (widget);
   GtkRangePrivate *priv = range->priv;
 
-  gtk_css_gadget_snapshot (priv->contents_gadget, snapshot);
+  gtk_css_gadget_snapshot (priv->trough_gadget, snapshot);
 
   /* Draw the slider last, so that e.g. the focus ring stays below it */
   gtk_css_gadget_snapshot (priv->slider_gadget, snapshot);
diff --git a/gtk/gtkscale.c b/gtk/gtkscale.c
index 3849eac..93125a9 100644
--- a/gtk/gtkscale.c
+++ b/gtk/gtkscale.c
@@ -86,11 +86,10 @@
  * ┊   ┊
  * │   ╰── mark
  * ├── [value]
- * ├── contents
- * │   ╰── trough
- * │       ├── slider
- * │       ├── [highlight]
- * │       ╰── [fill]
+ * ├── trough
+ * │   ├── slider
+ * │   ├── [highlight]
+ * │   ╰── [fill]
  * ╰── marks.bottom
  *     ├── mark
  *     ┊    ├── indicator
@@ -112,7 +111,7 @@
  * there is a subnode with name fill below the trough node that is used for
  * rendering the filled in part of the trough.
  *
- * If marks are present, there is a marks subnode before or after the contents
+ * If marks are present, there is a marks subnode before or after the trough
  * node, below which each mark gets a node with name mark. The marks nodes get
  * either the .top or .bottom style class.
  *
@@ -606,9 +605,6 @@ gtk_scale_size_allocate (GtkWidget     *widget,
   orientation = gtk_orientable_get_orientation (GTK_ORIENTABLE (widget));
   gtk_range_get_range_rect (GTK_RANGE (scale), &range_rect);
 
-  range_rect.x += allocation->x;
-  range_rect.y += allocation->y;
-
   if (orientation == GTK_ORIENTATION_HORIZONTAL)
     {
       int marks_height = 0;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]