fix(ui): align infotip popover focus styles (#37377)

This commit is contained in:
yyh 2026-06-12 16:14:29 +08:00 committed by GitHub
parent 1f6b7a3c35
commit 514fddb60c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 7 additions and 1 deletions

View File

@ -29,6 +29,11 @@ describe('PopoverContent', () => {
await expect.element(screen.getByRole('group', { name: 'default positioner' })).toHaveAttribute('data-side', 'bottom')
await expect.element(screen.getByRole('group', { name: 'default positioner' })).toHaveAttribute('data-align', 'center')
await expect.element(screen.getByRole('dialog', { name: 'default popover' })).toHaveTextContent('Default content')
await expect.element(screen.getByRole('dialog', { name: 'default popover' })).toHaveClass(
'outline-hidden',
'focus:outline-hidden',
'focus-visible:outline-hidden',
)
})
it('should apply parsed custom placement and custom offsets when placement props are provided', async () => {

View File

@ -57,6 +57,7 @@ export function PopoverContent({
<BasePopover.Popup
className={cn(
'rounded-xl border-[0.5px] border-components-panel-border bg-components-panel-bg shadow-lg',
'outline-hidden focus:outline-hidden focus-visible:outline-hidden',
'origin-(--transform-origin) transition-[transform,scale,opacity] data-ending-style:scale-95 data-ending-style:opacity-0 data-starting-style:scale-95 data-starting-style:opacity-0 motion-reduce:transition-none',
popupClassName,
)}

View File

@ -71,7 +71,7 @@ export function Infotip({
aria-label={ariaLabel}
onClick={handleClick}
className={cn(
'inline-flex size-4 shrink-0 cursor-pointer items-center justify-center border-0 bg-transparent p-0 focus-visible:ring-1 focus-visible:ring-components-input-border-hover focus-visible:outline-hidden',
'inline-flex size-4 shrink-0 cursor-pointer items-center justify-center border-0 bg-transparent p-0 focus-visible:ring-2 focus-visible:ring-state-accent-solid focus-visible:outline-hidden',
className,
)}
>