Merge pull request #370 from antab/app-selection

App selection UI improvement

2 files changed, 36 insertions, 40 deletions

diff --git a/src-qt5/core-utils/lumina-config/AppDialog.h b/src-qt5/core-utils/lumina-config/AppDialog.h
index ea7615e2..4eb6471c 100644
--- a/src-qt5/core-utils/lumina-config/AppDialog.h
+++ b/src-qt5/core-utils/lumina-config/AppDialog.h
@@ -26,10 +26,15 @@ public:
 	AppDialog(QWidget *parent = 0) : QDialog(parent), ui(new Ui::AppDialog){
 	  ui->setupUi(this); //load the designer file
 	  appreset = false;
-	  ui->comboBox->clear();
+	  ui->listApps->clear();
           QList<XDGDesktop*> APPS = LXDG::sortDesktopNames(APPSLIST->apps(false,false)); //Don't show all/hidden
 	  for(int i=0; i<APPS.length(); i++){
-	    ui->comboBox->addItem( LXDG::findIcon(APPS[i]->icon,"application-x-executable"), APPS[i]->name, APPS[i]->filePath);
+	    QListWidgetItem *app = new QListWidgetItem(LXDG::findIcon(APPS[i]->icon,"application-x-executable"), APPS[i]->name);
+	    app->setData(Qt::UserRole, APPS[i]->filePath);
+	    ui->listApps->addItem(app);
+	  }
+	  if(ui->listApps->count()){
+	    ui->listApps->setCurrentItem(ui->listApps->item(0));
 	  }
 	  this->setWindowIcon( LXDG::findIcon("system-search","") );
 	  if(parent!=0){
@@ -56,7 +61,10 @@ public:
 		
 private slots:
 	void on_buttonBox_accepted(){
-	  appselected = ui->comboBox->currentData().toString();
+	  QListWidgetItem *item = ui->listApps->currentItem();
+	  if(item != nullptr){
+	    appselected = item->data(Qt::UserRole).toString();
+	  }
 	  this->close();
 	}
 	void on_buttonBox_rejected(){
@@ -68,6 +76,26 @@ private slots:
 	    this->close();
 	  }
 	}
+	void on_listApps_itemDoubleClicked(QListWidgetItem *item){
+	  appselected = item->data(Qt::UserRole).toString();
+	  this->close();
+	}
+	void on_lineSearch_textChanged(const QString &term){
+	  QListWidgetItem *first_visible = nullptr;
+	  for(int i = 0; i < ui->listApps->count(); i++){
+	    QListWidgetItem *item = ui->listApps->item(i);
+	    bool visible = item->text().contains(term, Qt::CaseInsensitive);
+	    item->setHidden(!visible);
+	    if(visible && first_visible == nullptr){
+	      first_visible = item;
+	    }
+	  }
+	  //Select the first app
+	  ui->listApps->setCurrentItem(first_visible);
+	  if(first_visible != nullptr){
+	    ui->listApps->scrollToItem(first_visible);
+	  }
+	}
 };
 
 #endif
diff --git a/src-qt5/core-utils/lumina-config/AppDialog.ui b/src-qt5/core-utils/lumina-config/AppDialog.ui
index 63323f7d..ec7de974 100644
--- a/src-qt5/core-utils/lumina-config/AppDialog.ui
+++ b/src-qt5/core-utils/lumina-config/AppDialog.ui
@@ -7,7 +7,7 @@
     <x>0</x>
     <y>0</y>
     <width>348</width>
-    <height>91</height>
+    <height>300</height>
    </rect>
   </property>
   <property name="windowTitle">
@@ -15,46 +15,14 @@
   </property>
   <layout class="QVBoxLayout" name="verticalLayout">
    <item>
-    <spacer name="verticalSpacer_2">
-     <property name="orientation">
-      <enum>Qt::Vertical</enum>
-     </property>
-     <property name="sizeHint" stdset="0">
-      <size>
-       <width>20</width>
-       <height>40</height>
-      </size>
-     </property>
-    </spacer>
-   </item>
-   <item>
-    <widget class="QComboBox" name="comboBox">
-     <property name="sizePolicy">
-      <sizepolicy hsizetype="Preferred" vsizetype="Preferred">
-       <horstretch>0</horstretch>
-       <verstretch>0</verstretch>
-      </sizepolicy>
-     </property>
-     <property name="iconSize">
-      <size>
-       <width>32</width>
-       <height>32</height>
-      </size>
+    <widget class="QLineEdit" name="lineSearch">
+     <property name="placeholderText">
+      <string>Search for....</string>
      </property>
     </widget>
    </item>
    <item>
-    <spacer name="verticalSpacer">
-     <property name="orientation">
-      <enum>Qt::Vertical</enum>
-     </property>
-     <property name="sizeHint" stdset="0">
-      <size>
-       <width>20</width>
-       <height>40</height>
-      </size>
-     </property>
-    </spacer>
+    <widget class="QListWidget" name="listApps"/>
    </item>
    <item>
     <widget class="QDialogButtonBox" name="buttonBox">