Skip to content

Commit

Permalink
fix: core linedTable variant works again (#3722)
Browse files Browse the repository at this point in the history
* fix: core `linedTable` works again

* Create great-horses-hope.md
  • Loading branch information
sebald authored Feb 19, 2024
1 parent 61635aa commit c61895d
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 6 deletions.
6 changes: 6 additions & 0 deletions .changeset/great-horses-hope.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
"@marigold/components": patch
"@marigold/theme-core": patch
---

fix: core `linedTable` variant works again
2 changes: 1 addition & 1 deletion packages/components/src/Table/Table.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ const meta = {
control: {
type: 'select',
},
options: ['none', 'grid'],
options: ['none', 'grid', 'linedTable'],
description: 'variant for the table',
},
size: {
Expand Down
12 changes: 7 additions & 5 deletions themes/theme-core/src/components/Table.styles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,18 +30,20 @@ export const Table: ThemeComponent<'Table'> = {
},
}
),
row: cva([
'group-aria-[multiselectable]/table:[&>*:first-child]:w-12',
'selected:bg-bg-selected focus:outline-outline-focus',
row: cva(
[
'group-aria-[multiselectable]/table:[&>*:first-child]:w-12',
'selected:bg-bg-selected focus:outline-outline-focus',
],
{
variants: {
variant: {
linedTable: ['border-border-inverted border-b'],
grid: ['border-border-inverted border'],
},
},
},
]),
}
),
cell: cva(['text-text-base p-2', 'focus:outline-outline-focus'], {
variants: {
variant: {
Expand Down

0 comments on commit c61895d

Please sign in to comment.