Improve file selection in file system widget

filesystemwidget.cpp

@@ -17,7 +17,7 @@ FilesystemWidget::FilesystemWidget(QAbstractItemModel *model, QWidget *parent) :
 
     setLayout(layout);
 
-    connect(m_listView, SIGNAL(doubleClicked(QModelIndex)), this, SLOT(fileClicked(QModelIndex)));
+    connect(m_listView->selectionModel(), &QItemSelectionModel::currentChanged, this, &FilesystemWidget::fileClicked);
 }
 
 void FilesystemWidget::setDir(const QString &dir)
@@ -34,9 +34,10 @@ void FilesystemWidget::selectFile(int row)
     m_listView->scrollTo(index);
 }
 
-void FilesystemWidget::fileClicked(const QModelIndex &index)
+void FilesystemWidget::fileClicked(const QModelIndex &index, const QModelIndex &)
 {
-    emit fileSelected(index.row());
+    if(index.isValid())
+        emit fileSelected(index.row());
 }
 
 Filetree::Filetree(QWidget *parent) : QTreeView(parent)

filesystemwidget.h

@@ -16,7 +16,7 @@ public:
     void setDir(const QString &dir);
 private slots:
     void selectFile(int row);
-    void fileClicked(const QModelIndex &index);
+    void fileClicked(const QModelIndex &index, const QModelIndex &);
 signals:
     void fileSelected(int row);
 };

imageringlist.cpp

@@ -210,30 +210,38 @@ void ImageRingList::loadFile(int row)
 {
     if(row < m_images.size())
     {
-        m_firstImage = m_currImage = m_lastImage = m_images.begin()+row;
-        if(m_images.empty())
-            return;
-
-        (*m_currImage)->load();
-
-        m_width = DEFAULT_WIDTH<m_images.size()/2 ? DEFAULT_WIDTH : m_images.size()/2;
-        if(m_liveMode)
-            m_width = 0;
-
-        for(int i=0; i<m_width; i++)
+        int diff = m_currImage != m_images.end() ? row - (m_currImage - m_images.begin()) : -100;
+        if(diff == 1)
+            increment();
+        else if(diff == -1)
+            decrement();
+        else
         {
-            m_firstImage = decrement(m_firstImage);
-            (*m_firstImage)->load();
-            m_lastImage = increment(m_lastImage);
-            (*m_lastImage)->load();
-        }
-        if(m_lastImage != m_firstImage)
-        {
-            QList<ImagePtr>::iterator iter = increment(m_lastImage);
-            while(m_firstImage != iter)
+            m_firstImage = m_currImage = m_lastImage = m_images.begin()+row;
+            if(m_images.empty())
+                return;
+
+            (*m_currImage)->load();
+
+            m_width = DEFAULT_WIDTH<m_images.size()/2 ? DEFAULT_WIDTH : m_images.size()/2;
+            if(m_liveMode)
+                m_width = 0;
+
+            for(int i=0; i<m_width; i++)
             {
-                (*iter)->release();
-                iter = increment(iter);
+                m_firstImage = decrement(m_firstImage);
+                (*m_firstImage)->load();
+                m_lastImage = increment(m_lastImage);
+                (*m_lastImage)->load();
+            }
+            if(m_lastImage != m_firstImage)
+            {
+                QList<ImagePtr>::iterator iter = increment(m_lastImage);
+                while(m_firstImage != iter)
+                {
+                    (*iter)->release();
+                    iter = increment(iter);
+                }
             }
         }
     }
@@ -333,6 +341,7 @@ void ImageRingList::setFiles(const QStringList files, const QString &currentFile
         index = 0;
 
     endResetModel();
+    m_currImage = m_images.end();
     loadFile(index);
 }
 

mainwindow.cpp

@@ -216,9 +216,11 @@ void MainWindow::keyPressEvent(QKeyEvent *event)
     switch (event->key())
     {
     case Qt::Key_Left:
+    case Qt::Key_Up:
         m_ringList->decrement();
         break;
     case Qt::Key_Right:
+    case Qt::Key_Down:
         m_ringList->increment();
         break;
     default: