Table Browser: progress dialog in copy to clipboard

mgrojo · mgrojo · commit d1a61c87e55d · 2024-06-23T22:16:08.000+02:00
A progress dialog appears if the copy operation is going to take more than 2 seconds. The operation can be aborted. See issues #3664 and #3632.
diff --git a/src/ExtendedTableWidget.cpp b/src/ExtendedTableWidget.cpp
@@ -26,6 +26,7 @@
 #include <QComboBox>
 #include <QPainter>
 #include <QShortcut>
+#include <QProgressDialog>
 
 #include <limits>
 
@@ -506,7 +507,6 @@ void ExtendedTableWidget::reloadSettings()
 
 void ExtendedTableWidget::copyMimeData(const QModelIndexList& fromIndices, QMimeData* mimeData, const bool withHeaders, const bool inSQL)
 {
-    QApplication::setOverrideCursor(Qt::WaitCursor);
     QModelIndexList indices = fromIndices;
 
     // Remove all indices from hidden columns, because if we don't, we might copy data from hidden columns as well which is very
@@ -624,6 +624,11 @@ void ExtendedTableWidget::copyMimeData(const QModelIndexList& fromIndices, QMime
         }
     }
 
+    QProgressDialog progress(this);
+    progress.setWindowModality(Qt::ApplicationModal);
+    progress.setRange(*rowsInIndexes.begin(), *rowsInIndexes.end());
+    progress.setMinimumDuration(2000);
+
     // Iterate over rows x cols checking if the index actually exists when needed, in order
     // to support non-rectangular selections.
     for(const int row : rowsInIndexes) {
@@ -713,14 +718,19 @@ void ExtendedTableWidget::copyMimeData(const QModelIndexList& fromIndices, QMime
         else
             htmlResult.append("</tr>");
         result.append(rowSepText);
+
+        progress.setValue(row);
+        // Abort the operation if the user pressed ESC key or Cancel button
+        if (progress.wasCanceled()) {
+            return;
+        }
     }
 
     if (!inSQL) {
         htmlResult.append("</table></body></html>");
         mimeData->setHtml(htmlResult);
     }
     mimeData->setText(result);
-    QApplication::restoreOverrideCursor();
 }
 
 void ExtendedTableWidget::copy(const bool withHeaders, const bool inSQL )
