Blank cell unexpectedly inserted (IE9 only) #128

Closed
brianlow opened this Issue Sep 4, 2012 · 14 comments

Projects

None yet

3 participants

@brianlow

JsFiddle:
http://jsfiddle.net/Mottie/haeVa/4/

Screenshot in IE9:
http://dl.dropbox.com/u/1187645/TableSorterIssue.png

In Internet Explorer 9: the occasional, random row in the table has an extra blank cell inserted in the second column. In this fiddle see row 87. It is rendering as if I had inserted an extra td element in the row. Small changes in the data lead to wildly different rows exhibiting this problem (or no rows at all). Renders correctly in Chrome.

Details:
IE 9.0.8112.16421 in IE9 standard mode, Win 7 64-bit

@brianlow

FYI.. it doesn't appear to be a problem with special characters. The random data in the jsFiddle is all lowercase letters. The jsFiddle is as small as I could make it - removing any additional column/rows and the problem did not occur.

@thezoggy
Collaborator

confirmed this happens for row 87, then again on row 163.. (note that the tbody tr's start at 0). IE9. if you put it in compatibility mode it goes away. if you delete the row the issue just shifts itself.. tried jquery 1.7.2 as 1.8.0 is buggy... same problem. tried tablesorter 2.4 (unreleased) and confirmed the problem went away..

@Mottie
Owner

Thanks for checking this Zoggy... I've been busy taking care of my father, who just got out of the hospital, so I haven't had much time for working on this script lately.

@thezoggy
Collaborator

no worries, i just got back from vacation (mexico) yesterday. so im still getting back into the swing of things myself

@brianlow

Thanks. I'll watch for 2.4.

@Mottie
Owner

I'm not sure what I did to fix this issue, but version 2.4 is now live!

@Mottie Mottie closed this Sep 27, 2012
@brianlow

We still see this problem on IE9 (in our app and in the jsfiddle mentioned above see row 87)

@Mottie Mottie reopened this Sep 27, 2012
@thezoggy
Collaborator

i confirm i do still see the problem with IE9 using the latest (2.4) on mottie's github.

@thezoggy
Collaborator

replaced the github js with the dropbox version.. problem went away. d/led and compared both files (gh vs dropbox) and they match.

@thezoggy
Collaborator

ok.. so if i tried using the gh file but add a '?test' at the end.. it magically works now. looks to be some cache issue on jsfiddle's site?
http://jsfiddle.net/haeVa/2/

@Mottie
Owner

Hmm, wow that is odd. Thanks for helping out thezoggy! We'll have to wait until next week to chat, I have a busy weekend coming up.

@Mottie
Owner

I discovered that if you completely remove all white space from the table HTML, the problem disappears - see http://jsfiddle.net/haeVa/5/

Then, I found this blog post - http://hsmoore.com/blog/ie9-adding-empty-table-cells-in-large-table/

So, I don't think I'll add code to strip out the HTML white-space because it'll just slow down the initialization.

@Mottie Mottie closed this Oct 13, 2012
@thezoggy
Collaborator

let the jquery ui dev guys know about that bug.. hehe they were equally surprised about that bug:

[02:48pm] <_zoggy_> scott_gonzalez: http://hsmoore.com/blog/ie9-adding-empty-table-cells-in-large-table/
[02:48pm] <scott_gonzalez> WTF?
[02:49pm] <scott_gonzalez> I've never heard of that before. That's good to know.
[02:56pm] <ryanneufeld> _zoggy_: I am also wtf on that one.
[02:56pm] <DaveMethvin> _zoggy_: wow, i'd never heard of that bug
@Mottie
Owner

Adding a comment here to share another (untested) fix I found - use this css for IE9:

td { white-space: nowrap; }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment