File 2002-fix-shortcuts.patch of Package spectacle

Overview Repositories Revisions Requests Users Attributes Meta

File 2002-fix-shortcuts.patch of Package spectacle (Revision f754057aabd1106030ff06450cde0b8d)

Currently displaying revision f754057aabd1106030ff06450cde0b8d , Show latest

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 5aa0bc32b0d13a525864a6ba521d40317c06958b..592096d7245206ff1489bc37e648ea28ec8862a5 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -39,6 +39,7 @@ add_executable(spectacle
     Gui/SettingsDialog/SettingsDialog.cpp
     Gui/SettingsDialog/GeneralOptionsPage.cpp
     Gui/SettingsDialog/ShortcutsOptionsPage.cpp
+    Gui/Action.cpp
     Platforms/PlatformLoader.cpp
     Platforms/Platform.cpp
     Platforms/PlatformNull.cpp
diff --git a/src/Gui/Action.cpp b/src/Gui/Action.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..c1454d986166da54fa34c166fd7040cf0ec4720b
--- /dev/null
+++ b/src/Gui/Action.cpp
@@ -0,0 +1,24 @@
+#include "Action.h"
+
+#include <QKeyEvent>
+
+Action::~Action()
+{
+    parent()->removeEventFilter(this);
+}
+
+bool Action::eventFilter(QObject *watched, QEvent *event)
+{
+    if (watched == parent() && event->type() == QEvent::KeyPress) {
+        const QKeyEvent *keyEvent = static_cast<const QKeyEvent *>(event);
+        for (const QKeySequence::StandardKey &shortcut : m_shortcuts) {
+            if (keyEvent->matches(shortcut)) {
+                Q_EMIT activated();
+
+                return true;
+            }
+        }
+    }
+
+    return QObject::eventFilter(watched, event);
+}
diff --git a/src/Gui/Action.h b/src/Gui/Action.h
new file mode 100644
index 0000000000000000000000000000000000000000..691c621ab50df504dd40067c5d9ff42e7deb52bd
--- /dev/null
+++ b/src/Gui/Action.h
@@ -0,0 +1,84 @@
+#ifndef ACTION_H
+#define ACTION_H
+
+#include <QKeySequence>
+#include <QObject>
+
+class Action : public QObject
+{
+    Q_OBJECT
+
+public:
+    template<typename Func>
+    explicit Action(QObject *parent, const QList<QKeySequence::StandardKey> &shortcuts, const typename QtPrivate::FunctionPointer<Func>::Object *receiver, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : QObject(parent)
+        , m_shortcuts(shortcuts)
+    {
+        parent->installEventFilter(this);
+
+        connect(this, &Action::activated, receiver, std::move(slot), type);
+    }
+
+    template<typename Func>
+    explicit Action(const typename QtPrivate::FunctionPointer<Func>::Object *parent, const QList<QKeySequence::StandardKey> &shortcuts, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : QObject(parent)
+        , m_shortcuts(shortcuts)
+    {
+        parent->installEventFilter(this);
+
+        connect(this, &Action::activated, parent, std::move(slot), type);
+    }
+
+    template<typename Func>
+    explicit Action(QObject *parent, const QList<QKeySequence::StandardKey> &shortcuts, const QObject *context, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : QObject(parent)
+        , m_shortcuts(shortcuts)
+    {
+        parent->installEventFilter(this);
+
+        connect(this, &Action::activated, context, std::move(slot), type);
+    }
+
+    template<typename Func>
+    explicit Action(QObject *parent, QKeySequence::StandardKey shortcut, const typename QtPrivate::FunctionPointer<Func>::Object *receiver, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : Action(parent, QList<QKeySequence::StandardKey>() << shortcut, receiver, slot, type)
+    {}
+
+    template<typename Func>
+    explicit Action(const typename QtPrivate::FunctionPointer<Func>::Object *parent, QKeySequence::StandardKey shortcut, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : Action(parent, QList<QKeySequence::StandardKey>() << shortcut, slot, type)
+    {}
+
+    template<typename Func>
+    explicit Action(QObject *parent, QKeySequence::StandardKey shortcut, const QObject *context, Func slot, Qt::ConnectionType type = Qt::AutoConnection)
+        : Action(parent, QList<QKeySequence::StandardKey>() << shortcut, context, slot, type)
+    {}
+
+    template<typename Func>
+    explicit Action(QObject *parent, const QList<QKeySequence::StandardKey> &shortcuts, Func slot)
+        : QObject(parent)
+        , m_shortcuts(shortcuts)
+    {
+        parent->installEventFilter(this);
+
+        connect(this, &Action::activated, std::move(slot));
+    }
+
+    template<typename Func>
+    explicit Action(QObject *parent, QKeySequence::StandardKey shortcut, Func slot)
+        : Action(parent, QList<QKeySequence::StandardKey>() << shortcut, slot)
+    {}
+
+    ~Action() override;
+
+Q_SIGNALS:
+    void activated();
+
+protected:
+    bool eventFilter(QObject *watched, QEvent *event) override;
+
+private:
+    const QList<QKeySequence::StandardKey> m_shortcuts;
+};
+
+#endif // ACTION_H
diff --git a/src/Gui/CaptureWindow.cpp b/src/Gui/CaptureWindow.cpp
index 8da9b7a0f719576b6fc053cff1f3ca5a18b369b8..1bcfa5ede3d4158a9875211c55eeb4a537755535 100644
--- a/src/Gui/CaptureWindow.cpp
+++ b/src/Gui/CaptureWindow.cpp
@@ -7,6 +7,7 @@
 
 #include "CaptureWindow.h"
 
+#include "Action.h"
 #include "Config.h"
 #include "Gui/SpectacleWindow.h"
 #include "SpectacleCore.h"
@@ -111,6 +112,27 @@ void CaptureWindow::setMode(CaptureWindow::Mode mode)
             {QStringLiteral("parent"), QVariant::fromValue(contentItem())}
         };
         setSource(QUrl(QStringLiteral("qrc:/src/Gui/ImageCaptureOverlay.qml")), initialProperties);
+
+        new Action(this, QKeySequence::Save, this, &CaptureWindow::save);
+        new Action(this, QKeySequence::SaveAs, this, &CaptureWindow::saveAs);
+        new Action(this, QKeySequence::Copy, this, &CaptureWindow::copyImage);
+        new Action(this, QKeySequence::Print, this, &CaptureWindow::showPrintDialog);
+        new Action(this, QKeySequence::Undo, [this]() {
+            if (isAnnotating()) {
+                auto document = SpectacleCore::instance()->annotationDocument();
+                if (document->undoStackDepth() > 0) {
+                    document->undo();
+                }
+            }
+        });
+        new Action(this, QKeySequence::Redo, [this]() {
+            if (isAnnotating()) {
+                auto document = SpectacleCore::instance()->annotationDocument();
+                if (document->redoStackDepth() > 0) {
+                    document->redo();
+                }
+            }
+        });
     } else if (mode == Video) {
         
     }
@@ -147,37 +169,6 @@ void CaptureWindow::mousePressEvent(QMouseEvent *event)
     SpectacleWindow::mousePressEvent(event);
 }
 
-void CaptureWindow::keyReleaseEvent(QKeyEvent *event)
-{
-    SpectacleWindow::keyReleaseEvent(event);
-    if (event->isAccepted()) {
-        return;
-    }
-    if (event->matches(QKeySequence::Save)) {
-        event->accept();
-        save();
-    } else if (event->matches(QKeySequence::SaveAs)) {
-        event->accept();
-        saveAs();
-    } else if (event->matches(QKeySequence::Copy)) {
-        event->accept();
-        copyImage();
-    } else if (event->matches(QKeySequence::Print)) {
-        event->accept();
-        showPrintDialog();
-    }
-    auto document = SpectacleCore::instance()->annotationDocument();
-    if (!event->isAccepted() && document) {
-        if (document->undoStackDepth() > 0 && event->matches(QKeySequence::Undo)) {
-            event->accept();
-            document->undo();
-        } else if (document->redoStackDepth() > 0 && event->matches(QKeySequence::Redo)) {
-            event->accept();
-            document->redo();
-        }
-    }
-}
-
 void CaptureWindow::showEvent(QShowEvent *event)
 {
     SpectacleWindow::showEvent(event);
diff --git a/src/Gui/CaptureWindow.h b/src/Gui/CaptureWindow.h
index e8222ba3e049e002324bf58c64130917ea165dac..0dd618fa6effbc500f98eec15c4d7c894b94536c 100644
--- a/src/Gui/CaptureWindow.h
+++ b/src/Gui/CaptureWindow.h
@@ -45,7 +45,6 @@ Q_SIGNALS:
 
 protected:
     void mousePressEvent(QMouseEvent *event) override;
-    void keyReleaseEvent(QKeyEvent *event) override;
     void showEvent(QShowEvent *event) override;
 
 private:
diff --git a/src/Gui/SpectacleWindow.cpp b/src/Gui/SpectacleWindow.cpp
index 2e71763f50a153a3c9359d43da726de9571932f7..98a5fc262276520ca19fdf231e764c3dc5b81c76 100644
--- a/src/Gui/SpectacleWindow.cpp
+++ b/src/Gui/SpectacleWindow.cpp
@@ -8,6 +8,7 @@
 
 #include "SpectacleWindow.h"
 
+#include "Action.h"
 #include "ExportManager.h"
 #include "SpectacleCore.h"
 #include "spectacle_gui_debug.h"
@@ -97,6 +98,22 @@ SpectacleWindow::SpectacleWindow(QQmlEngine *engine, QWindow *parent)
     // set up QML
     setResizeMode(QQuickView::SizeRootObjectToView);
     m_context->setContextProperty(QStringLiteral("contextWindow"), this);
+
+    new Action(this, { QKeySequence::Cancel, QKeySequence::Close, QKeySequence::Quit }, [this]() {
+        auto spectacleCore = SpectacleCore::instance();
+        if (spectacleCore->captureTimeRemaining() > 0) {
+            spectacleCore->cancelScreenshot();
+        } else {
+            Q_EMIT SpectacleWindow::engine()->quit();
+        }
+    });
+    new Action(this, QKeySequence::Preferences, this, &SpectacleWindow::showPreferencesDialog);
+    new Action(this, QKeySequence::New, []() {
+        SpectacleCore::instance()->takeNewScreenshot();
+    });
+    new Action(this, QKeySequence::HelpContents, [this]() {
+        m_helpMenu->showAppHelp();
+    });
 }
 
 SpectacleWindow::~SpectacleWindow()
@@ -513,50 +530,4 @@ void SpectacleWindow::mousePressEvent(QMouseEvent *event)
     }
 }
 
-void SpectacleWindow::keyPressEvent(QKeyEvent *event)
-{
-    // Events need to be processed normally first for events to reach items
-    QQuickView::keyPressEvent(event);
-    if (event->isAccepted()) {
-        return;
-    }
-    m_pressedKeys = event->key() | event->modifiers();
-}
-
-void SpectacleWindow::keyReleaseEvent(QKeyEvent *event)
-{
-    // Events need to be processed normally first for events to reach items
-    QQuickView::keyReleaseEvent(event);
-    if (event->isAccepted()) {
-        return;
-    }
-    // Cancel defaults to Escape in QPlatformTheme.
-    // Handling this here fixes https://bugs.kde.org/show_bug.cgi?id=428478
-    if ((event->matches(QKeySequence::Quit)
-        || event->matches(QKeySequence::Close)
-        || event->matches(QKeySequence::Cancel))
-        // We need to check if these were pressed previously or else pressing escape
-        // in a dialog will quit spectacle when you release the escape key.
-        && m_pressedKeys == event->key() | event->modifiers()
-    ) {
-        event->accept();
-        auto spectacleCore = SpectacleCore::instance();
-        if (spectacleCore->captureTimeRemaining() > 0) {
-            spectacleCore->cancelScreenshot();
-        } else {
-            Q_EMIT engine()->quit();
-        }
-    } else if (event->matches(QKeySequence::Preferences)) {
-        event->accept();
-        showPreferencesDialog();
-    } else if (event->matches(QKeySequence::New)) {
-        event->accept();
-        SpectacleCore::instance()->takeNewScreenshot();
-    } else if (event->matches(QKeySequence::HelpContents)) {
-        event->accept();
-        m_helpMenu->showAppHelp();
-    }
-    m_pressedKeys = {};
-}
-
 #include "moc_SpectacleWindow.cpp"
diff --git a/src/Gui/SpectacleWindow.h b/src/Gui/SpectacleWindow.h
index 77be2cec7366465a46ceade570966033321d84bb..e2e73d42df70a35768b46f6c1a1f2a2b9a5eddd9 100644
--- a/src/Gui/SpectacleWindow.h
+++ b/src/Gui/SpectacleWindow.h
@@ -119,8 +119,6 @@ protected:
     void setSource(const QUrl &source, const QVariantMap &initialProperties);
 
     void mousePressEvent(QMouseEvent *event) override;
-    void keyPressEvent(QKeyEvent *event) override;
-    void keyReleaseEvent(QKeyEvent *event) override;
 
     static QVector<SpectacleWindow *> s_spectacleWindowInstances;
     static bool s_synchronizingVisibility;
@@ -135,6 +133,4 @@ protected:
     const std::unique_ptr<HelpMenu> m_helpMenu;
     const std::unique_ptr<QQmlContext> m_context;
     std::unique_ptr<QQmlComponent> m_component;
-
-    QKeySequence m_pressedKeys;
 };
diff --git a/src/Gui/ViewerWindow.cpp b/src/Gui/ViewerWindow.cpp
index 185c683daf44a271ee4761c86c84d7774d8da2c9..24c64981772cddca7928082fe0d0d1f407a1ce7d 100644
--- a/src/Gui/ViewerWindow.cpp
+++ b/src/Gui/ViewerWindow.cpp
@@ -7,6 +7,7 @@
 
 #include "ViewerWindow.h"
 
+#include "Action.h"
 #include "SpectacleCore.h"
 #include "spectacle_gui_debug.h"
 
@@ -108,6 +109,27 @@ void ViewerWindow::setMode(ViewerWindow::Mode mode)
         updateMinimumSize();
         connect(rootItem, SIGNAL(minimumWidthChanged()), this, SLOT(updateMinimumSize()));
         connect(rootItem, SIGNAL(minimumHeightChanged()), this, SLOT(updateMinimumSize()));
+
+        new Action(this, QKeySequence::Save, this, &ViewerWindow::save);
+        new Action(this, QKeySequence::SaveAs, this, &ViewerWindow::saveAs);
+        new Action(this, QKeySequence::Copy, this, &ViewerWindow::copyImage);
+        new Action(this, QKeySequence::Print, this, &ViewerWindow::showPrintDialog);
+        new Action(this, QKeySequence::Undo, [this]() {
+            if (isAnnotating()) {
+                auto document = SpectacleCore::instance()->annotationDocument();
+                if (document->undoStackDepth() > 0) {
+                    document->undo();
+                }
+            }
+        });
+        new Action(this, QKeySequence::Redo, [this]() {
+            if (isAnnotating()) {
+                auto document = SpectacleCore::instance()->annotationDocument();
+                if (document->redoStackDepth() > 0) {
+                    document->redo();
+                }
+            }
+        });
     } else if (mode == Video) {
         
     }
