feat(dialogSettings): Allow to remove folder in monitoring tab with D…

…elete or BackSpace key
MathiusD · Feb 19, 2024 · ecec774 · ecec774
1 parent c13cf65
commit ecec774
Show file tree

Hide file tree

Showing 3 changed files with 37 additions and 3 deletions.
diff --git a/src/dialogsettings.cpp b/src/dialogsettings.cpp
@@ -1,3 +1,5 @@
+#include <Qt>
+
 #include <QFileDialog>
 #include <QColorDialog>
 #include <QMessageBox>
@@ -117,6 +119,10 @@ DialogSettings::DialogSettings( QWidget *parent)
     mAccountModel = new ModelAccountTree(this, treeAccounts);
     treeAccounts->header()->setSectionResizeMode(QHeaderView::ResizeToContents);
     treeAccounts->setCurrentIndex(mAccountModel->index(0, 0));
+    treeAccounts->onKeyPressed(this, [](void * handle, QKeyEvent * event)
+    {
+        static_cast<DialogSettings*>(handle)->onKeyPressedOnTreeAccount(event);
+    });
 
     // New emails tab
     mModelNewEmails = new ModelNewEmails( this );
@@ -150,6 +156,14 @@ DialogSettings::DialogSettings( QWidget *parent)
 #endif
 }
 
+void DialogSettings::onKeyPressedOnTreeAccount(QKeyEvent * event)
+{
+    if (event->key() == Qt::Key_Delete || event->key() == Qt::Key_Backspace)
+    {
+        accountRemove();
+    }
+}
+
 void DialogSettings::accept()
 {
     BirdtrayApp* app = BirdtrayApp::get();
@@ -285,8 +299,13 @@ void DialogSettings::accountEditIndex(const QModelIndex &index)
 
 void DialogSettings::accountRemove()
 {
-    QModelIndex index = treeAccounts->currentIndex();
+    if ( treeAccounts->currentIndex().isValid() )
+        accountRemoveIndex( treeAccounts->currentIndex() );
+}
 
+
+void DialogSettings::accountRemoveIndex(const QModelIndex &index)
+{
     if ( !index.isValid() )
         return;
 
@@ -311,7 +330,12 @@ void DialogSettings::newEmailEditIndex(const QModelIndex &index)
 
 void DialogSettings::newEmailRemove()
 {
-    mModelNewEmails->remove( treeNewEmails->currentIndex() );
+    newEmailRemoveIndex( treeNewEmails->currentIndex() );
+}
+
+void DialogSettings::newEmailRemoveIndex(const QModelIndex &index)
+{
+    mModelNewEmails->remove( index );
 }
 
 void DialogSettings::onCheckUpdateButton() {

diff --git a/src/dialogsettings.h b/src/dialogsettings.h
@@ -2,6 +2,7 @@
 #define SETTINGSDIALOG_H
 
 #include <QDialog>
+#include <QKeyEvent>
 #include <QProgressDialog>
 #include <QPalette>
 #include <QtCore/QStringListModel>
@@ -51,12 +52,14 @@ class DialogSettings : public QDialog, public Ui::DialogSettings
         void    accountEdit();
         void    accountEditIndex( const QModelIndex& index );
         void    accountRemove();
+        void    accountRemoveIndex( const QModelIndex& index );
 
         // New Email buttons
         void    newEmailAdd();
         void    newEmailEdit();
         void    newEmailEditIndex( const QModelIndex& index );
         void    newEmailRemove();
+        void    newEmailRemoveIndex( const QModelIndex& index );
 
         // Advanced buttons
         void    onCheckUpdateButton();
@@ -77,6 +80,8 @@ class DialogSettings : public QDialog, public Ui::DialogSettings
          */
         static void onTranslatorsDialog();
 
+        void onKeyPressedOnTreeAccount(QKeyEvent * event);
+
     private:
         void    changeIcon(QToolButton * button );
 

diff --git a/src/dialogsettings.ui b/src/dialogsettings.ui
@@ -291,7 +291,7 @@
           <item>
            <layout class="QHBoxLayout" name="horizontalLayout_2">
             <item>
-             <widget class="QTreeView" name="treeAccounts">
+             <widget class="QTreeViewWithKeyEvents" name="treeAccounts">
               <property name="enabled">
                <bool>true</bool>
               </property>
@@ -1167,6 +1167,11 @@ p, li { white-space: pre-wrap; }
    <extends>QPushButton</extends>
    <header>colorbutton.h</header>
   </customwidget>
+  <customwidget>
+    <class>QTreeViewWithKeyEvents</class>
+    <extends>QTreeView</extends>
+    <header>qtcomponents.h</header>
+  </customwidget>
  </customwidgets>
  <tabstops>
   <tabstop>tabWidget</tabstop>