Skip to content
This repository has been archived by the owner on Aug 6, 2024. It is now read-only.

Commit

Permalink
review comment fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
albinAppsmith committed Oct 25, 2023
1 parent 37d9b33 commit b004483
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
3 changes: 2 additions & 1 deletion packages/design-system/src/Table/Table.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export default {
style={{
width: "60%",
height: "50vh",
overflow: "scroll",
display: "table",
}}
>
<Story />
Expand Down Expand Up @@ -315,6 +315,7 @@ export const TableStory: Story = {
},
],
sticky: true,
tableLayout: "fixed",
},
render: (args) => <Table {...args} />,
};
Expand Down
6 changes: 3 additions & 3 deletions packages/design-system/src/Table/Table.styles.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,9 @@ export const StyledHeaderCell = styled.th.attrs(({ className }) => ({
className: clsx(TableHeaderCellClassName, className),
}))`
&& {
font-size: 14px;
font-size: var(--ads-v2-font-size-4);
font-style: normal;
font-weight: 400;
font-weight: var(--ads-v2-font-weight-normal);
color: var(--ads-v2-colors-content-label-default-fg);
font-family: var(--ads-v2-font-family);
border: none;
Expand All @@ -60,7 +60,7 @@ export const StyledCell = styled.td.attrs(({ className }) => ({
className: clsx(TableBodyCellClassName, className),
}))`
&& {
font-size: 14px;
font-size: var(--ads-v2-font-size-4);
padding: var(--ads-v2-spaces-6) var(--ads-v2-spaces-5);
color: var(--ads-v2-colors-content-label-default-fg);
font-family: var(--ads-v2-font-family);
Expand Down
4 changes: 1 addition & 3 deletions packages/design-system/src/Table/Table.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,7 @@ function Table({ className, emptyText = NoData, ...props }: TableProps) {
},
};
return (
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-expect-error
<RcTable
<RcTable<any>
{...props}
className={clsx(TableWrapperClassName, className)}
components={components}
Expand Down

0 comments on commit b004483

Please sign in to comment.