fix(tables): vertical spacings were wrong

Orange-OpenSource · Jan 2, 2020 · 02c7b9a · 02c7b9a
1 parent dcd2bbd
commit 02c7b9a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/scss/_tables.scss b/scss/_tables.scss
@@ -36,7 +36,7 @@
     display: inline-block;
     width: $o-table-icon-size;
     height: $o-table-icon-size;
-    margin: -#{$spacer / 10} map-get($spacers, 2) -#{$spacer / 10} 0;
+    margin: -#{$spacer / 5} map-get($spacers, 2) -#{$spacer / 5} 0;
     font-size: $o-table-icon-size;
   }
 
@@ -50,8 +50,8 @@
   }
 
   .custom-control {
-    margin-top: .1875rem;
-    margin-bottom: .1875rem;
+    margin-top: 0;
+    margin-bottom: 0;
   }
   // end mod
 
@@ -68,7 +68,6 @@
 .table-sm {
   th,
   td {
-    padding: $table-cell-padding-sm;
     line-height: $table-cell-line-height-sm;
   }
 }

diff --git a/scss/_variables.scss b/scss/_variables.scss
@@ -421,10 +421,9 @@ $hr-margin-y:                 $spacer !default;
 //
 // Customizes the `.table` component with basic values, each used across all table variations.
 
-$table-cell-padding:          .75rem map-get($spacers, 2) .875rem !default; // Boosted mod
-$table-cell-padding-sm:       .5rem map-get($spacers, 2) map-get($spacers, 2) !default; // Boosted mod
+$table-cell-padding:          .5625rem map-get($spacers, 2) map-get($spacers, 2) !default; // Boosted mod
 // stylelint-disable function-blacklist
-$table-cell-line-height:      calc(22 / 16) !default; // Boosted mod
+$table-cell-line-height:      calc(30 / 16) !default; // Boosted mod
 $table-cell-line-height-sm:   calc(20 / 16) !default; // Boosted mod
 // stylelint-enable function-blacklist