1814321 - Stop setting the columns multiple times on page load

Status: RESOLVED INVALID

(Thunderbird :: Folder and Message Lists, enhancement)

Description

[Alessandro Castellani [:aleca]]

The way we currently set the TreeViewTable is not optimal.
We first generate a dummy layout based on a default set of columns, via the setColumns() method on load.
This is because that custom element assumes columns are already present when populating the data.

Then, we reset the columns when a folder is selected.
Then, if the folder has a customized layout different from the default column states, we reset it again.

Nasty!

Let's update the TreeViewTable and the code initialization of the about3Pane in order to be able to start with an empty layout, and only set the columns one time, after we're sure the correct columns array was fetched from the selected folder.

Comment 1

[Geoff Lankow (:darktrojan)]

We have changed how it worked since filing this bug, so it's no longer relevant. I think how we set columns now is as good as it's going to get, and even if not, it's good enough.