Closed Bug 69179 Opened 24 years ago Closed 24 years ago

Column picker should be xbl-ified

Categories

(Core :: XUL, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla0.9.1

People

(Reporter: bugzilla, Assigned: hyatt)

Details

Clients currently have to build their own column picker and include 
treePopups.js.  All they should have to do is say <columnpicker/>.
Reassigning...
Assignee: vishy → blakeross
Component: XP Apps → XP Apps: GUI Features
This is not apps.  This is core tree widget functionality.
Component: XP Apps: GUI Features → XP Toolkit/Widgets: Trees
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>.
Hmmm...I wish you told me this earlier, since I have it working :-)
Status: NEW → ASSIGNED
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?
I've checked in and implemented a columnpicker for <outliner> (based off the
code you sent me, blake) on the branch.
Excellent -- giving this one to you.
Assignee: blakeross → hyatt
Status: ASSIGNED → NEW
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.
QA Contact: sairuh → jrgm
Setting to 0.9.1
Target Milestone: --- → mozilla0.9.1
This is fixed on the branch.  Marking resolved.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.