Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix issue that onFocus will trigger twice #156

Merged
merged 1 commit into from
Jan 20, 2017
Merged

Fix issue that onFocus will trigger twice #156

merged 1 commit into from
Jan 20, 2017

Conversation

RaoHai
Copy link
Contributor

@RaoHai RaoHai commented Jan 17, 2017

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling f62ea11 on RaoHai:focusTwice into ** on react-component:master**.

afc163 referenced this pull request in ant-design/ant-design Jan 17, 2017
@RaoHai @afc163
Customize Input Component of AutoComplete (#4483)
46c2008 
* cherry-pick

* remove useless files

* remove useless files

* Group ts

* update interface

* ReactElement
@RaoHai RaoHai added the bug label Jan 19, 2017
@RaoHai
Copy link
Contributor Author

RaoHai commented Jan 19, 2017

@yiminghe @yesmeck review 下。。

@yesmeck
Copy link
Member

yesmeck commented Jan 19, 2017

这个的 focus 是不是也要处理 https://github.com/RaoHai/select/blob/f62ea11c6bd4c32d4cf559fda4007c75716a3c5e/src/Select.jsx#L614 ？

@RaoHai
Copy link
Contributor Author

RaoHai commented Jan 20, 2017

不用处理。

@yiminghe yiminghe merged commit 4a2f353 into react-component:master Jan 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yesmeck yesmeck yesmeck approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Select 的onFocus事件 在设置showSearch={true}后，会触发2次
4 participants
@RaoHai @coveralls @yesmeck @yiminghe