Skip to content

Commit

Permalink
feat: handle justify-content for <DataTableAction /> and remove margi…
Browse files Browse the repository at this point in the history
…n-right for <Tag /> (#21)

* feat: add css var to hanel justify content and remove margin-right for <Tag />

* fix: less autofix

---------

Co-authored-by: Luca Tagliabue <[email protected]>
  • Loading branch information
lucataglia and Luca Tagliabue authored Nov 18, 2024
1 parent 459e0c2 commit bfe9c21
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions src/lib/components/data-table/DataTableAction.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import classNames from 'classnames';
const DataTableAction = ({
children,
noHide,
...other
}: {
children: React.ReactNode;
noHide?: string;
Expand All @@ -21,6 +22,7 @@ const DataTableAction = ({
role="presentation"
className={`l-data-table__action ${css}`}
onClick={handleOnClick}
{...other}
>
{children}
</div>
Expand Down
2 changes: 1 addition & 1 deletion src/lib/components/data-table/_styles.less
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
gap: 1rem;
display: flex;
flex-direction: row;
justify-content: flex-end;
justify-content: var(--data-table-action-justify-content, flex-end);

&--noHide {
opacity: 1;
Expand Down
1 change: 1 addition & 0 deletions src/lib/components/tag/_styles.less
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
height: 1rem;
padding: 0 .5rem;
width: fit-content;
margin: 0;

&:hover {
opacity: 1;
Expand Down

0 comments on commit bfe9c21

Please sign in to comment.