Skip to content

Commit

Permalink
Merge pull request #84 from aversini/fix(ButtonIcon)-default-display-…
Browse files Browse the repository at this point in the history
…should-not-be-block-

fix(ButtonIcon): default display should not be block (or flex)
  • Loading branch information
aversini authored Nov 26, 2023
2 parents e9c334f + f7ce436 commit 44c2eb0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ describe("ButtonIcon modifiers", () => {
const button = await screen.findByRole("button");
expect(button.className).toContain("p-1");
expectToHaveClasses(button, [
"flex",
"inline-flex",
"items-center",
"justify-center",
"rounded-full",
Expand Down
2 changes: 1 addition & 1 deletion packages/ui-components/src/components/Button/utilities.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ const getButtonSizesClasses = ({
});

case TYPE_ICON:
return clsx("flex items-center justify-center", {
return clsx("inline-flex items-center justify-center", {
"w-6 p-0": size === "small" || slim,
"w-8 p-1": size === "medium" && !slim,
"w-10 p-2": size === "large" && !slim,
Expand Down

0 comments on commit 44c2eb0

Please sign in to comment.