📏 fix: Size Agent Builder Tool and Skill Rows (#15134)

The info and remove buttons on each Tools/Skills row passed `size-6 p-0`
through className without a `size` prop. tailwind-merge 1.14.0 has no
`size-*` group, so `size-6` never conflicted with the default size
variant's `h-10 px-4 py-2` and the buttons rendered at 40px. Because they
sit at opacity-0 until hover, the rows read as 52px of mostly empty
padding.

Pass size="icon-xs" so the recipe replaces the default outright. Rows go
from 52px to 40px.
This commit is contained in:
Marco Beretta 2026-08-23 16:36:01 +02:00 committed by GitHub
parent 1d9c2fc591
commit 69bf8ffdf1
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -81,12 +81,13 @@ function ToolRowImpl({ item, onInfo, onRemove }: Props) {
>
<Button
variant="ghost"
size="icon-xs"
onClick={() => onInfo(item)}
aria-label={
configurable ? localize('com_ui_tools_configure') : localize('com_ui_tools_info')
}
className={cn(
'size-6 rounded-md p-0 text-text-secondary',
'rounded-md text-text-secondary',
'hover:text-text-secondary focus-visible:opacity-100',
)}
>
@ -94,10 +95,11 @@ function ToolRowImpl({ item, onInfo, onRemove }: Props) {
</Button>
<Button
variant="ghost"
size="icon-xs"
onClick={() => onRemove(item)}
aria-label={localize('com_ui_tools_remove')}
className={cn(
'size-6 rounded-md p-0 text-text-secondary',
'rounded-md text-text-secondary',
'hover:text-text-secondary focus-visible:opacity-100',
)}
>