Adwaita: Avoid accindental border on last treeview header
authorLapo Calamandrei <calamandrei@gmail.com>
Wed, 17 May 2017 16:09:59 +0000 (18:09 +0200)
committerLapo Calamandrei <calamandrei@gmail.com>
Wed, 17 May 2017 16:16:28 +0000 (18:16 +0200)
The :last-child selector supposed to reset the border was
overridden by the :hover selector. This is fixed by moving the
:last-child selector after the overriding one.
Thanks to Sebastian Keller for spotting.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=779078.

gtk/theme/Adwaita/_common.scss
gtk/theme/Adwaita/gtk-contained-dark.css
gtk/theme/Adwaita/gtk-contained.css

index ad93e78e4b79d85ae36b3457a1f3e1a14ce6f6fa..002e99e05f1533f56787dc352d6eed0af54b9683 100644 (file)
@@ -1878,8 +1878,6 @@ treeview.view {
         transition: none; //I shouldn't need this
       }
     }
-
-    button:last-child { &:backdrop, & { border-right-style: none; }}
   }
 
   button.dnd,
@@ -1925,6 +1923,8 @@ treeview.view {
       background-image: none;
     }
   }
+
+  &:last-child { &:backdrop, & { border-right-style: none; }}
 }
 
 
index 73764d5f5da0df412020e3fec75b5c0b54edb209..27476d897913ab0d079a0dd7712a36bbb46b6db3 100644 (file)
@@ -748,8 +748,6 @@ treeview.view header button:hover { color: #bcbdbc; box-shadow: none; transition
 
 treeview.view header button:active { color: #eeeeec; transition: none; }
 
-treeview.view header button:last-child:backdrop, treeview.view header button:last-child { border-right-style: none; }
-
 treeview.view button.dnd:active, treeview.view button.dnd:selected, treeview.view button.dnd:hover, treeview.view button.dnd, treeview.view header.button.dnd:active, treeview.view header.button.dnd:selected, treeview.view header.button.dnd:hover, treeview.view header.button.dnd { padding: 0 6px; color: #232729; background-image: none; background-color: #215d9c; border-style: none; border-radius: 0; box-shadow: inset 0 0 0 1px #232729; text-shadow: none; transition: none; }
 
 treeview.view acceleditor > label { background-color: #215d9c; }
@@ -762,6 +760,8 @@ treeview.view header button:backdrop { color: #626768; border-color: #33393b; bo
 
 treeview.view header button:backdrop:disabled { border-color: #33393b; background-image: none; }
 
+treeview.view header button:last-child:backdrop, treeview.view header button:last-child { border-right-style: none; }
+
 /********* Menus * */
 menubar, .menubar { -GtkWidget-window-dragging: true; padding: 0px; box-shadow: inset 0 -1px rgba(0, 0, 0, 0.1); }
 
index 4632144c22223c19850ea40ea3099206d08bb36b..5b2b2ee7f016b13b14d61bd34b8e02f8fec3ee2e 100644 (file)
@@ -756,8 +756,6 @@ treeview.view header button:hover { color: #636769; box-shadow: none; transition
 
 treeview.view header button:active { color: #2e3436; transition: none; }
 
-treeview.view header button:last-child:backdrop, treeview.view header button:last-child { border-right-style: none; }
-
 treeview.view button.dnd:active, treeview.view button.dnd:selected, treeview.view button.dnd:hover, treeview.view button.dnd, treeview.view header.button.dnd:active, treeview.view header.button.dnd:selected, treeview.view header.button.dnd:hover, treeview.view header.button.dnd { padding: 0 6px; color: #ffffff; background-image: none; background-color: #4a90d9; border-style: none; border-radius: 0; box-shadow: inset 0 0 0 1px #ffffff; text-shadow: none; transition: none; }
 
 treeview.view acceleditor > label { background-color: #4a90d9; }
@@ -770,6 +768,8 @@ treeview.view header button:backdrop { color: #babbbb; border-color: #e8e8e7; bo
 
 treeview.view header button:backdrop:disabled { border-color: #e8e8e7; background-image: none; }
 
+treeview.view header button:last-child:backdrop, treeview.view header button:last-child { border-right-style: none; }
+
 /********* Menus * */
 menubar, .menubar { -GtkWidget-window-dragging: true; padding: 0px; box-shadow: inset 0 -1px rgba(0, 0, 0, 0.1); }