69179 - Column picker should be xbl-ified

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Blake Ross]

Clients currently have to build their own column picker and include 
treePopups.js.  All they should have to do is say <columnpicker/>.

Comment 1

[Blake Ross]

Reassigning...

Comment 2

[David Hyatt]

This is not apps.  This is core tree widget functionality.

Comment 3

[David Hyatt]

It would be a good idea to hold off working on this in the old <tree>, since the
plan is to move any trees that would require such a widget over to the new
<outliner> instead.  Therefore one that worked with the current tree is not
something that we will need to do.

This bug should stay open to track a widget that can be used for <outliner>.

Comment 4

[Blake Ross]

Hmmm...I wish you told me this earlier, since I have it working :-)

Comment 5

[Blake Ross]

Hyatt: is it worth checking in the tree-based columnpicker until an rdf view 
for outliner is implemented?

And should I create an xblified version for outliner (based on what you've 
checked in so far), or would you rather do that?

Comment 6

[David Hyatt]

I've checked in and implemented a columnpicker for <outliner> (based off the
code you sent me, blake) on the branch.

Comment 7

[Blake Ross]

Excellent -- giving this one to you.

Comment 8

[Blake Ross]

By the way, the 'collapsed' checking (which wasn't in the patch you checked in) 
was actually to allow hiding columns without having them show up in the picker 
list ('hidden' can't be used for that in this case).  I know we currently do 
this to hide columns in Bookmarks and other places, so you may want to consider 
adding it back.

Comment 9

[Peter Trudelle]

Setting to 0.9.1

Comment 10

[David Hyatt]

This is fixed on the branch.  Marking resolved.