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

List:       kde-commits
Subject:    [kphotoalbum] Viewer: Re-enabled <space> and <backspace> as navigation alternative in the viewer.
From:       Tobias Leupold <tobias.leupold () web ! de>
Date:       2016-08-14 17:31:27
Message-ID: E1bYzFv-0000Q7-HZ () code ! kde ! org
[Download RAW message or body]

Git commit b299a4be2378585671fbd01efa0f20e3251971ca by Tobias Leupold.
Committed on 14/08/2016 at 17:30.
Pushed by tleupold into branch 'master'.

Re-enabled <space> and <backspace> as navigation alternative in the viewer.

M  +2    -11   Viewer/ViewerWidget.cpp

http://commits.kde.org/kphotoalbum/b299a4be2378585671fbd01efa0f20e3251971ca

diff --git a/Viewer/ViewerWidget.cpp b/Viewer/ViewerWidget.cpp
index 78d7f29..03de3b9 100644
--- a/Viewer/ViewerWidget.cpp
+++ b/Viewer/ViewerWidget.cpp
@@ -301,12 +301,7 @@ void Viewer::ViewerWidget::createSkipMenu()
 
     action = m_actions->addAction( QString::fromLatin1("viewer-next"), this, SLOT(showNext()) );
     action->setText( i18nc("@action:inmenu","Show Next") );
-    action->setShortcut( Qt::Key_PageDown );
-    // bah, they don't use references
-    QKeySequence viewerNextShortcut = action->shortcut();
-    // FIXME KF5-port: QKeySequence does not allow an alternate key binding:
-//    viewerNextShortcut.setAlternate( Qt::Key_Space );
-    action->setShortcut( viewerNextShortcut );
+    action->setShortcuts(QList<QKeySequence>() << Qt::Key_PageDown << Qt::Key_Space);
     popup->addAction( action );
     m_forwardActions.append(action);
 
@@ -330,11 +325,7 @@ void Viewer::ViewerWidget::createSkipMenu()
 
     action = m_actions->addAction( QString::fromLatin1("viewer-prev"), this, SLOT(showPrev()) );
     action->setText( i18nc("@action:inmenu","Show Previous") );
-    action->setShortcut( Qt::Key_PageUp );
-    QKeySequence viewerPrevShortcut = action->shortcut();
-    // FIXME KF5-port: QKeySequence does not allow an alternate key binding:
-//    viewerPrevShortcut.setAlternate(Qt::Key_Backspace);
-    action->setShortcut(viewerPrevShortcut);
+    action->setShortcuts(QList<QKeySequence>() << Qt::Key_PageUp << Qt::Key_Backspace);
     popup->addAction( action );
     m_backwardActions.append(action);
 
[prev in list] [next in list] [prev in thread] [next in thread]