-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
Fixed non-selectable values in TypeaheadAnswer #232
Conversation
✅ Deploy Preview for s-forms-kbss ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
FYI @VojtechLunak |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@LaChope So it solves the issue in typeahead but the same solution cannot be applied to inteligent-tree-component?
@LaChope in what state is this PR? |
@blcham it does work in s-forms but not in s-forms-demo... edit: It actually works, it was issue in s-forms-demo itself, see kbss-cvut/s-forms-demo#14 |
LGTM |
Resolves #231 and #218.
It seems like react-select does not work well in combination of react-window (see this issue).
Instead react-windowed-select is working (see JedWatson/react-select#2850 (comment))