1979880 - Reset manual folder sorting does not work for sub folders

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Alexander Bergmann]

First let me thank you for implementing a much appreciated option to reset the manual folder sorting!

I have tested this option in Thunderbird Beta 142.0b1 (64-bit) under Windows 11 (64-bit). And I think I have discovered a corner-case, which does not yet work as expected. (I wasn't sure, whether to report this as a new bug or reopen this bug report.)

I have created the following folder structure - three top folders A, B, C and three sub folders 1, 2, 3 of folder A:

A
A->1
A->2
A->3
B
C

What does work: Reordering the top folders and then resetting the manual folder sorting:

B
C
A
A->1
A->2
A->3

What does not work: Reordering the sub folders and then resetting the manual folder sorting:

A
A->2
A->3
A->1
B
C

(Copy of Bug 1972710 Comment 4)

Comment 1

[Alexander Bergmann]

(In reply to Martin Giger [:freaktechnik] from Bug 1972710 comment #5)

To summarize, it's not recursing into subfolders?

If the code is indeed detecting a change from the default folder sorting and therefore resetting the folder sorting, then the sub folders are reset as well. However, the code does not detect a change, if the only change from the default folder sorting has been made to sub folders.

Comment 2

[Martin Giger [:freaktechnik]]

Thank you! Marking this as a dependency, since it can't occur without that enhancement.

Comment 3

[itagagaki]

I think it would be better to have the option to choose of whether or not to proceed recursively into subfolders.

Comment 4

[Alexander Bergmann]

This bug with the STR in the description is now affecting Thunderbird 142.0 Release, Thunderbird 143.0b1 Beta and Thunderbird 144.0a1 Daily.

Comment 7

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1979880 - Fix 'Reset Folder Order' for subfolders. r=tobyp,mkmelin

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/ecc2ac50c645
Fix 'Reset Folder Order' for subfolders. r=tobyp,mkmelin