From 7aaccdf805a0c05deb8b009d64707bbd8cdea160 Mon Sep 17 00:00:00 2001 From: Pino Toscano Date: Sun, 1 Mar 2009 14:38:04 +0000 Subject: [PATCH] backport: - turn a sequence of if() into a single switch() - add Up (previous page) and Down (next page) keys for presentation mode svn path=/branches/KDE/4.2/kdegraphics/okular/; revision=933725 --- ui/presentationwidget.cpp | 38 +++++++++++++++++++++++++------------- 1 file changed, 25 insertions(+), 13 deletions(-) diff --git a/ui/presentationwidget.cpp b/ui/presentationwidget.cpp index 72e51f678..68bb39da5 100644 --- a/ui/presentationwidget.cpp +++ b/ui/presentationwidget.cpp @@ -398,20 +398,32 @@ void PresentationWidget::keyPressEvent( QKeyEvent * e ) { if (m_width == -1) return; - if ( e->key() == Qt::Key_Left || e->key() == Qt::Key_Backspace || e->key() == Qt::Key_PageUp ) - slotPrevPage(); - else if ( e->key() == Qt::Key_Right || e->key() == Qt::Key_Space || e->key() == Qt::Key_PageDown ) - slotNextPage(); - else if ( e->key() == Qt::Key_Home ) - slotFirstPage(); - else if ( e->key() == Qt::Key_End ) - slotLastPage(); - else if ( e->key() == Qt::Key_Escape ) + switch ( e->key() ) { - if ( !m_topBar->isHidden() ) - m_topBar->hide(); - else - close(); + case Qt::Key_Left: + case Qt::Key_Backspace: + case Qt::Key_PageUp: + case Qt::Key_Up: + slotPrevPage(); + break; + case Qt::Key_Right: + case Qt::Key_Space: + case Qt::Key_PageDown: + case Qt::Key_Down: + slotNextPage(); + break; + case Qt::Key_Home: + slotFirstPage(); + break; + case Qt::Key_End: + slotLastPage(); + break; + case Qt::Key_Escape: + if ( !m_topBar->isHidden() ) + m_topBar->hide(); + else + close(); + break; } }