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

Allow cssInfoBlock to be the first tbody element #68

Closed
wants to merge 1 commit into from

Conversation

kristerkari
Copy link

Currently you can't have tbody with a cssInfoBlock class as the first tbody element. To fix this, I've added a simple loop to check for cssInfoBlock class rather than selecting the first tbody element.

I don't know about the additional plugins (I hope that this does not break any compatibility), but I guess a similar check should be added to those as well.

@Mottie
Copy link
Owner

Mottie commented May 10, 2012

Thanks! I'll look into this :)

But I think it might be easier to just use:

tBodies = $(table.tBodies).filter(':not(.' + c.cssInfoBlock + ')')[0]

I'll look into it more when I get a chance.

@kristerkari
Copy link
Author

Yeah, I guess it is simpler to just use one jQuery selector to do the job. Less code :)

@Mottie
Copy link
Owner

Mottie commented May 11, 2012

Ok, I've added the method I shared above in version 2.3.2. Thanks for sharing!

@Mottie Mottie closed this May 11, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants