[gnome-shell/wip/carlosg/pad-osd-fixes: 12/12] padOsd: Apply specific CSS to Button/Leader SVG classes

[Carlos Garnacho <carlosg src gnome org>]

commit a5371316b66a685e23b75965df935faf594bd5ac
Author: Carlos Garnacho <carlosg gnome org>
Date:   Fri May 29 17:12:09 2020 +0200

    padOsd: Apply specific CSS to Button/Leader SVG classes
    
    Applying a fill operation on the Leader line path seems to close
    it, resulting in filled polygon. Bug or not this is not the intended
    result here, we can do less ambiguously by not specifying the fill
    CSS property to the Leader class.
    
    Fixes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/2570

 js/ui/padOsd.js | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
---
diff --git a/js/ui/padOsd.js b/js/ui/padOsd.js
index 932f8597b3..4ab8a7baa4 100644
--- a/js/ui/padOsd.js
+++ b/js/ui/padOsd.js
@@ -383,10 +383,8 @@ var PadDiagram = GObject.registerClass({
 
         for (let i = 0; i < this._activeButtons.length; i++) {
             let ch = String.fromCharCode('A'.charCodeAt() + this._activeButtons[i]);
-            css += '.%s {'.format(ch);
-            css += '    stroke: %s !important;'.format(ACTIVE_COLOR);
-            css += '    fill: %s !important;'.format(ACTIVE_COLOR);
-            css += '}';
+            css += '.%s.Leader { stroke: %s !important; }'.format(ch, ACTIVE_COLOR);
+            css += '.%s.Button { stroke: %s !important; fill: %s !important; }'.format(ch, ACTIVE_COLOR, 
ACTIVE_COLOR);
         }
 
         return css;