mirror of
https://gitee.com/baidu/amis.git
synced 2024-11-29 18:48:45 +08:00
fix: SearchBox组件开启输入法提交数据错误
This commit is contained in:
parent
19b040fc6f
commit
f684c1856b
@ -31,6 +31,7 @@ class InputInner extends React.Component<InputProps, InputState> {
|
||||
@autobind
|
||||
handleComposition(e: React.CompositionEvent<HTMLInputElement>) {
|
||||
this.isOnComposition = e.type !== 'compositionend';
|
||||
|
||||
if (!this.isOnComposition) {
|
||||
this.handleChange(e as any);
|
||||
}
|
||||
@ -47,6 +48,17 @@ class InputInner extends React.Component<InputProps, InputState> {
|
||||
});
|
||||
}
|
||||
|
||||
@autobind
|
||||
handleKeyDown(e: React.KeyboardEvent<any>) {
|
||||
const {onKeyDown} = this.props;
|
||||
|
||||
if (this.isOnComposition) {
|
||||
return;
|
||||
}
|
||||
|
||||
onKeyDown?.(e);
|
||||
}
|
||||
|
||||
render() {
|
||||
const {forwardedRef, ...rest} = this.props;
|
||||
|
||||
@ -57,6 +69,7 @@ class InputInner extends React.Component<InputProps, InputState> {
|
||||
value={this.state.value}
|
||||
ref={forwardedRef}
|
||||
onChange={this.handleChange}
|
||||
onKeyDown={this.handleKeyDown}
|
||||
onCompositionStart={this.handleComposition}
|
||||
onCompositionUpdate={this.handleComposition}
|
||||
onCompositionEnd={this.handleComposition}
|
||||
|
@ -8,6 +8,7 @@ import {uncontrollable} from 'amis-core';
|
||||
import {autobind, isMobile} from 'amis-core';
|
||||
import {LocaleProps, localeable} from 'amis-core';
|
||||
import chain from 'lodash/chain';
|
||||
import Input from './Input';
|
||||
|
||||
export interface HistoryRecord {
|
||||
/** 历史记录值 */
|
||||
@ -306,17 +307,17 @@ export class SearchBox extends React.Component<SearchBoxProps, SearchBoxState> {
|
||||
)}
|
||||
style={style}
|
||||
>
|
||||
<input
|
||||
<Input
|
||||
name={name}
|
||||
ref={this.inputRef}
|
||||
disabled={disabled}
|
||||
placeholder={__(placeholder || 'placeholder.enter')}
|
||||
value={inputValue ?? ''}
|
||||
autoComplete="off"
|
||||
onFocus={this.handleFocus}
|
||||
onBlur={this.handleBlur}
|
||||
onChange={this.handleChange}
|
||||
onKeyDown={this.handleKeyDown}
|
||||
value={inputValue ?? ''}
|
||||
disabled={disabled}
|
||||
placeholder={__(placeholder || 'placeholder.enter')}
|
||||
autoComplete="off"
|
||||
/>
|
||||
|
||||
{!mini && clearable && inputValue && !disabled ? (
|
||||
|
@ -6,10 +6,11 @@
|
||||
* 3. 可清除 clearable
|
||||
* 4. mini 模式
|
||||
* 5. 立即搜索 searchImediately、样式 className
|
||||
* 6. Composition触发
|
||||
*/
|
||||
|
||||
import React from 'react';
|
||||
import {fireEvent, render} from '@testing-library/react';
|
||||
import {fireEvent, render, screen} from '@testing-library/react';
|
||||
import '../../src';
|
||||
import {render as amisRender} from '../../src';
|
||||
import {makeEnv, wait} from '../helper';
|
||||
@ -36,7 +37,7 @@ test('Renderer:Searchbox', async () => {
|
||||
)
|
||||
);
|
||||
|
||||
await wait(200);
|
||||
await wait(500);
|
||||
|
||||
expect(fetcher).toHaveBeenCalledTimes(1);
|
||||
expect(fetcher.mock.calls[0][0].query).toEqual({
|
||||
@ -212,3 +213,34 @@ test('Renderer:Searchbox with searchImediately & className', async () => {
|
||||
keywords: 'aabb'
|
||||
});
|
||||
});
|
||||
|
||||
test('6. Renderer: Searchbox is not supposed to be triggered with composition input', async () => {
|
||||
const onQuery = jest.fn();
|
||||
const {container} = render(amisRender({
|
||||
type: "search-box",
|
||||
name: "keywords",
|
||||
}, {
|
||||
onQuery
|
||||
}))
|
||||
|
||||
const inputEl = container.querySelector('.cxd-SearchBox input')!;
|
||||
expect(inputEl).toBeInTheDocument();
|
||||
|
||||
/** 第一次输入 Enter 后,文本填入 */
|
||||
|
||||
fireEvent.compositionStart(inputEl);
|
||||
fireEvent.keyDown(inputEl, { key: 'Enter', keyCode: 13 });
|
||||
await wait(200);
|
||||
expect(onQuery).not.toHaveBeenCalled();
|
||||
|
||||
/** 退出输入法,触发搜索 */
|
||||
fireEvent.compositionEnd(inputEl);
|
||||
fireEvent.change(inputEl, {target: {value: 'test'}});
|
||||
fireEvent.keyDown(inputEl, { key: 'Enter', keyCode: 13 });
|
||||
await wait(200);
|
||||
|
||||
expect(onQuery).toHaveBeenCalledTimes(1);
|
||||
expect(onQuery.mock.calls[0][0]).toEqual({
|
||||
keywords: 'test'
|
||||
});
|
||||
})
|
||||
|
@ -22,6 +22,7 @@ exports[`Renderer:Searchbox 1`] = `
|
||||
autocomplete="off"
|
||||
name="keywords"
|
||||
placeholder="请输入"
|
||||
type="text"
|
||||
value="searchkey"
|
||||
/>
|
||||
<a
|
||||
@ -49,6 +50,7 @@ exports[`Renderer:Searchbox with clearable 1`] = `
|
||||
autocomplete="off"
|
||||
name="keywords"
|
||||
placeholder="请输入"
|
||||
type="text"
|
||||
value="tmpvalue"
|
||||
/>
|
||||
<div
|
||||
@ -93,6 +95,7 @@ exports[`Renderer:Searchbox with enhance 1`] = `
|
||||
autocomplete="off"
|
||||
name="keywords"
|
||||
placeholder="请输入"
|
||||
type="text"
|
||||
value=""
|
||||
/>
|
||||
<a
|
||||
@ -120,6 +123,7 @@ exports[`Renderer:Searchbox with mini 1`] = `
|
||||
autocomplete="off"
|
||||
name="keywords"
|
||||
placeholder="请输入"
|
||||
type="text"
|
||||
value=""
|
||||
/>
|
||||
<a
|
||||
@ -143,6 +147,7 @@ exports[`Renderer:Searchbox with mini 2`] = `
|
||||
autocomplete="off"
|
||||
name="keywords"
|
||||
placeholder="请输入"
|
||||
type="text"
|
||||
value="what?"
|
||||
/>
|
||||
<a
|
||||
|
Loading…
Reference in New Issue
Block a user