Closed Bug 1890731 Opened 3 months ago Closed 1 month ago

Create a TreeView wrapper that automatically handles the column and sort events

Categories

(Thunderbird :: General, task)

Tracking

(Not tracked)

RESOLVED FIXED
128 Branch

People

(Reporter: darktrojan, Assigned: darktrojan)

References

(Blocks 1 open bug)

Details

Attachments

(7 files)

TreeView has the UI event listeners for many types of sorting and column arrangement events, but mostly it just forwards the events to be implemented elsewhere. Most instances of the element will handle these events in exactly the same way, so I'm creating a wrapper class which does all of that handling in a tidy, self-contained package. I hope to eventually move all of this logic into the TreeView class but for now I don't want to risk unintended changes in the existing uses, about:3pane and about:addressbook.

As a part of this I want to overhaul PROTO_TREE_VIEW (from jsTreeView.js) to make it capable of sorting the view without additional code, and to remove a bunch of nsITreeView cruft it will no longer need. The sorting can be done now, the cruft will have to wait until we stop using the XUL tree with JS views.

Attachment #9395969 - Attachment description: WIP: Bug 1890731 - Create a TreeView wrapper that automatically handles the column and sort events → Bug 1890731 - Create a TreeView wrapper that automatically handles the column and sort events
Attachment #9398299 - Attachment description: WIP: Bug 1890731 - Begin replacing nsITreeView with TreeDataAdapter → Bug 1890731 - Begin replacing nsITreeView with TreeDataAdapter
Attachment #9398300 - Attachment description: WIP: Bug 1890731 - Add sorting to TreeDataAdapter → Bug 1890731 - Add sorting to TreeDataAdapter
Attachment #9398301 - Attachment description: WIP: Bug 1890731 - Convert ABView into a TreeDataAdapter → Bug 1890731 - Convert ABView into a TreeDataAdapter
Attachment #9398302 - Attachment description: WIP: Bug 1890731 - Replace XUL tree in contacts sidebar and search window with TreeViews → Bug 1890731 - Replace XUL tree in contacts sidebar and search window with TreeViews
Attachment #9395969 - Attachment description: Bug 1890731 - Create a TreeView wrapper that automatically handles the column and sort events → Bug 1890731 - Create a TreeView wrapper that automatically handles the column and sort events. r=aleca,freaktechnik
Attachment #9398299 - Attachment description: Bug 1890731 - Begin replacing nsITreeView with TreeDataAdapter → Bug 1890731 - Begin replacing nsITreeView with TreeDataAdapter. r=aleca,freaktechnik
Attachment #9398300 - Attachment description: Bug 1890731 - Add sorting to TreeDataAdapter → Bug 1890731 - Add sorting to TreeDataAdapter. r=aleca,freaktechnik
Attachment #9398301 - Attachment description: Bug 1890731 - Convert ABView into a TreeDataAdapter → Bug 1890731 - Convert ABView into a TreeDataAdapter. r=aleca,freaktechnik
Attachment #9398302 - Attachment description: Bug 1890731 - Replace XUL tree in contacts sidebar and search window with TreeViews → Bug 1890731 - Replace XUL tree in contacts sidebar and search window with TreeViews. r=aleca,freaktechnik
Target Milestone: --- → 128 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/be1edc170ed9
Create a TreeView wrapper that automatically handles the column and sort events. r=aleca
https://hg.mozilla.org/comm-central/rev/d8aa15c2c9dd
Begin replacing nsITreeView with TreeDataAdapter. r=freaktechnik,aleca
https://hg.mozilla.org/comm-central/rev/c58fd8188685
Add sorting to TreeDataAdapter. r=freaktechnik,aleca
https://hg.mozilla.org/comm-central/rev/b199369ce2dd
Convert ABView into a TreeDataAdapter. r=freaktechnik,aleca
https://hg.mozilla.org/comm-central/rev/d34f74e86210
Replace XUL tree in contacts sidebar and search window with TreeViews. r=aleca
https://hg.mozilla.org/comm-central/rev/c9ba6b258597
Remove OS-specific styles from contacts sidebar and tidy up a little. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 1 month ago1 month ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: