1819247 - (sn-treeperf) [meta] Improve the performance of the new TreeView

Status: RESOLVED WORKSFORME

(Thunderbird :: Folder and Message Lists, task, P3)

Description

[Alessandro Castellani [:aleca]]

Meta bug to track the work to improve the performance of the new TreeView widget

Comment 1

[Sean Burke [:leftmostcat]]

Attachment: Tree performance demo following idle tick patch

This serves as a demo of tree performance, current as of the closing of bug 1830979. It's independent of Thunderbird and can be run by unzipping the file and navigating to the resulting tree-demo.xhtml. It's incomplete, but adequately demonstrates the performance issues we're experiencing.

Comment 2

[Markus Stange [:mstange]]

(In reply to Sean Burke [:leftmostcat] from comment #1)

Created attachment 9335053 [details]
Tree performance demo following idle tick patch

This serves as a demo of tree performance, current as of the closing of bug 1830979. It's independent of Thunderbird and can be run by unzipping the file and navigating to the resulting tree-demo.xhtml. It's incomplete, but adequately demonstrates the performance issues we're experiencing.

When profiling scrolling in this testcase, the time is spent under the "Styles" marker but it's really doing Reflow / Layout. It's happening under the Styles marker because of a container query.
So you want to improve the layout time, by making the rows independent of each other so that only the newly-added rows have to be reflowed. This probably means that you can't use a <table>.

Comment 3

[Alessandro Castellani [:aleca]]

Closing this for now as we're in a pretty good state with the performance of the tree view.
In the next cycle we will explore the possibility of using a different widget and improving things when needed.

Comment 4

[Wayne Mery (:wsmwk)]

=> WFM because it doesn't have a patch