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

Method to update typeahead label for custom datamodel #115

Conversation

JohnKuan
Copy link
Contributor

add a method to allow better control of text update to the in-build TextEditingController

@danvick
Copy link
Collaborator

danvick commented Aug 22, 2019

Hi @JohnKuan,
Thanks for your contribution.
This is something that has been requested for quite a while.
I'll have a look to review the changes before merging and let you know if any action is needed.

Have a good one.
Cheers!

@danvick danvick self-assigned this Aug 22, 2019
@JohnKuan
Copy link
Contributor Author

Sure @danvick.

@danvick danvick merged commit b4db766 into flutter-form-builder-ecosystem:master Aug 29, 2019
@danvick
Copy link
Collaborator

danvick commented Aug 29, 2019

Hi @JohnKuan,
I love your implementation of this feature. I've fixed a few issues including the assertion for type checking.
How do you like changing onSuggestionSelectedLabel to a more intuitive (IMO) selectionToTextTransformer?

@JohnKuan
Copy link
Contributor Author

JohnKuan commented Sep 2, 2019

Hi @danvick,

I just saw your merge. Thank you for fixing the issues on the type assertion.

I also noticed you removed the onSuggestionSelected field from the params. I actually do have a use for it, which is separated from populating the textfieldcontroller.text. I tried to utilised the selectionToTextTransformer field but it resulted in an infinite init loop on my use case. Would it okay to restore that field?

@danvick
Copy link
Collaborator

danvick commented Sep 3, 2019

Hello @JohnKuan,
I apologize for removing the field, I'll work to restore it ASAP. I had realized that I wasn't using it initially, that's why I had it removed.
I'll be rolling out v3.5.2 in a few which will contain the fix; I'll let you know when I do.

@danvick
Copy link
Collaborator

danvick commented Sep 3, 2019

Now about this infinite loop, is it something that's caused by the changes I had introduced? Can I have the sample code that causes this?

@danvick
Copy link
Collaborator

danvick commented Sep 3, 2019

3.5.2 is published.

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.

2 participants