@@ -223,35 +245,4 @@ void ViewerWindow::resizeEvent(QResizeEvent *event)
     }
 }
 
-void ViewerWindow::keyReleaseEvent(QKeyEvent *event)
-{
-    SpectacleWindow::keyReleaseEvent(event);
-    if (event->isAccepted() || m_mode == Dialog) {
-        return;
-    }
-    if (event->matches(QKeySequence::Save)) {
-        event->accept();
-        save();
-    } else if (event->matches(QKeySequence::SaveAs)) {
-        event->accept();
-        saveAs();
-    } else if (event->matches(QKeySequence::Copy)) {
-        event->accept();
-        copyImage();
-    } else if (event->matches(QKeySequence::Print)) {
-        event->accept();
-        showPrintDialog();
-    }
-    auto document = SpectacleCore::instance()->annotationDocument();
-    if (!event->isAccepted() && document) {
-        if (document->undoStackDepth() > 0 && event->matches(QKeySequence::Undo)) {
-            event->accept();
-            document->undo();
-        } else if (document->redoStackDepth() > 0 && event->matches(QKeySequence::Redo)) {
-            event->accept();
-            document->redo();
-        }
-    }
-}
-
 #include "moc_ViewerWindow.cpp"
diff --git a/src/Gui/ViewerWindow.h b/src/Gui/ViewerWindow.h
index a347f395ba7d8e865f2e04ec867bfb941f84976a..901450a8cbb29b9dfbff07afab4f75c17f034345 100644
--- a/src/Gui/ViewerWindow.h
+++ b/src/Gui/ViewerWindow.h
@@ -53,7 +53,6 @@ Q_SIGNALS:
 protected:
     bool event(QEvent *event) override;
     void resizeEvent(QResizeEvent *event) override;
-    void keyReleaseEvent(QKeyEvent *event) override;
 
 private:
     explicit ViewerWindow(Mode mode, QQmlEngine *engine, QWindow *parent = nullptr);

Places

File 2002-fix-shortcuts.patch of Package spectacle (Revision f754057aabd1106030ff06450cde0b8d)

Places