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] Add selectOnClick option #290

Merged
merged 1 commit into from
Dec 16, 2016
Merged

[FEATURE] Add selectOnClick option #290

merged 1 commit into from
Dec 16, 2016

Conversation

offirgolan
Copy link
Collaborator

Resolves #289.

Select a row on click. If this is set to false and multiSelect is enabled, using click + shift, cmd, or ctrl will still work as intended, while clicking on the row will not set the row as selected.

/cc @karellm. Can you please check out this branch and test it out in your app?

@karellm
Copy link

karellm commented Dec 16, 2016

@offirgolan Thanks for the prompt answer! Testing it out right now.

@karellm
Copy link

karellm commented Dec 16, 2016

@offirgolan The behaviour looks exactly like what I had in mind. I'm not sure I'm the best person to do it but let me know if you also want a code review!

Thanks a lot for the great support 👍

@offirgolan
Copy link
Collaborator Author

@karellm great! Np at all, I realized the change was pretty minimal so I took a stab at it. I'll make a new minor release with this later today.

@offirgolan offirgolan merged commit 7ec547b into master Dec 16, 2016
@offirgolan offirgolan deleted the select-on-click branch December 16, 2016 22:02
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