Added warning dialogue to ActivityDelete action (#1339)

* Added warning dialogue to ActivityDelete action

* Combine the warning dialogues into a single choice

Author: mrvisscher

activity_browser/actions/activity/activity_delete.py

@@ -27,27 +27,30 @@ def run(activity_keys: List[tuple]):
         # retrieve activity objects from the controller using the provided keys
         activities = [bd.get_activity(key) for key in activity_keys]
 
+        warning_text = f"Are you certain you want to delete {len(activities)} activity/activities?"
+
         # check for downstream processes
         if any(len(act.upstream()) > 0 for act in activities):
             # warning text
-            text = (
-                "One or more activities have downstream processes. Deleting these activities will remove the "
-                "exchange from the downstream processes, this can't be undone.\n\nAre you sure you want to "
-                "continue?"
+            warning_text += (
+                "\n\nOne or more activities have downstream processes. Deleting these activities will remove the "
+                "exchange from the downstream processes as well."
             )
 
-            # alert the user
-            choice = QtWidgets.QMessageBox.warning(
-                application.main_window,
-                "Activity/Activities has/have downstream processes",
-                text,
-                QtWidgets.QMessageBox.Yes | QtWidgets.QMessageBox.No,
-                QtWidgets.QMessageBox.No,
-            )
+        # alert the user
+        choice = QtWidgets.QMessageBox.warning(
+            application.main_window,
+            "Deleting activity/activities",
+            warning_text,
+            QtWidgets.QMessageBox.Yes | QtWidgets.QMessageBox.No,
+            QtWidgets.QMessageBox.No,
+        )
+
+        # return if the user cancels
+        if choice == QtWidgets.QMessageBox.No:
+            return
+
 
-            # return if the user cancels
-            if choice == QtWidgets.QMessageBox.No:
-                return
 
         # use the activity controller to delete multiple activities
         for act in activities: