Some tweaks

Author: lassekongo83

src/sass/_colors.scss

@@ -133,6 +133,7 @@ $top_hilight: $borders_edge;
 $alt_borders_color: if($variant == 'light', gtkmix(black, $window_bg_color, 24%), gtkmix(black, $window_bg_color, 18%));
 $suggested_bg_color: $accent_bg_color;
 $suggested_border_color: $selected_borders_color;
+$drop_target_color: $accent_bg_color;
 
 // XFCE
 $panel_bg_color: gtkcolor(panel_bg_color);

src/sass/widgets/_buttons.scss

@@ -74,6 +74,7 @@ button {
       @include button(active);
 
       transition: $button_transition;
+      @include focus-ring($transition: $button_transition);
     }
 
     &:backdrop {
@@ -116,9 +117,9 @@ button {
 
     @at-root %button_basic_drop_active,
     &:drop(active) {
-      color: $accent_bg_color;
-      border-color: $accent_bg_color;
-      box-shadow: inset 0 0 0 1px $accent_bg_color;
+      color: $drop_target_color;
+      border-color: $drop_target_color;
+      box-shadow: inset 0 0 0 1px $drop_target_color;
     }
   }
 

src/sass/widgets/_color-chooser.scss

@@ -7,7 +7,7 @@ colorswatch {
 
   &:drop(active), & { border-style: none; } // FIXME: implement a proper drop(active) state
 
-  $_colorswatch_radius: 5px;
+  $_colorswatch_radius: $button_radius;
 
   // base color corners rounding
   // to avoid the artifacts caused by rounded corner anti-aliasing the base color
@@ -80,15 +80,15 @@ colorswatch {
     box-shadow: none;
 
     &.light overlay {
-      border-color: $accent_bg_color;
-      box-shadow: inset 0 0 0 2px if($variant == 'light', gtkshade($accent_bg_color, 0.93), $borders_color),
-                  inset 0 0 0 1px $accent_bg_color;
+      border-color: $drop_target_color;
+      box-shadow: inset 0 0 0 2px if($variant == 'light', gtkshade($drop_target_color, 0.93), $borders_color),
+                  inset 0 0 0 1px $drop_target_color;
     }
 
     &.dark overlay {
-      border-color: $accent_bg_color;
+      border-color: $drop_target_color;
       box-shadow: inset 0 0 0 2px if($variant == 'light', transparentize(black, 0.7), $borders_color),
-                  inset 0 0 0 1px $accent_bg_color;
+                  inset 0 0 0 1px $drop_target_color;
     }
   }
 

src/sass/widgets/_entries.scss

@@ -53,8 +53,8 @@ entry {
 
     &:drop(active) {
       &:focus, & {
-        border-color: $accent_bg_color;
-        box-shadow: inset 0 0 0 1px $accent_bg_color;
+        border-color: $drop_target_color;
+        box-shadow: inset 0 0 0 1px $drop_target_color;
       }
     }
 
@@ -89,7 +89,7 @@ entry {
 
   .linked:not(.vertical) > &:drop(active) + &,
   .linked:not(.vertical) > &:drop(active) + button,
-  .linked:not(.vertical) > &:drop(active) + combobox > box > button.combo { border-left-color: $accent_bg_color; }
+  .linked:not(.vertical) > &:drop(active) + combobox > box > button.combo { border-left-color: $drop_target_color; }
 
   // Vertically linked entries
   // FIXME: take care of "colored" entries
@@ -117,7 +117,7 @@ entry {
     + entry:focus.error:not(:only-child) { border-top-color: entry_focus_border($error_bg_color); }
 
     + %entry:drop(active):not(:only-child),
-    + entry:drop(active):not(:only-child) { border-top-color: $accent_bg_color; }
+    + entry:drop(active):not(:only-child) { border-top-color: $drop_target_color; }
 
     // this takes care of coloring the top border of the focused entry subsequent widget.
     // :not(:only-child) is a specificity bump hack.
@@ -139,7 +139,7 @@ entry {
       + %entry,
       + entry,
       + button,
-      + combobox > box > button.combo { border-top-color: $accent_bg_color; }
+      + combobox > box > button.combo { border-top-color: $drop_target_color; }
     }
   }
 

src/sass/widgets/_file-chooser.scss

@@ -63,20 +63,20 @@ placessidebar {
     &.sidebar-placeholder-row {
       padding: 0 8px;
       min-height: 2px;
-      background-image: image($accent_bg_color);
+      background-image: image($drop_target_color);
       background-clip: content-box;
     }
 
     &.sidebar-new-bookmark-row { color: $accent_bg_color; }
 
     &:drop(active):not(:disabled) {
       color: $accent_bg_color;
-      box-shadow: inset 0 1px $accent_bg_color,
-                  inset 0 -1px $accent_bg_color;
+      box-shadow: inset 0 1px $drop_target_color,
+                  inset 0 -1px $drop_target_color;
 
       &:selected {
         color: $accent_fg_color;
-        background-color: $accent_bg_color;
+        background-color: $drop_target_color;
       }
     }
   }

src/sass/widgets/_links.scss

@@ -50,6 +50,8 @@ button:visited {
     text-shadow: none;
   }
 
+  @include focus-ring($transition: $button_transition);
+
   > label {
     @extend %link;
 

src/sass/widgets/_misc.scss

@@ -106,9 +106,9 @@ cursor-handle {
 
 :not(decoration):not(window):drop(active):focus,
 :not(decoration):not(window):drop(active) { // FIXME needs to be done widget by widget, this wildcard should really die
-  border-color: $accent_bg_color;
-  box-shadow: inset 0 0 0 1px $accent_bg_color;
-  caret-color: $accent_bg_color;
+  border-color: $drop_target_color;
+  box-shadow: inset 0 0 0 1px $drop_target_color;
+  caret-color: $drop_target_color;
 }
 
 stackswitcher button.text-button { min-width: 100px; } // FIXME aggregate with buttons