diff --git a/oxlint-suppressions.json b/oxlint-suppressions.json index 9b760d2f0c5..578512c2c93 100644 --- a/oxlint-suppressions.json +++ b/oxlint-suppressions.json @@ -5142,14 +5142,6 @@ "count": 1 } }, - "web/app/components/workflow/panel/chat-variable-panel/index.tsx": { - "jsx_a11y/click-events-have-key-events": { - "count": 1 - }, - "jsx_a11y/no-static-element-interactions": { - "count": 1 - } - }, "web/app/components/workflow/panel/chat-variable-panel/type.ts": { "erasable-syntax-only/enums": { "count": 1 diff --git a/web/app/components/workflow/panel/chat-variable-panel/__tests__/index.spec.tsx b/web/app/components/workflow/panel/chat-variable-panel/__tests__/index.spec.tsx index 4467a69c075..800b8452de8 100644 --- a/web/app/components/workflow/panel/chat-variable-panel/__tests__/index.spec.tsx +++ b/web/app/components/workflow/panel/chat-variable-panel/__tests__/index.spec.tsx @@ -197,7 +197,7 @@ describe('ChatVariablePanel', () => { it('should toggle the tips area and close the panel', async () => { const user = userEvent.setup() - const { container } = render() + render() expect(screen.getByText('workflow.chatVariable.panelDescription')).toBeInTheDocument() @@ -205,10 +205,7 @@ describe('ChatVariablePanel', () => { await user.click(toggleTipButton) expect(screen.queryByText('workflow.chatVariable.panelDescription')).not.toBeInTheDocument() - const closeButton = container.querySelector( - '.flex.size-6.cursor-pointer.items-center.justify-center', - ) as HTMLElement - await user.click(closeButton) + await user.click(screen.getByRole('button', { name: 'common.operation.close' })) expect(mockSetShowChatVariablePanel).toHaveBeenCalledWith(false) }) diff --git a/web/app/components/workflow/panel/chat-variable-panel/index.tsx b/web/app/components/workflow/panel/chat-variable-panel/index.tsx index ef3851d7c28..6bb898b8812 100644 --- a/web/app/components/workflow/panel/chat-variable-panel/index.tsx +++ b/web/app/components/workflow/panel/chat-variable-panel/index.tsx @@ -217,12 +217,14 @@ const ChatVariablePanel = () => { > -
$['operation.close'], { ns: 'common' })} + className="flex size-6 cursor-pointer appearance-none items-center justify-center rounded-md border-0 bg-transparent p-0 focus-visible:ring-2 focus-visible:ring-state-accent-solid focus-visible:outline-hidden" onClick={() => setShowChatVariablePanel(false)} > - -
+