Bug 309041 - Aggregation setting is not folder dependant, only affects first tab
Summary: Aggregation setting is not folder dependant, only affects first tab
Status: REPORTED
Alias: None
Product: kmail2
Classification: Applications
Component: message list (show other bugs)
Version: 4.8.5
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-26 12:19 UTC by Dexter Magnific
Modified: 2012-10-27 09:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dexter Magnific 2012-10-26 12:19:29 UTC
When multiple tabs are opened for different folders, setting the aggregation (Menu View -> Message List -> Aggregation) is always applied to the first tab, not the currently focused tab.

In addition, the aggregation setting is not folder dependant. When I set aggregation A for folder Inbox and switch to folder Trash for instance, Trash has also aggregation A instead of its last aggregation when left.

Reproducible: Always
Comment 1 Laurent Montel 2012-10-26 22:32:22 UTC
"When multiple tabs are opened for different folders, setting the aggregation (Menu View -> Message List -> Aggregation) is always applied to the first tab, not the currently focused tab."

Ok will look at. It's right I think that it's not implemented.
On my todo.

"In addition, the aggregation setting is not folder dependant. When I set aggregation A for folder Inbox and switch to folder Trash for instance, Trash has also aggregation A instead of its last aggregation when left."
it's right I confirm it.
I will fix it soon.
Comment 2 Laurent Montel 2012-10-27 09:01:31 UTC
Git commit d4385a7a0f75fbd7c3ae9d02cc65f83c58f3a559 by Montel Laurent.
Committed on 27/10/2012 at 11:01.
Pushed by mlaurent into branch 'master'.

Start to fix Bug 309041 - Aggregation setting is not folder dependant,

only affects first tab

Now we apply setting to current tab.

M  +17   -22   messagelist/core/view.cpp
M  +9    -7    messagelist/core/view.h
M  +12   -0    messagelist/core/widgetbase.cpp
M  +6    -0    messagelist/core/widgetbase.h
M  +24   -0    messagelist/messagelistutil.cpp
M  +2    -0    messagelist/messagelistutil.h
M  +30   -2    messagelist/pane.cpp
M  +7    -0    messagelist/pane.h

http://commits.kde.org/kdepim/d4385a7a0f75fbd7c3ae9d02cc65f83c58f3a559
Comment 3 Laurent Montel 2012-10-27 09:02:10 UTC
I put it in 4.10 not 4.9.x because it's a big changes.

Regards