key navigation didn't work in toolbars, which had separators (e.g. qpdf or

drawpad) - now this is accounted for and they are skipped

1 files changed, 1 insertions, 1 deletions

diff --git a/library/qpemenubar.cpp b/library/qpemenubar.cpp
index f0171ba..c658d10 100644
--- a/library/qpemenubar.cpp
+++ b/library/qpemenubar.cpp
@@ -26,193 +26,193 @@
 #include <qtimer.h>
 
 
 class QMenuBarHack : public QMenuBar
 {
 public:
     int activeItem() const { return actItem; }
 
     void goodbye()
     {
         activateItemAt(-1);
         for ( unsigned int i = 0; i < count(); i++ ) {
             QMenuItem *mi = findItem( idAt(i) );
             if ( mi->popup() ) {
                 mi->popup()->hide();
             }
         }
     }
 };
 
 
 QPEMenuToolFocusManager *QPEMenuToolFocusManager::me = 0;
 
 QPEMenuToolFocusManager::QPEMenuToolFocusManager() : QObject()
 {
     qApp->installEventFilter( this );
 }
 
 void QPEMenuToolFocusManager::addWidget( QWidget *w )
 {
     list.append( GuardedWidget(w) );
 }
 
 void QPEMenuToolFocusManager::removeWidget( QWidget *w )
 {
     list.remove( GuardedWidget(w) );
 }
 
 void QPEMenuToolFocusManager::setActive( bool a )
 {
     if ( a ) {
         oldFocus = qApp->focusWidget();
         QValueList<GuardedWidget>::Iterator it;
         it = list.begin();
         while ( it != list.end() ) {
             QWidget *w = (*it);
             if ( w && w->isEnabled() && w->isVisible() &&
                  w->topLevelWidget() == qApp->activeWindow() ) {
                 setFocus( w );
                 return;
             }
             ++it;
         }
     } else {
         if ( inFocus ) {
             if ( inFocus->inherits( "QMenuBar" ) )
                 ((QMenuBarHack *)(QWidget *)inFocus)->goodbye();
             if ( inFocus->hasFocus() ) {
                 if ( oldFocus && oldFocus->isVisible() && oldFocus->isEnabled() ) {
                     oldFocus->setFocus();
                 } else {
                     inFocus->clearFocus();
                 }
             }
         }
         inFocus = 0;
         oldFocus = 0;
     }
 }
 
 bool QPEMenuToolFocusManager::isActive() const
 {
     return !inFocus.isNull();
 }
 
 void QPEMenuToolFocusManager::moveFocus( bool next )
 {
     if ( !isActive() )
         return;
 
     int n = list.count();
     QValueList<GuardedWidget>::Iterator it;
     it = list.find( inFocus );
     if ( it == list.end() )
         it = list.begin();
     while ( --n ) {
         if ( next ) {
             ++it;
             if ( it == list.end() )
                 it = list.begin();
         } else {
             if ( it == list.begin() )
                 it = list.end();
             --it;
         }
         QWidget *w = (*it);
-        if ( w && w->isEnabled() && w->isVisible() &&
+        if ( w && w->isEnabled() && w->isVisible() && !w->inherits("QToolBarSeparator") &&
              w->topLevelWidget() == qApp->activeWindow() ) {
             setFocus( w, next );
             return;
         }
     }
 }
 
 void QPEMenuToolFocusManager::initialize()
 {
     if ( !me )
         me = new QPEMenuToolFocusManager;
 }
 
 QPEMenuToolFocusManager *QPEMenuToolFocusManager::manager()
 {
     if ( !me )
         me = new QPEMenuToolFocusManager;
 
     return me;
 }
 
 void QPEMenuToolFocusManager::setFocus( QWidget *w, bool next )
 {
     inFocus = w;
 //    qDebug( "Set focus on %s", w->className() );
     if ( inFocus->inherits( "QMenuBar" ) ) {
         QMenuBar *mb = (QMenuBar *)(QWidget *)inFocus;
         if ( next )
             mb->activateItemAt( 0 );
         else
             mb->activateItemAt( mb->count()-1 );
     }
     inFocus->setFocus();
 }
 
 bool QPEMenuToolFocusManager::eventFilter( QObject *object, QEvent *event )
 {
     if ( event->type() == QEvent::KeyPress ) {
         QKeyEvent *ke = (QKeyEvent *)event;
         if ( isActive() ) {
             if ( object->inherits( "QButton" ) ) {
                 switch ( ke->key() ) {
                     case Key_Left:
                         moveFocus( FALSE );
                         return TRUE;
 
                     case Key_Right:
                         moveFocus( TRUE );
                         return TRUE;
 
                     case Key_Up:
                     case Key_Down:
                         return TRUE;
                 }
             } else if ( object->inherits( "QPopupMenu" ) ) {
                 // Deactivate when a menu item is selected
                 if ( ke->key() == Key_Enter || ke->key() == Key_Return ||
                      ke->key() == Key_Escape ) {
                     QTimer::singleShot( 0, this, SLOT(deactivate()) );
                 }
             } else if ( object->inherits( "QMenuBar" ) ) {
                 int dx = 0;
                 switch ( ke->key() ) {
                     case Key_Left:
                         dx = -1;
                         break;
 
                     case Key_Right:
                         dx = 1;
                         break;
                 }
 
                 QMenuBarHack *mb = (QMenuBarHack *)object;
                 if ( dx && mb->activeItem() >= 0 ) {
                     int i = mb->activeItem();
                     int c = mb->count();
                     int n = c;
                     while ( n-- ) {
                         i = i + dx;
                         if ( i == c ) {
                             mb->goodbye();
                             moveFocus( TRUE );
                             return TRUE;
                         } else if ( i < 0 ) {
                             mb->goodbye();
                             moveFocus( FALSE );
                             return TRUE;
                         }
                         QMenuItem *mi = mb->findItem( mb->idAt(i) );
                         if ( mi->isEnabled() && !mi->isSeparator() ) {
                             break;
                         }
                     }
                 }
             }
         }