[nautilus/907-graphical-issues-with-updated-path-bar: 193/193] toolbar: Always show pathbar background

[António Fernandes <antoniof src gnome org>]

commit e0b1b0242df4e819d006f06a9b577031a38e45f4
Author: António Fernandes <antoniof gnome org>
Date:   Mon Aug 5 22:31:21 2019 +0100

    toolbar: Always show pathbar background
    
    We hide the pathbar background when the window so small that the
    pathbar is no longer centered. This was introduced to make the
    pathbar flat.
    
    However, now that the pathbar is buttonized again, it doesn't make it
    flat, so there is no point. The transformation is only a distraction.
    Furthermore, it is prone to styling glitches.
    
    So, give up on the transformation and always have a pathbar background,
    effectively reverting commit 080400bd243ab86d82954eaff0c866c3ac97f3f2
    
    Fixes: https://gitlab.gnome.org/GNOME/nautilus/issues/907

 src/nautilus-toolbar.c               | 29 -----------------------------
 src/resources/css/Adwaita.css        | 30 +++++++++++++-----------------
 src/resources/ui/nautilus-toolbar.ui |  1 +
 3 files changed, 14 insertions(+), 46 deletions(-)
---
diff --git a/src/nautilus-toolbar.c b/src/nautilus-toolbar.c
index d14f17867..4ecb5ff6a 100644
--- a/src/nautilus-toolbar.c
+++ b/src/nautilus-toolbar.c
@@ -848,31 +848,6 @@ undo_manager_changed (NautilusToolbar *self)
     update_menu_item (self->redo_button, self, "redo", redo_active, redo_label);
 }
 
-static void
-on_notify_width_maximized (NautilusContainerMaxWidth *container,
-                           GParamSpec                *pspec,
-                           gpointer                   user_data)
-{
-    NautilusToolbar *self;
-    GtkStyleContext *context;
-    gboolean width_maximized;
-
-    self = NAUTILUS_TOOLBAR (user_data);
-    context = gtk_widget_get_style_context (self->path_bar_container);
-    width_maximized = nautilus_container_max_width_get_width_maximized (container);
-
-    g_return_if_fail (gtk_style_context_has_class (context, "width-maximized") != width_maximized);
-
-    if (width_maximized)
-    {
-        gtk_style_context_add_class (context, "width-maximized");
-    }
-    else
-    {
-        gtk_style_context_remove_class (context, "width-maximized");
-    }
-}
-
 static void
 on_location_entry_close (GtkWidget       *close_button,
                          NautilusToolbar *self)
@@ -936,10 +911,6 @@ nautilus_toolbar_constructed (GObject *object)
     self->path_bar = g_object_new (NAUTILUS_TYPE_PATH_BAR, NULL);
     gtk_container_add (GTK_CONTAINER (self->path_bar_container),
                        self->path_bar);
-    g_signal_connect (self->toolbar_switcher_container_max_width,
-                      "notify::width-maximized",
-                      G_CALLBACK (on_notify_width_maximized),
-                      self);
 
     self->location_entry = nautilus_location_entry_new ();
     gtk_container_add (GTK_CONTAINER (self->location_entry_container),
diff --git a/src/resources/css/Adwaita.css b/src/resources/css/Adwaita.css
index 666650975..546d3e98a 100644
--- a/src/resources/css/Adwaita.css
+++ b/src/resources/css/Adwaita.css
@@ -45,33 +45,29 @@
 
 /* Path bar */
 
-.nautilus-path-bar button { /* undecorate the buttons */
-  margin: 0px;
-}
-
-.nautilus-path-bar button:not(:checked) image { opacity: 0.8; } /* dim the icon when not checked */
-
 .path-bar-box {
-  transition: border 200ms;
-  transition: background-color 200ms;
   border-radius: 5px;
-}
-
-.path-bar-box.width-maximized {
   border: 1px @borders solid;
   background-color: @theme_bg_color;
+  padding-right: 6px;
+}
+
+.nautilus-path-bar button {
+  margin: 0px;
 }
 
-.path-bar-box.width-maximized button:first-child {
-    border-radius: 3.5px 0px 0px 3.5px;
-    border-width: 0px 1px 0px 0px;
+.nautilus-path-bar button:first-child {
+  border-width: 0px 1px 0px 0px;
+  border-radius: 3.5px 0px 0px 3.5px;
 }
 
-.path-bar-box.width-maximized button:not(:first-child) {
-    border-width: 0px 1px 0px 1px;
-    border-radius: 0px 0px 0px 0px;
+.nautilus-path-bar button:not(:first-child) {
+  border-width: 0px 1px 0px 1px;
+  border-radius: 0px 0px 0px 0px;
 }
 
+.nautilus-path-bar button:not(:checked) image { opacity: 0.8; } /* dim the icon when not checked */
+
 /* Make the tags fit into the box */
 entry.search > * {
   margin: 5px;
diff --git a/src/resources/ui/nautilus-toolbar.ui b/src/resources/ui/nautilus-toolbar.ui
index 0e8ebaf2c..51a882838 100644
--- a/src/resources/ui/nautilus-toolbar.ui
+++ b/src/resources/ui/nautilus-toolbar.ui
@@ -602,6 +602,7 @@
       <object class="GtkBox">
         <property name="visible">True</property>
         <property name="can_focus">False</property>
+        <property name="margin_start">72</property>
         <child>
           <object class="GtkRevealer" id="operations_revealer">
             <property name="visible">True</property>