Conversation list model #8351
Merged
Conversation list model #8351
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
paw-hub
force-pushed
the
8223-conversation-list-model
branch
from
20ba81d to
72a61ad
Compare
BijinDev
force-pushed
the
8223-conversation-list-model
branch
from
1ded82d to
0367ecb
Compare
BijinDev
force-pushed
the
8223-conversation-list-model
branch
from
4d5eaf9 to
a3c9c89
Compare
40 tasks
BijinDev
force-pushed
the
8223-conversation-list-model
branch
from
a3c9c89 to
2e54f1d
Compare
charlag
reviewed
Jan 30, 2025
charlag
reviewed
Jan 31, 2025
charlag
reviewed
Feb 3, 2025
|
Another thing I noticed: when you apply or remove a label this change is not shown in the mail list immediately. You have to reload to see it. |
paw-hub
force-pushed
the
8223-conversation-list-model
branch
2 times, most recently
from
0175167 to
fbb87ef
Compare
This should now be fixed. Thanks for reporting this! |
paw-hub
force-pushed
the
8223-conversation-list-model
branch
4 times, most recently
from
1c7ea72 to
cd43326
Compare
paw-hub
force-pushed
the
8223-conversation-list-model
branch
from
cd43326 to
f71566a
Compare
charlag
reviewed
Feb 6, 2025
|
To not forget: switching between folders is slow |
|
It seems to wait until redraw (if you move your cursor a bit after selecting the new folder, it will switch). I wonder how this got borked? |
charlag
force-pushed
the
8223-conversation-list-model
branch
from
2ee60b2 to
dfb4d04
Compare
- This list model groups all emails by conversation in the mail list - Also adds setting for grouping by conversation (i.e. enables this) - The setting is overridden for SENT and DRAFT folder types (it uses the other per-mail list model instead) - Adds MailSetListModel which provides a common interface for the MailViewModel so that additional list models can be created; it is used for MailListModel and the new ConversationListModel, but it can be extended to even more list models in the future if it is ever desired. Note: This does not include actions. Any action you do (delete, archive, etc.) will be applied to the selected email. The purpose of this commit is to just implement the list model. See #5051 for more information. Closes #8223 Co-authored-by: bir <[email protected]> Co-authored-by: ivk <[email protected]>
charlag
force-pushed
the
8223-conversation-list-model
branch
from
980e994 to
c0d7723
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.