Use KStandardAction for flamegraph forward/back actions.
authorMilian Wolff <mail@milianw.de>
Mon, 13 Feb 2017 23:25:11 +0000 (00:25 +0100)
committerMilian Wolff <mail@milianw.de>
Mon, 13 Feb 2017 23:27:30 +0000 (00:27 +0100)
src/analyze/gui/flamegraph.cpp

index a16690b18c2ef9e0bbda235760c5d3d10f8119c6..8ea8b06156d1ac980902e59ed8c1997dd2b5ea53 100644 (file)
@@ -39,6 +39,7 @@
 
 #include <KColorScheme>
 #include <KLocalizedString>
+#include <KStandardAction>
 #include <ThreadWeaver/ThreadWeaver>
 
 enum CostType
@@ -438,18 +439,8 @@ FlameGraph::FlameGraph(QWidget* parent, Qt::WindowFlags flags)
     layout()->addWidget(m_view);
     layout()->addWidget(m_displayLabel);
 
-    {
-        auto action = new QAction(tr("back"), this);
-        action->setShortcuts({QKeySequence::Back, Qt::Key_Backspace});
-        connect(action, &QAction::triggered, this, &FlameGraph::navigateBack);
-        addAction(action);
-    }
-    {
-        auto action = new QAction(tr("forward"), this);
-        action->setShortcuts(QKeySequence::Forward);
-        connect(action, &QAction::triggered, this, &FlameGraph::navigateForward);
-        addAction(action);
-    }
+    addAction(KStandardAction::back(this, &FlameGraph::navigateBack, this));
+    addAction(KStandardAction::forward(this, &FlameGraph::navigateForward, this));
 }
 
 FlameGraph::~FlameGraph() = default;