[gnome-shell-sass] theme: Apply spacing to .overview-icon-with-label correctly



commit ae856f19a3568970470fe14fa6cc4b812dc24ecc
Author: nana-4 <hnmaigo gmail com>
Date:   Wed Feb 5 01:03:05 2020 +0900

    theme: Apply spacing to .overview-icon-with-label correctly
    
    - Remove margin-bottom for StIcon, which is not only ineffective, but
      also created a bug in app folders.
    - Remove ">" which is invalid for overview icons that are not app
      folders.
    - Apply spacing to the correct target "StBoxLayout", not the parent
      .overview-icon.overview-icon-with-label.
    
    Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/2129

 widgets/_app-grid.scss | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
---
diff --git a/widgets/_app-grid.scss b/widgets/_app-grid.scss
index cc2b7a7..ba6eb60 100644
--- a/widgets/_app-grid.scss
+++ b/widgets/_app-grid.scss
@@ -11,7 +11,6 @@ $app_icon_padding: 24px;
 
   .overview-icon {
     icon-size: $app_icon_size;
-    StIcon { margin-bottom: $base_margin; } // margin on icon so label isn't close
   }
 }
 
@@ -36,9 +35,12 @@ $app_grid_fg_color: #fff;
 %app-well-app {
   @include overview-icon($app_grid_fg_color);
 
-  > .overview-icon.overview-icon-with-label {
+  .overview-icon.overview-icon-with-label {
     padding: 10px 8px 5px 8px;
-    spacing: $base_spacing;
+
+    > StBoxLayout {
+      spacing: $base_spacing;
+    }
   }
 }
 


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