chore: PR review updates

jaaydenh · jaaydenh · commit 589a27a8e7d5 · 2025-11-10T22:07:37.000Z
diff --git a/site/src/components/CopyableValue/CopyableValue.tsx b/site/src/components/CopyableValue/CopyableValue.tsx
@@ -40,10 +40,10 @@ export const CopyableValue: FC<CopyableValueProps> = ({
 		<TooltipProvider delayDuration={100}>
 			<Tooltip
 				open={tooltipOpen}
-				onOpenChange={(next) => {
+				onOpenChange={(shouldBeOpen) => {
 					// Always keep the tooltip open when in focus to handle issues when onOpenChange is unexpectedly false
-					if (!next && isFocused) return;
-					setTooltipOpen(next);
+					if (!shouldBeOpen && isFocused) return;
+					setTooltipOpen(shouldBeOpen);
 				}}
 			>
 				<TooltipTrigger asChild>
diff --git a/site/src/pages/IconsPage/IconsPage.tsx b/site/src/pages/IconsPage/IconsPage.tsx
@@ -141,7 +141,7 @@ const IconsPage: FC = () => {
 					)}
 					{searchedIcons.map((icon) => (
 						<CopyableValue key={icon.url} value={icon.url}>
-							<div className="flex flex-col gap-4 items-center max-w-full m-3">
+							<div className="flex flex-col gap-4 items-center max-w-full p-3">
 								<img
 									alt={icon.url}
 									src={icon.url}
