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

List:       kde-commits
Subject:    branches/kdepim/enterprise/kdepim/libkdepim
From:       Allen Winter <winter () kde ! org>
Date:       2010-09-22 14:53:20
Message-ID: 20100922145320.9DCADAC888 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1178259 by winterz:

revertlast because it breaks up/down arrowing in some cases.
kolab/issue4562
MERGE: e35tag, trunk (check 4.5 too)


 M  +1 -8      addresseelineedit.cpp  


--- branches/kdepim/enterprise/kdepim/libkdepim/addresseelineedit.cpp #1178258:1178259
@@ -1088,9 +1088,6 @@
     }
 
     if ( ke->key() == Key_Up ) {
-      if ( e->type() != QEvent::KeyRelease ) {
-        return true;
-      }
       //kdDebug() << "EVENTFILTER: Key_Up currentIndex=" << currentIndex << endl;
       // figure out if the item we would be moving to is one we want
       // to ignore. If so, go one further
@@ -1113,12 +1110,9 @@
           completionBox()->setCurrentItem( itemAbove );
           completionBox()->setSelected( currentIndex, true );
         }
-      }
       return true;
-    } else if ( ke->key() == Key_Down  ) {
-      if ( e->type() != QEvent::KeyRelease ) {
-        return true;
       }
+    } else if ( ke->key() == Key_Down  ) {
       // same strategy for downwards
       //kdDebug() << "EVENTFILTER: Key_Down. currentIndex=" << currentIndex << endl;
       QListBoxItem *itemBelow = completionBox()->item( currentIndex );
@@ -1145,7 +1139,6 @@
       if ( item && itemIsHeader(item) ) {
         completionBox()->setSelected( currentIndex, true );
       }
-      return true;
     } else if ( e->type() == QEvent::KeyRelease &&
                 ( ke->key() == Key_Tab || ke->key() == Key_Backtab ) ) {
       //kdDebug() << "EVENTFILTER: Key_Tab. currentIndex=" << currentIndex << endl;
[prev in list] [next in list] [prev in thread] [next in thread]