Skip to content

Conversation

kagd
Copy link

@kagd kagd commented Jun 22, 2017

I discovered this issue when using ant-design. Since Ant-Design defaults filterOption to false I needed to pass set filterOption to true. When I did that I discovered that no options were being filtered. Here is a codepen showing this.

I tracked the problem line down to FilterMixin.js#L16. When you pass in true it doesn't use defaultFilter, but rather returns true for each option.

Grant Klinsing added 2 commits June 21, 2017 23:47
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 96.585% when pulling 566ef7b on kagd:master into 68bf5fe on react-component:master.

@yesmeck
Copy link
Member

yesmeck commented Jun 23, 2017

Thanks!

@yesmeck yesmeck merged commit b784436 into react-component:master Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants