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

(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
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.

Interpolate function is called after onSelect is called
2 participants