Previously INBOX.Archive and Archive were indistinguishable.
Details
Details
- Reviewers
machniak - Group Reviewers
Roundcube - Commits
- rRPKd4c00a9555d0: Do not ignore INBOX for the displayed folder hierarchy
Diff Detail
Diff Detail
- Repository
- rRPK roundcubemail-plugins-kolab
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
FWIW, this code was just like this from the beginning, so I don't know if there ever was a valid usecase for it.
Can't think of one though.
Comment Actions
I'm not sure I like this. While fixing the issue you also changed how the list will look like. I.e. flat list instead of a tree-like list. If we do this anyway we should do the same in kolab_delegation plugin.
Comment Actions
Instead of not showing a tree at all, show it for INBOX as well.
This solves the original concern, while keeping the hierarchy intact.
This is also relevant because the previous patch did not work with localized special folder names (such as "Posteingang" for INBOX).
I applied the same change for delegation as well.
Comment Actions
FWIW, I assume the original code meant to make the case prettier where all folders are below inbox. I think for the affected UI's it shouldn't be a big deal, and avoids the confusing situation we can have right now.