-
Notifications
You must be signed in to change notification settings - Fork 33
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
hide and disable the table selection inputs #220
Conversation
@@ -54,6 +54,7 @@ function initialize( | |||
rows = 11.5, // maximum number of rows to show | |||
width = {}, // object of column name to width, or overall table width | |||
multiple = true, | |||
select: selectable = true, // is the table selectable? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I find selectable
a bit more descriptive than select
as an option. Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no preference; in the code I've been using selectable because the name is already used by the select function.
Do we normally include the changes to the documentation (README) to these PRs or are those done separately? |
Merged into #262. |
closes #136