added IconSizeHelper top Popover trigger

linked-planet · May 6, 2024 · 366b802 · 366b802
1 parent 12e95d1
commit 366b802
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/library/src/components/DropdownMenu.tsx b/library/src/components/DropdownMenu.tsx
@@ -454,15 +454,15 @@ const Trigger = forwardRef<HTMLButtonElement, TriggerProps>(
 				{...rest}
 			>
 				{children}
-				<div
-					className={`h-full hidden items-center w-6 ${
+				<IconSizeHelper
+					className={`h-full hidden items-center justify-center w-6 ${
 						hideChevron ? "" : "group-data-[state=open]:flex"
 					}`}
 				>
 					<ChevronUpIcon label="" size="medium" />
-				</div>
+				</IconSizeHelper>
 				<IconSizeHelper
-					className={`h-full hidden items-center w-6 ${
+					className={`h-full hidden items-center justify-center w-6 ${
 						hideChevron ? "" : "group-data-[state=closed]:flex"
 					}`}
 				>

diff --git a/library/src/components/Popover.tsx b/library/src/components/Popover.tsx
@@ -6,6 +6,7 @@ import { twMerge } from "tailwind-merge"
 import { getPortal } from "../utils"
 import { Button, type ButtonProps } from "./Button"
 import { overlayBaseStyle } from "./styleHelper"
+import { IconSizeHelper } from "./IconSizeHelper"
 
 const portalDivId = "uikts-popover" as const
 
@@ -38,20 +39,20 @@ const Trigger = forwardRef<HTMLButtonElement, TriggerProps>(
 				{...rest}
 			>
 				{children}
-				<div
-					className={`h-full hidden items-center w-6 ${
+				<IconSizeHelper
+					className={`h-full hidden items-center justify-center w-6 ${
 						hideChevron ? "" : "group-data-[state=open]:flex"
 					}`}
 				>
 					<ChevronUpIcon label="" size="medium" />
-				</div>
-				<div
-					className={`h-full hidden items-center w-6 ${
+				</IconSizeHelper>
+				<IconSizeHelper
+					className={`h-full items-center justify-center hidden w-6 ${
 						hideChevron ? "" : "group-data-[state=closed]:flex"
 					}`}
 				>
 					<ChevronDownIcon label="" size="medium" />
-				</div>
+				</IconSizeHelper>
 			</Button>
 		)
 	},