fix: keep monitor order when converting to downloads (fix #3136)

Bionus · Apr 25, 2024 · a164b1e · a164b1e
1 parent 234b4a8
commit a164b1e
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/gui/src/tabs/monitors-tab.cpp b/src/gui/src/tabs/monitors-tab.cpp
@@ -121,9 +121,13 @@ void MonitorsTab::removeSelected()
 
 void MonitorsTab::convertSelected()
 {
-	QSet<int> rows;
+	// We don't use a QSet because the order of rows is important
+	QList<int> rows;
 	for (const QModelIndex &index : ui->tableMonitors->selectionModel()->selection().indexes()) {
-		rows.insert(m_monitorTableModel->mapToSource(index).row());
+		const int row = m_monitorTableModel->mapToSource(index).row();
+		if (!rows.contains(row)) {
+			rows.append(row);
+		}
 	}
 
 	for (const int row : rows) {