Hi,
I'm just subscribed, so first thanks to all devs for all efforts on making RC really great.
I noticed that threaded view support is planned for 0.4-beta1, so I attach a reworked patch for a threaded mailbox view mode. It is not fully perfect, but is a good (re)starting point for that anyway.
Changes comparing to roundcubemail-threading-20090706.patch from http://www.atomice.com/blog/?page_id=34:
- Ported to 0.3-SVN-r2863
- Implemented auto expand of threads with unread messages only.
- Added GUI configuration option for autoexpand control (on, off, unread).
- Autoexpansion works in both static and JS message lists.
- Automatically change between 'unread children' and message icons in a
list (in parent message rows) when child message is marked read/unread. 6. Count messages, not threads in a message view pane. 7. Fixed navigation in a message view pane when threading is enabled. 8. Fixed setting folders with non-ASCII named to threaded mode. 9. Fixed incorrectly displayed rows (children of collapsed row) after multi-level expand/collapse. Simplified expand logic. 10. Fixed expand indicator is not changing when expanding/collapsing with keyboard. 11. Fixed plus key on a numeric keypad is not working (at least on linux Fedora 11 + Firefox 3.5.2). 12. Added support for whole thread expand/collapse with clicking/pressing hotkeys while holding the Control key. 13. Save expand state in env everywhere (after keypresses too). 14. Added russian translation. 15. Fixed indentation and braces style in JS to comply with RC coding style.
Known issues:
- Messages from a threaded folder are sorted not in a thread-aware
order but rather in a natural order when browsing with GUI controls at the bottom of a message view pane. This is probably a minor issue because it is more convenient anyway to browse through threaded discussions with a preview pane enabled. Maybe we need to explicitly enable a preview pane for a threaded folders. 2. Last selected message is neither focused nor centered in a list when returning to a list from a message view pane. I think it is minor issue too.
This patch (and original one too) is sometimes too intrusive, so it needs to be checked at least twice just not to break things.
Thanks for your time, comments are welcome.
Best, Vladislav
--- 8< --- detachments --- 8< --- The following attachments have been detached and are available for viewing. http://detached.gigo.com/rc/X6/LiAx90cn/roundcubemail-SVN-28.patch http://detached.gigo.com/rc/X6/LiAx90cn/unread_children.png Only click these links if you trust the sender, as well as this message. --- 8< --- detachments --- 8< ---
List info: http://lists.roundcube.net/dev/