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

List:       kde-commits
Subject:    branches/KDE/3.4/kdegraphics/kpdf/ui
From:       Albert Astals Cid <tsdgeos () terra ! es>
Date:       2005-06-03 17:39:57
Message-ID: 1117820397.778493.19899.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 421674 by aacid:

 revert fix for 106546


 M  +16 -8     pageview.cpp  


--- branches/KDE/3.4/kdegraphics/kpdf/ui/pageview.cpp #421673:421674
@@ -577,14 +577,18 @@
         {
             // part doesn't get this key event because of the keyboard grab
             d->findTimeoutTimer->stop(); // restore normal operation during possible \
                messagebox is displayed
-	    //commenting as seems to cause bug 106546 and does not seem to give anything
-            //releaseKeyboard();
+            // it is needed to grab the keyboard becase people may have Space \
assigned to a  +            // accel and without grabbing the keyboard you can not \
vim-search for space +            // because it activates the accel
+            releaseKeyboard();
             if ( d->document->continueSearch( PAGEVIEW_SEARCH_ID ) )
                 d->messageWindow->display( i18n("Text found: \
\"%1\".").arg(d->typeAheadString.lower()),  PageViewMessage::Find, 3000 );
             d->findTimeoutTimer->start( 3000, true );
-	    //commenting as seems to cause bug 106546 and does not seem to give anything
-            // grabKeyboard();
+            // it is needed to grab the keyboard becase people may have Space \
assigned to a  +            // accel and without grabbing the keyboard you can not \
vim-search for space +            // because it activates the accel
+            grabKeyboard();
         }
         // esc and return: end search
         else if( e->key() == Key_Escape || e->key() == Key_Return )
@@ -623,8 +627,10 @@
             connect( d->findTimeoutTimer, SIGNAL( timeout() ), this, SLOT( \
findAheadStop() ) );  }
         d->findTimeoutTimer->start( 3000, true );
-        //commenting as seems to cause bug 106546 and does not seem to give anything
-        // grabKeyboard();
+        // it is needed to grab the keyboard becase people may have Space assigned \
to a  +        // accel and without grabbing the keyboard you can not vim-search for \
space +        // because it activates the accel
+        grabKeyboard();
         return;
     }
 
@@ -1828,8 +1834,10 @@
     d->typeAheadActive = false;
     d->typeAheadString = "";
     d->messageWindow->display( i18n("Find stopped."), PageViewMessage::Find, 1000 );
-    //commenting as seems to cause bug 106546 and does not seem to give anything
-    //releaseKeyboard();
+    // it is needed to grab the keyboard becase people may have Space assigned to a 
+    // accel and without grabbing the keyboard you can not vim-search for space
+    // because it activates the accel
+    releaseKeyboard();
 }
 
 void PageView::slotZoom()


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