[nautilus/wip/antoniof/gtk4-preparation-drop-outgoing-api: 10/13] pathbar: Drop gtk_widget_{g, s}et_clip()




commit 378a75de6195117017cc2da5a095cab266ea71b3
Author: Ernestas Kulik <ernestask gnome org>
Date:   Wed Jul 11 13:33:14 2018 +0300

    pathbar: Drop gtk_widget_{g,s}et_clip()
    
    It's gone in GTK4 and removing this doesn't seem to cause any visual or
    behavorial change at the moment.
    
    Originally it was a workaround to some visual problem with button
    shadows. Our pathbar buttons don't have shadows nowadays, and even if
    testing with added shadows, there is no apparent issue nowadays.
    
    Rebased and ammended by António Fernandes <antoniof gnome org>

 src/nautilus-pathbar.c | 34 ----------------------------------
 1 file changed, 34 deletions(-)
---
diff --git a/src/nautilus-pathbar.c b/src/nautilus-pathbar.c
index a7e966732..098a08075 100644
--- a/src/nautilus-pathbar.c
+++ b/src/nautilus-pathbar.c
@@ -458,37 +458,6 @@ nautilus_path_bar_map (GtkWidget *widget)
     GTK_WIDGET_CLASS (nautilus_path_bar_parent_class)->map (widget);
 }
 
-#define BUTTON_BOTTOM_SHADOW 1
-
-static void
-union_with_clip (GtkWidget *widget,
-                 gpointer   clip)
-{
-    GtkAllocation widget_clip;
-
-    if (!gtk_widget_is_drawable (widget))
-    {
-        return;
-    }
-
-    gtk_widget_get_clip (widget, &widget_clip);
-
-    gdk_rectangle_union (&widget_clip, clip, clip);
-}
-
-static void
-_set_simple_bottom_clip (GtkWidget *widget,
-                         gint       pixels)
-{
-    GtkAllocation clip;
-
-    gtk_widget_get_allocation (widget, &clip);
-    clip.height += pixels;
-
-    gtk_container_forall (GTK_CONTAINER (widget), union_with_clip, &clip);
-    gtk_widget_set_clip (widget, &clip);
-}
-
 /* This is a tad complicated */
 static void
 nautilus_path_bar_size_allocate (GtkWidget     *widget,
@@ -517,7 +486,6 @@ nautilus_path_bar_size_allocate (GtkWidget     *widget,
     /* No path is set so we don't have to allocate anything. */
     if (self->button_list == NULL)
     {
-        _set_simple_bottom_clip (widget, BUTTON_BOTTOM_SHADOW);
         return;
     }
     direction = gtk_widget_get_direction (widget);
@@ -638,8 +606,6 @@ nautilus_path_bar_size_allocate (GtkWidget     *widget,
         child = BUTTON_DATA (list->data)->button;
         gtk_widget_set_child_visible (child, FALSE);
     }
-
-    _set_simple_bottom_clip (widget, BUTTON_BOTTOM_SHADOW);
 }
 
 static void


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