Removed sortGroupRequested() as per review comments

keepassxreboot · Sep 29, 2021 · 56cd461 · 56cd461
1 parent d807872
commit 56cd461
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/src/gui/group/GroupView.cpp b/src/gui/group/GroupView.cpp
@@ -47,10 +47,10 @@ GroupView::GroupView(Database* db, QWidget* parent)
 
     // keyboard shortcuts to sort children of a group
     auto shortcut = new QShortcut(Qt::CTRL + Qt::Key_Down, this);
-    connect(shortcut, &QShortcut::activated, this, [this]() { sortGroupRequested(false); });
+    connect(shortcut, &QShortcut::activated, this, [this]() { sortGroups(false); });
 
     shortcut = new QShortcut(Qt::CTRL + Qt::Key_Up, this);
-    connect(shortcut, &QShortcut::activated, this, [this]() { sortGroupRequested(true); });
+    connect(shortcut, &QShortcut::activated, this, [this]() { sortGroups(true); });
 
     modelReset();
 
@@ -75,11 +75,6 @@ void GroupView::contextMenuShortcutPressed()
     }
 }
 
-void GroupView::sortGroupRequested(bool reverse)
-{
-    sortGroups(reverse);
-}
-
 void GroupView::changeDatabase(const QSharedPointer<Database>& newDb)
 {
     m_model->changeDatabase(newDb.data());

diff --git a/src/gui/group/GroupView.h b/src/gui/group/GroupView.h
@@ -46,7 +46,6 @@ private slots:
     void syncExpandedState(const QModelIndex& parent, int start, int end);
     void modelReset();
     void contextMenuShortcutPressed();
-    void sortGroupRequested(bool reverse);
 
 protected:
     void dragMoveEvent(QDragMoveEvent* event) override;