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

aiExclude should also work with columns with 'mDataProp' #3

Closed
devdemon opened this issue Jul 11, 2011 · 2 comments
Closed

aiExclude should also work with columns with 'mDataProp' #3

devdemon opened this issue Jul 11, 2011 · 2 comments

Comments

@devdemon
Copy link

Hi,

Currently aiExclude only accepts integers, but if your columns have an mDataProp (or sName) defined, we should be able to use those names. Using colreorder makes column numbers useless.

Great work on this!

@DataTables
Copy link
Collaborator

This is an excellent point and one I have been debating quite a bit of late with myself. There is now significant overlap between sName, mDataProp, aTargets and column indexes to refer to columns. They all serve their own purpose of course, but you are absolutely correct that there should be an easier mapping between them.

I think the correct thing to do at the moment with how things stand in DataTables 1.x is to use sName, since it is the closest to referring to each columns individually (again - the others could do that...). So yup, perfectly valid and very good suggestion!

In DataTables 2.x (when I get going on that...) I will look at tying these related properties together a lot closer, and providing mapping abilities. For example wherever you can refer to a column index, you should also be ale to refer to sName (or whatever...).

@DataTables
Copy link
Collaborator

Just commented of a pull request (issue #7) which addresses this issue. Closing this issue to allow the discussion to continue in issue #7.

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

No branches or pull requests

0 participants