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.

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

Feature autocomplete-select onChange #1466

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

c3dr0x
Copy link
Contributor

@c3dr0x c3dr0x commented Sep 28, 2017

[Component] Autocomplete-select

Fix #1448

Description

Add a new option prop : wholeItem.

Setting it to true will change the behavior of component to manage in value, the whole item instead of its key. Same goes for onChange method.

Breaking change

If the override renderOptions method is used, then it should be updated because now the options within state contains items and not items' labels.

@c3dr0x c3dr0x self-assigned this Sep 28, 2017
@c3dr0x c3dr0x requested a review from Hartorn September 28, 2017 11:37
@c3dr0x c3dr0x changed the title Feature autocomplete on change Feature autocomplete-select on change Sep 28, 2017
@c3dr0x c3dr0x changed the title Feature autocomplete-select on change Feature autocomplete-select onChange Sep 28, 2017
@Hartorn Hartorn force-pushed the feature-autocomplete-onChange branch from d587f40 to cd0dd68 Compare October 8, 2017 19:27
@Hartorn
Copy link
Contributor

Hartorn commented Oct 8, 2017

@c3dr0x I rebased it, and made some changes : _setValue needs the whole props value, to have latest keyresolver and so on.

Still need to test it, but static analyse looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Autocomplete-Select] Keep all data in options, and allow to retrieve them in onChange
5 participants