make refresh Not In Timeline smart bin more efficient

Author: ddennedy

src/docks/filesdock.cpp

@@ -570,7 +570,7 @@ FilesDock::FilesDock(QWidget *parent)
     ui->treeView->setExpanded(homeIndex, true);
     ui->treeView->scrollTo(homeIndex);
     ui->treeView->setCurrentIndex(homeIndex);
-    QTimer::singleShot(0, this, [=](){
+    QTimer::singleShot(0, this, [ = ]() {
         ui->treeView->setVisible(Settings.filesFoldersOpen());
     });
     connect(ui->treeView, &QWidget::customContextMenuRequested, this, [ = ](const QPoint & pos) {

src/docks/playlistdock.cpp

@@ -1755,8 +1755,7 @@ void PlaylistDock::onPlaylistCleared()
 void PlaylistDock::refreshTimelineSmartBins()
 {
     auto items = ui->treeWidget->selectedItems();
-    if (!items.isEmpty() && SmartBinNotInTimeline == items.first()->data(0, Qt::UserRole).toInt()
-            && m_proxyModel->rowCount() > 0) {
+    if (!items.isEmpty() && SmartBinNotInTimeline == items.first()->data(0, Qt::UserRole).toInt()) {
         m_proxyModel->setSmartBin(SmartBinNotInTimeline);
     }
 }

src/mainwindow.cpp

@@ -485,7 +485,9 @@ void MainWindow::setupAndConnectDocks()
     connect(m_timelineDock->model(), SIGNAL(created()), SLOT(onMultitrackCreated()));
     connect(m_timelineDock->model(), SIGNAL(closed()), SLOT(onMultitrackClosed()));
     connect(m_timelineDock->model(), SIGNAL(modified()), SLOT(onMultitrackModified()));
-    connect(m_timelineDock->model(), &MultitrackModel::modified, m_playlistDock,
+    connect(m_timelineDock->model(), &QAbstractItemModel::rowsInserted, m_playlistDock,
+            &PlaylistDock::refreshTimelineSmartBins);
+    connect(m_timelineDock->model(), &QAbstractItemModel::rowsRemoved, m_playlistDock,
             &PlaylistDock::refreshTimelineSmartBins);
     connect(m_timelineDock->model(), SIGNAL(durationChanged()), SLOT(onMultitrackDurationChanged()));
     connect(m_timelineDock, SIGNAL(clipOpened(Mlt::Producer *)), SLOT(openCut(Mlt::Producer *)));