[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    [Bug 87857] Selecting an email and pressing delete is unnessarily
From:       Martin Koller <m.koller () surfeu ! at>
Date:       2007-01-05 18:17:54
Message-ID: 20070105181754.30983.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=87857         
m.koller surfeu at changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From m.koller surfeu at  2007-01-05 19:17 -------
SVN commit 620346 by mkoller:

BUG: 87857

always clear selection in finalizeMove, especially important when there
will be no item for selection afterwards as this avoids O(n^2) in
prepareMove()
Patch from Ismail Onur Filiz


 M  +1 -1      kmheaders.cpp  


--- branches/KDE/3.5/kdepim/kmail/kmheaders.cpp #620345:620346
 @ -1476,9 +1476,9  @
 void KMHeaders::finalizeMove( HeaderItem *item, int contentX, int contentY )
 {
   emit selected( 0 );
+  clearSelection();
 
   if ( item ) {
-    clearSelection();
     setCurrentItem( item );
     setSelected( item, true );
     setSelectionAnchor( currentItem() );
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel
[prev in list] [next in list] [prev in thread] [next in thread]