Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

Commit f6fb309

Browse files
authored
fix: Ensure children are conditionally rendered in Tooltip component (#100)
* fix: Ensure children are conditionally rendered in Tooltip component * fix: Add padding class to Select component for improved layout
1 parent b3ffe0a commit f6fb309

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/components/extenders/Select.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,7 @@ export default function Select<
273273
setValue={v => setValue(v as Value)}
274274
value={value as string}
275275
defaultValue={multiple ? [] : undefined}>
276-
<Ariakit.Select className={mergeClass(base(), className)}>
276+
<Ariakit.Select className={mergeClass(base(), "pr-md", className)}>
277277
<div className={valueStyle()}>{label}</div>
278278
<div className={icon()}>
279279
{loading ? <Icon className="animate-spin" remix="RiLoader4Fill" /> : <Icon remix="RiArrowDropDownLine" />}

src/components/primitives/Tooltip.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export default function Tooltip({ helper, onOpen, children, icon = true, classNa
3535
<RadixTooltip.Trigger asChild>
3636
{/* biome-ignore lint/a11y/useKeyWithClickEvents: <explanation> */}
3737
<span className="flex items-center gap-sm" onClick={isTouchDevice ? () => setOpen(prev => !prev) : undefined}>
38-
<div>{children}</div>
38+
{children && <div>{children}</div>}
3939
{!!icon && <Icon className="text-main-12" remix="RiQuestionFill" alt="Tooltip" />}
4040
</span>
4141
</RadixTooltip.Trigger>

0 commit comments

Comments
 (0)