Previously INBOX.Archive and Archive were indistinguishable.
Details
- Reviewers
machniak - Group Reviewers
Roundcube - Commits
- rRPKd4c00a9555d0: Do not ignore INBOX for the displayed folder hierarchy
Diff Detail
- Repository
- rRPK roundcubemail-plugins-kolab
- Branch
- master
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 32647 Build 12607: arc lint + arc unit
Event Timeline
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.
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.
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.
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.