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

Updates #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Updates #7

wants to merge 4 commits into from

Conversation

smithk86
Copy link
Contributor

@smithk86 smithk86 commented Nov 2, 2011

Modified column exclusion array (amExclude) to match to mDataProp value of each column.
This will allow you to match to either the column index or an mDataProp value.

Update documentation for amExclude and bShowAll options.

…ue of each column.

This will allow you to match to either the column index or an mDataProp value.

Update documentation for amExclude and bShowAll options.
Conflicts:
	media/js/ColVis.js
@smithk86
Copy link
Contributor Author

smithk86 commented Nov 3, 2011

Created another option called iDrawDelay. This is helpful with large data sets because it allows you to alter multiple columns before the table is redrawn.

@DataTables
Copy link
Collaborator

Sorry for the massive delay in replying to this pull request. This looks like a really nice one, but I think we need to be able to parse aiExclude from older versions for backwards compatibility. DataTables 1.10 is going to be providing stronger support for 'sName' in the API, so I think it would probably be best to use sName rather than mDataProp as the trigger for the columns - this is because mDataProp can be a function in DataTables, while sName is an internal reference name for the client-side developer only.

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