Closed Bug 1814321 Opened 1 year ago Closed 11 months ago

Stop setting the columns multiple times on page load

Categories

(Thunderbird :: Folder and Message Lists, enhancement)

Thunderbird 111
enhancement

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: aleca, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [Supernova3p])

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.

Whiteboard: [Supernova]

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.

Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → INVALID
Whiteboard: [Supernova] → [Supernova3p]
You need to log in before you can comment on or make changes to this bug.