Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(fix) call onSelect with interpolated data #222

Closed
wants to merge 1 commit into from

Conversation

MartinNuc
Copy link

@MartinNuc MartinNuc commented Jul 13, 2019

This should fix #221

I am trying to use this library with multi selection on.

Problem is that onSelect is called with a single value not the whole array. The reason is because interpolate function is used to update internal state but not for calling onSelect.

I believe this should fix the issue but I am new to this library so I would like to ask for a deeper code review. Thank you.

@MartinNuc MartinNuc force-pushed the interpolated-changes branch 2 times, most recently from e230846 to 9ad3d87 Compare July 13, 2019 19:54
@MartinNuc MartinNuc force-pushed the interpolated-changes branch from 9ad3d87 to 0491eb0 Compare July 13, 2019 19:55
@MartinNuc MartinNuc changed the title (fix) call onSelect after running interpolate function (fix) call onSelect with interpolated data Jul 13, 2019
@MartinNuc
Copy link
Author

Is it possible to merge? Otherwise I will close this PR as it has been here for one year already.

@MartinNuc
Copy link
Author

MartinNuc commented Jun 11, 2020

I am closing this PR. Apparently this open source project is dead ☠️

@MartinNuc MartinNuc closed this Jun 11, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interpolate function is called after onSelect is called
2 participants