Closed Bug 512559 Opened 15 years ago Closed 15 years ago

Use XUL template sorting to sort the add-ons lists

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.3a1
Tracking Status
status1.9.2 --- beta1-fixed

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

In order to properly sort the contents of two separate datasources into the same list we should just switch to using the XUL template sorting functions rather than doing it manually. The exception is the search pane where we have heard and footer items that have to be where we want them to be.
Assignee: nobody → dtownsend
Attached patch patch rev 1Splinter Review
Some nice code removal here though since it is an API change it requires SR. There is no reason that others should be calling these functions though.
Attachment #397401 - Flags: superreview?(benjamin)
Attachment #397401 - Flags: review?(robert.bugzilla)
Status: NEW → ASSIGNED
Attachment #397401 - Flags: review?(robert.bugzilla) → review+
Attachment #397401 - Flags: superreview?(benjamin) → superreview+
Landed: http://hg.mozilla.org/mozilla-central/rev/a81a4395dac3
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [needs baking]
Target Milestone: --- → mozilla1.9.3a1
Comment on attachment 397401 [details] [diff] [review]
patch rev 1

This is necessary for the lightweight themes UI, it also drops some interface methods so we need to land it before b1.
Attachment #397401 - Flags: approval1.9.2?
Whiteboard: [needs baking]
If this is necessary for lightweight themes UI, which blocks, then this blocks.
Flags: blocking1.9.2+
Attachment #397401 - Flags: approval1.9.2?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: