122847 - nsXULSortService needs to work with listbox

Status: VERIFIED FIXED

(Core Graveyard :: RDF, defect)

Description

[Joe Hewitt (gone)]

nsXULSortService has hard-coded <tree> logic which needs to be massaged to work
with listbox.

Comment 1

[Joe Hewitt (gone)]

Marking nsbeta1 because I need this to convert Search to use listbox (and
probably others)

Comment 2

[Peter Trudelle]

nsbeta1+ per Nav triage team

Comment 3

[Joe Hewitt (gone)]

I've got this to work finally.  The sort service is now generic enough that it
can sort just about any rdf-template generated content.  I have tested it with
listbox, menupopup, and outliner.  If you want to persist sorts across sessions,
you can just set sortActive, sortDirection, and sortResource on the element that
has databases="" on it.

Comment 4

[Joe Hewitt (gone)]

Attachment: patch

Comment 5

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 73486 [details] [diff] [review]
patch

[s]r=[e-mail]

Comment 6

[Jaime Rodriguez, Jr.]

ADT2 per ADT/Nav triage.

Comment 7

[Blake Ross]

In the absence of waterson, I think tingley or rjc would be the next best to
review this.

Comment 8

[Robert John Churchill]

Comment on attachment 73486 [details] [diff] [review]
patch

r=rjc

Comment 9

[Joe Hewitt (gone)]

fixed

Comment 10

[Tom Everingham]

verified:  04/09/02 tr builds, win nt4, linux rh6, mac osx