[kmail2] [Bug 227516] changing sorting changes columns

[Ingo Klöcker]

https://bugs.kde.org/show_bug.cgi?id=227516

Ingo Klöcker <kloecker at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
   Version Fixed In|                            |4.11.4
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/kdep
                   |                            |im/5949d3215d1ab06b8733476c
                   |                            |22f3094cecedad8f

--- Comment #6 from Ingo Klöcker <kloecker at kde.org> ---
Git commit 5949d3215d1ab06b8733476c22f3094cecedad8f by Ingo Klöcker.
Committed on 15/11/2013 at 22:42.
Pushed by kloecker into branch 'KDE/4.11'.

Use dedicated slots for handling exactly the actions they should handle

Previously, all actions in the header context menu were handled by the
same method slotHeaderContextMenuTriggered(), even actions in submenus
which was clearly not intended. For example, selecting sorting
By Date/Time of Most Recent in Subtree in the Sorting submenu did also
enable the Sender column (because the action for showing/hiding the
Sender column had the same index set as user data as the action for
sorting By Date/Time of Most Recent in Subtree). Usage of dedicated
slots attached to the actions' signals instead of a single slot
attached to the menu's triggered() signal (which is also emitted for
actions in submenus) fixes this.
FIXED-IN: 4.11.4
REVIEW: 113901

M  +44   -45   messagelist/core/view.cpp
M  +17   -3    messagelist/core/view.h

http://commits.kde.org/kdepim/5949d3215d1ab06b8733476c22f3094cecedad8f

-- 
You are receiving this mail because:
You are the assignee for the bug.