Skip to content

Commit

Permalink
fix: don't show ActionBar.Group border when empty
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisvxd committed Oct 4, 2024
1 parent 31c1f9b commit 4345165
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
10 changes: 9 additions & 1 deletion packages/core/components/ActionBar/styles.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
display: flex;
width: auto;
padding: 4px;
padding-right: 0;
padding-right: 4px;
border-top-left-radius: 8px;
border-top-right-radius: 8px;
border-radius: 8px;
Expand Down Expand Up @@ -33,6 +33,14 @@
padding-right: 4px;
}

.ActionBar-group:last-of-type {
padding-right: 0;
}

.ActionBar-group:empty {
display: none;
}

.ActionBar-action {
background: transparent;
border: none;
Expand Down
4 changes: 1 addition & 3 deletions packages/core/components/DraggableComponent/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ const DefaultActionBar = ({
children: ReactNode;
}) => (
<ActionBar label={label}>
<ActionBar.Group>
<DefaultOverride>{children}</DefaultOverride>
</ActionBar.Group>
<ActionBar.Group>{children}</ActionBar.Group>
</ActionBar>
);

Expand Down

0 comments on commit 4345165

Please sign in to comment.