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: remove should not trigger popup show #162

Closed
wants to merge 1 commit into from
Closed

fix: remove should not trigger popup show #162

wants to merge 1 commit into from

Conversation

paranoidjk
Copy link
Member

@paranoidjk paranoidjk commented Feb 8, 2017 •
edited
Loading

业务线的反馈，mutliple 模式下点击删除icon每次都会触发列表的显示，比较烦人

demo： https://ant.design/components/select-cn/#components-select-demo-multiple

@paranoidjk paranoidjk changed the title fix: remove do not trigger popup show [WIP] fix: remove do not trigger popup show Feb 8, 2017
@paranoidjk paranoidjk changed the title [WIP] fix: remove do not trigger popup show fix: remove should not trigger popup show Feb 8, 2017
@coveralls
Copy link

coveralls commented Feb 8, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.02%) to 95.462% when pulling 474b773 on paranoidjk:fix-select-remove into 1059910 on react-component:master.

@paranoidjk paranoidjk mentioned this pull request Feb 9, 2017
Closed
@paranoidjk
Copy link
Member Author

ping @afc163

@yiminghe
Copy link
Member

同已有实践一致 https://select2.github.io/examples.html

@afc163 afc163 assigned yiminghe and unassigned yiminghe Feb 15, 2017
@paranoidjk paranoidjk closed this Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yiminghe yiminghe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@paranoidjk @coveralls @yiminghe