sorttable.js breaks too much
Bug #345139 reported by
Brian Murray
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Eleanor Berger | ||
Bug Description
In sorttable.js the following code exists:
if ((' ' + cell.className + ' ').indexOf(" default-sort ") != -1) {
break;
}
if ((' ' + cell.className + ' ').indexOf(" default-revsort ") != -1) {
break;
}
if ((' ' + cell.className + ' ').indexOf(" initial-sort ") != -1) {
break;
}
However, the break statements make it so that you can't use something like 'class=
Revision history for this message
| Brian Murray (brian-murray) wrote | #1 |
| Changed in launchpad: | |
| assignee: | nobody → intellectronica |
| importance: | Undecided → Medium |
| milestone: | none → 2.2.4 |
| status: | New → Triaged |
| tags: | added: javascript |
| affects: | launchpad → malone |
| Changed in malone: | |
| milestone: | 2.2.4 → none |
| milestone: | none → 2.2.4 |
Revision history for this message
| Eleanor Berger (intellectronica) wrote | #3 |
| affects: | malone → launchpad-foundations |
| Changed in launchpad-foundations: | |
| milestone: | 2.2.4 → none |
| milestone: | none → 2.2.4 |
| Changed in launchpad-foundations: | |
| milestone: | 2.2.4 → 2.2.5 |
| assignee: | Tom Berger (intellectronica) → nobody |
| Changed in launchpad-foundations: | |
| assignee: | nobody → Tom Berger (intellectronica) |
| milestone: | 2.2.5 → 2.2.6 |
Revision history for this message
| Eleanor Berger (intellectronica) wrote | #4 |
| Changed in launchpad-foundations: | |
| status: | Triaged → Fix Committed |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I'm using this in the following the report:
http://
which I'd like sorted by bug gravity in descending order which isn't possible. I've already added the necessary formatting to the report:
<th class="default-sort initial-
so if sorttable.js is fixed the report should be correctly sorted.