Bug 309041

Summary: Aggregation setting is not folder dependant, only affects first tab
Product: [Applications] kmail2 Reporter: Dexter Magnific <dextermagnific>
Component: message listAssignee: kdepim bugs <kdepim-bugs>
Status: REPORTED ---    
Severity: wishlist CC: montel
Priority: NOR    
Version: 4.8.5   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:

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