Adjust table row default height to 44px (#2595)

kirbydesign · Dec 19, 2022 · 9b4ab1a · 9b4ab1a
1 parent 92fc2a8
commit 9b4ab1a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/libs/designsystem/src/lib/components/data-table/table/table.component.scss b/libs/designsystem/src/lib/components/data-table/table/table.component.scss
@@ -1,6 +1,9 @@
 @use '@kirbydesign/core/src/scss/utils';
 @use '@kirbydesign/core/src/scss/interaction-state';
 
+// Height variables
+$cell-height: 44px;
+
 // Color variables
 $divider-color: utils.get-color('medium');
 $highlight-color: utils.get-color('light');
@@ -40,7 +43,8 @@ th {
 }
 
 td {
-  padding: utils.size('s');
+  height: $cell-height;
+  padding: utils.size('xxxs') utils.size('s');
   font-weight: utils.font-weight('normal');
   font-size: utils.font-size('s');
   color: utils.get-text-color('black');

diff --git a/libs/designsystem/src/lib/components/data-table/table/table.component.spec.ts b/libs/designsystem/src/lib/components/data-table/table/table.component.spec.ts
@@ -61,7 +61,7 @@ describe('TableComponent', () => {
 
     it('should style [tbody] cells correctly', () => {
       expect(element.children.namedItem('tbody').firstChild.firstChild).toHaveComputedStyle({
-        padding: `${size('s')}`,
+        padding: `${size('xxxs')} ${size('s')}`,
         'font-weight': fontWeight('normal'),
         'font-size': fontSize('s'),
         color: getTextColor('black'),