1805807 - Message thread breaks when selecting a folder with fewer messages than the previous

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Alessandro Castellani [:aleca]]

This is exclusive to the Ash repository.

STR

• Select a folder with a lot of messages (in my case it was Trash)
• Scroll to the bottom of the message list
• Select another folder that has fewer messages

The tree explodes with multiple: JavaScript error: chrome://messenger/content/tree-view-listbox.mjs, line 1037: TypeError: can't access property "previousElementSibling" of null

We should enforce a scrollToTop() on the TreeView when the folder selection is changed, or an optional boolean to scroll back to the top when the list is invalidated.

Comment 1

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1805807 - Fix error when changing TreeViewListbox's view. r=aleca

Comment 2

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1805807 - When switching folders, invalidate the thread tree before setting the view. r=aleca

Also: set up the columns before setting the view.

Depends on D167639

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/540de7697f73
Fix error when changing TreeViewListbox's view. r=aleca
https://hg.mozilla.org/comm-central/rev/e5715072b71a
When switching folders, invalidate the thread tree before setting the view. r=aleca