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



commit d1606774b9f8500c51bd0e4fed6de9cc683bd69e
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
    
    https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1290

 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 85576b50e7..55d014e69e 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;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]