[libhandy] preferences-group: Refine the spacing



commit f79172286f2726739bfe4c2e55b930042e573365
Author: Adrien Plazas <kekun plazas laposte net>
Date:   Thu Dec 17 12:50:12 2020 +0100

    preferences-group: Refine the spacing
    
    This also implements the spacing via CSS.
    
    Fixes https://gitlab.gnome.org/GNOME/libhandy/-/issues/384

 src/hdy-preferences-group.ui       |  1 -
 src/themes/Adwaita-dark.css        |  4 ++++
 src/themes/Adwaita.css             |  4 ++++
 src/themes/HighContrast.css        |  4 ++++
 src/themes/HighContrastInverse.css |  4 ++++
 src/themes/_fallback-base.scss     | 12 ++++++++++++
 src/themes/fallback.css            |  4 ++++
 7 files changed, 32 insertions(+), 1 deletion(-)
---
diff --git a/src/hdy-preferences-group.ui b/src/hdy-preferences-group.ui
index c8656077..20f3bcac 100644
--- a/src/hdy-preferences-group.ui
+++ b/src/hdy-preferences-group.ui
@@ -6,7 +6,6 @@
       <object class="GtkBox" id="box">
         <property name="visible">True</property>
         <property name="orientation">vertical</property>
-        <property name="spacing">12</property>
         <child>
           <object class="GtkLabel" id="title">
             <property name="can_focus">False</property>
diff --git a/src/themes/Adwaita-dark.css b/src/themes/Adwaita-dark.css
index 2577143f..01aed3f4 100644
--- a/src/themes/Adwaita-dark.css
+++ b/src/themes/Adwaita-dark.css
@@ -83,6 +83,10 @@ statuspage > scrolledwindow > viewport > box > clamp > box > .title { margin-bot
 
 statuspage > scrolledwindow > viewport > box > clamp > box > .description { margin-bottom: 36px; }
 
+preferencesgroup > box > label:not(:first-child) { margin-top: 6px; }
+
+preferencesgroup > box > box:not(:first-child) { margin-top: 12px; }
+
 /*************************** Check and Radio buttons * */
 popover.combo list { min-width: 200px; }
 
diff --git a/src/themes/Adwaita.css b/src/themes/Adwaita.css
index 9824ca68..18ac2871 100644
--- a/src/themes/Adwaita.css
+++ b/src/themes/Adwaita.css
@@ -83,6 +83,10 @@ statuspage > scrolledwindow > viewport > box > clamp > box > .title { margin-bot
 
 statuspage > scrolledwindow > viewport > box > clamp > box > .description { margin-bottom: 36px; }
 
+preferencesgroup > box > label:not(:first-child) { margin-top: 6px; }
+
+preferencesgroup > box > box:not(:first-child) { margin-top: 12px; }
+
 /*************************** Check and Radio buttons * */
 popover.combo list { min-width: 200px; }
 
diff --git a/src/themes/HighContrast.css b/src/themes/HighContrast.css
index 7d295e26..ca762c92 100644
--- a/src/themes/HighContrast.css
+++ b/src/themes/HighContrast.css
@@ -83,6 +83,10 @@ statuspage > scrolledwindow > viewport > box > clamp > box > .title { margin-bot
 
 statuspage > scrolledwindow > viewport > box > clamp > box > .description { margin-bottom: 36px; }
 
+preferencesgroup > box > label:not(:first-child) { margin-top: 6px; }
+
+preferencesgroup > box > box:not(:first-child) { margin-top: 12px; }
+
 /*************************** Check and Radio buttons * */
 popover.combo list { min-width: 200px; }
 
diff --git a/src/themes/HighContrastInverse.css b/src/themes/HighContrastInverse.css
index a1735558..1468db73 100644
--- a/src/themes/HighContrastInverse.css
+++ b/src/themes/HighContrastInverse.css
@@ -83,6 +83,10 @@ statuspage > scrolledwindow > viewport > box > clamp > box > .title { margin-bot
 
 statuspage > scrolledwindow > viewport > box > clamp > box > .description { margin-bottom: 36px; }
 
+preferencesgroup > box > label:not(:first-child) { margin-top: 6px; }
+
+preferencesgroup > box > box:not(:first-child) { margin-top: 12px; }
+
 /*************************** Check and Radio buttons * */
 popover.combo list { min-width: 200px; }
 
diff --git a/src/themes/_fallback-base.scss b/src/themes/_fallback-base.scss
index 0f5cb6cf..26438e9e 100644
--- a/src/themes/_fallback-base.scss
+++ b/src/themes/_fallback-base.scss
@@ -169,4 +169,16 @@ statuspage > scrolledwindow > viewport > box {
   }
 }
 
+// Preferences
 
+preferencesgroup > box {
+  // Add space between the description and the title.
+  > label:not(:first-child) {
+    margin-top: 6px;
+  }
+
+  // Add space between the box and the labels.
+  > box:not(:first-child) {
+    margin-top: 12px;
+  }
+}
diff --git a/src/themes/fallback.css b/src/themes/fallback.css
index cffb9221..a1c7d479 100644
--- a/src/themes/fallback.css
+++ b/src/themes/fallback.css
@@ -82,3 +82,7 @@ statuspage > scrolledwindow > viewport > box > clamp > box > .icon { margin-bott
 statuspage > scrolledwindow > viewport > box > clamp > box > .title { margin-bottom: 12px; }
 
 statuspage > scrolledwindow > viewport > box > clamp > box > .description { margin-bottom: 36px; }
+
+preferencesgroup > box > label:not(:first-child) { margin-top: 6px; }
+
+preferencesgroup > box > box:not(:first-child) { margin-top: 12px; }


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