Sorting doesn't work in IE 8 #170

Closed
johnjiang opened this Issue Nov 1, 2012 · 6 comments

Comments

Projects
None yet
3 participants

Example url: http://mottie.github.com/tablesorter/docs/example-widget-sticky-header.html

Tested using jquery 1.6.4 and 1.8.2

Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E; InfoPath.2; MS-RTC LM 8)
Timestamp: Thu, 1 Nov 2012 06:12:45 UTC

Message: 'normalized' is null or not an object
Line: 457
Char: 6
Code: 0
URI: http://mottie.github.com/tablesorter/js/jquery.tablesorter.js

Owner

Mottie commented Nov 1, 2012

This definitely seems to be a problem with the sticky header widget. This is probably due to the widget duplicating the thead, which is new in version 2.4+, instead of just duplicating the rows within the thead. I'll look into fixing this soon.

Thanks mate. Out of curiosity and somewhat unrelated. Is it possible to turn on sticky headers without sorting the table in any way?

Owner

Mottie commented Nov 2, 2012

I got the idea to copy the thead from jmosbech's plugin StickyTableHeaders, if that's what you mean.

I guess what I mean is currently I'm using 2 jquery plugins, tablesorter and this floating header plugin that does exactly what sticky headers does (http://archive.plugins.jquery.com/project/floatHeader). I don't want to have two plugins doing, when one does exactly what the other does.

So I guess the question is, is it possible to run the widgets independently of any sorting? For some tables that I maintain, it doesn't make sense for them to be sorted but they stretch for so long that it makes sense to have sticky headers turned on.

frinkz commented Nov 2, 2012

It sure would be nice if sorting itself was a widget, and we could just choose which widgets to run without having the bulk of the sorting code in there.

Mottie closed this Nov 14, 2012

Thanks for getting this fix in so quickly! Truly appreciate your work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment