Last modified: 2009-01-02 00:24:03 UTC
Created attachment 5609 [details] Patch against r44879 Sometimes it would be useful when specific rows of a sortable table would not change their position while reordering. E.g. repeated captions should be sticky. I propose to overload the class unsortable for rows.
This is similar to bug 16347.
(In reply to comment #1) > This is similar to bug 16347. Why did you mark it as invalid?
I didn't. The original bug-reporter did. I don't know why, though.
Patch applied with r45060.
It seems that this patch is creating some trouble with tablesorting for users in some cases. Please see reports here: http://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Sortable_tables_.26_Firefox_3.0.5
I see: Only users with bad coded scripts are affected. The line > for(var i in staticRows) { needs to be changed to > for(var i = 0; i < staticRows.length; ++i) {
(In reply to comment #6) > I see: Only users with bad coded scripts are affected. > > The line > > for(var i in staticRows) { > needs to be changed to > > for(var i = 0; i < staticRows.length; ++i) { Already did it in r45304. Reclosing unless it can be demonstrated that a) this commit caused the problem, b) it's not already fixed. (And in that case, might be better to file a bug depending on this one.)
