-
Notifications
You must be signed in to change notification settings - Fork 260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Grouping of message threads (threading/conversations) [$320] #17
Comments
Firstly thanks for your work. Any thread/conversation support is essential to me. I prefer see all related mails on the same page. |
@sim6 cool you enjoy the app! :) Since you are a developer, would you be interested in contributing a bit? We are an open project and anyone is welcome! |
I would love if this thread/conversation view includes all mail regardless of the folder in which it is saved. Outlook (yes I know) does this and I find it really helps me when I get replies in my INBOX. The thread shows all the mails of the conversation regardless of the folder I saved the previous mails to. |
@adsworth yup, agree. Want to contribute to the bounty on this issue? ;) https://www.bountysource.com/issues/37463009-grouping-of-message-threads-threading-conversations cc @sim6 as well. |
Just in case anybody wants to work on this: please first write down how you'd implement this. There are many aspects to consider for this feature to work properly. Thanks. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Since this is a complex feature, I would like to split it a bit so we can cope with the complexity. Update: Converted into a project https://github.com/nextcloud/mail/projects/11
0) Change front-end so an opened message is identified by its database ID (and not accountId, folderId, UID)
1) Change database schema so `mail_messages` also have a `parent_message_id` and a `parent_id` and try to populate both on insert. Also try to find any dangling `parent_message_id`s and update those rows.
2) Change front-end so for each opened message you also see the previous message. **But do not de-duplicate inside the message list** because this is complex and deserves a step of its own.
3) Only show each thread once per message list.
|
Note: I would, for the first implementation, ignore any threading algorithm but the one based on |
🔖 http://kb.mozillazine.org/Stop_threading_by_subject
|
This comment has been minimized.
This comment has been minimized.
Wait it's mostly just the backe-end changes. I did no intend to close this ticket yet :P |
We have the full conversation-style view now. The last piece of the puzzle is the collapsing of message of the same thread in the message list and that will be worked on as #3652. Hence I'm closing this as (mostly) done :) |
Similar as in Gmail, Apple Mail for iOS, or Facebook private messages or what have you. Read messages in that thread should be compressed & expandable (also like in Gmail and iOS Mail), and inside the thread the sorting should go recent messages on the bottom (because of reading direction, and that’s how comment threads work).
There should be no indentation or crap like that, just timestamp-based sorting. In the future every individual message in the thread could show a »Reply« button on hover to reply to that specific message. But for now I’d just keep it simple like a one-stream comment thread.
For reference:
Max Ogden's JS implementation of said algorithmThere is a $220 open bounty on this issue. Add to the bounty at Bountysource.
There is a $220 open bounty on this issue. Add to the bounty at Bountysource.
The text was updated successfully, but these errors were encountered: