Fix artifacts in share button, SL #3387

When the SugarRadioToolButton is inside a SugarPaletteWindowWidget
and the state is active we make the background color button_grey. When a
GtkToolButton is inside of a SugarPaletteWindowWidget and the state is active
we make the background color transparent, otherwise this fills up all the space
around the SugarRadioToolButton.

For all the other appearences of a GtkToolButton we make the background color
button_grey when the state is active which fixes partly SL #3590.

Initial-patch: Carlos Garnacho <carlosg@gnome.org>
Signed-off-by: Simon Schampijer <simon@laptop.org>
Tested-by: Gonzalo Odiard <gonzalo@laptop.org>

1 files changed, 11 insertions, 16 deletions

diff --git a/gtk3/theme/gtk-widgets.css.em b/gtk3/theme/gtk-widgets.css.em
index fe5ff75..9671e29 100644
--- a/gtk3/theme/gtk-widgets.css.em
+++ b/gtk3/theme/gtk-widgets.css.em
@@ -270,11 +270,6 @@ GtkComboBox .separator {
     color: alpha(@theme_base_color, 0.0);
 }
 
-.toolbar GtkToggleButton.button:active,
-SugarPaletteWindowWidget GtkToggleButton.button:active {
-    background-color: @button_grey;
-}
-
 /* Notebooks */
 
 .notebook {
@@ -504,7 +499,9 @@ padding: 0px;
 }
 
 .toolbar GtkToolButton .button,
-SugarPaletteWindowWidget GtkToolButton .button {
+SugarPaletteWindowWidget SugarRadioToolButton *,
+SugarPaletteWindowWidget GtkToolButton .button{
+    background-color: transparent;
     border-radius: $(toolbutton_padding)px;
     padding: $(default_padding)px;
 }
@@ -512,22 +509,20 @@ SugarPaletteWindowWidget GtkToolButton .button {
 .toolbar GtkToolButton .button:prelight,
 SugarPaletteWindowWidget GtkToolButton .button:prelight {
     background-color: @black;
-    border-radius: 0px;
-    border-width: 0px;
 }
 
-.toolbar GtkToolButton .button:active,
+SugarPaletteWindowWidget SugarRadioToolButton *:active {
+    background-color: @button_grey;
+    border-radius: $(toolbutton_padding)px;
+}
+
 SugarPaletteWindowWidget GtkToolButton .button:active {
-    background-color: @black;
-    border-radius: 0px;
+    background-color: @transparent;
 }
 
-.toolbar GtkToolButton .button:active:prelight,
-SugarPaletteWindowWidget GtkToolButton .button:active:prelight {
+.toolbar GtkToolButton .button:active {
     background-color: @button_grey;
-    border-radius: $(subcell_size)px;
-    border-width: $(default_padding)px;
-    border-color: transparent;
+    border-radius: $(toolbutton_padding)px;
 }
 
 /* Scales */