From 122115c0bad966972329822d5b0fea60b95a87ca Mon Sep 17 00:00:00 2001
From: Karthick Ramachandran <kramachandran@roblox.com>
Date: Thu, 24 Mar 2022 13:38:43 -0700
Subject: [PATCH] make stop job message clearer (#12252)

---
 .changelog/12252.txt                                 |  3 +++
 ui/app/templates/allocations/allocation/index.hbs    | 12 ++++++------
 .../templates/allocations/allocation/task/index.hbs  |  4 ++--
 ui/app/templates/clients/client/index.hbs            |  2 +-
 .../components/job-page/parts/latest-deployment.hbs  |  2 +-
 ui/app/templates/components/job-page/parts/title.hbs |  8 ++++----
 ui/app/templates/components/job-version.hbs          |  4 ++--
 7 files changed, 19 insertions(+), 16 deletions(-)
 create mode 100644 .changelog/12252.txt

diff --git a/.changelog/12252.txt b/.changelog/12252.txt
new file mode 100644
index 000000000..408f0fca6
--- /dev/null
+++ b/.changelog/12252.txt
@@ -0,0 +1,3 @@
+```release-note:improvement
+ui: make buttons with confirmation more descriptive of their actions
+```
diff --git a/ui/app/templates/allocations/allocation/index.hbs b/ui/app/templates/allocations/allocation/index.hbs
index 0069a9fba..e463ad9eb 100644
--- a/ui/app/templates/allocations/allocation/index.hbs
+++ b/ui/app/templates/allocations/allocation/index.hbs
@@ -42,9 +42,9 @@
         <TwoStepButton
           data-test-stop
           @alignRight={{true}}
-          @idleText="Stop"
-          @cancelText="Cancel"
-          @confirmText="Yes, Stop"
+          @idleText="Stop Alloc"
+          @cancelText="Cancel Stop"
+          @confirmText="Yes, Stop Alloc"
           @confirmationMessage="Are you sure? This will reschedule the allocation on a different client."
           @awaitingConfirmation={{this.stopAllocation.isRunning}}
           @disabled={{or
@@ -56,9 +56,9 @@
         <TwoStepButton
           data-test-restart
           @alignRight={{true}}
-          @idleText="Restart"
-          @cancelText="Cancel"
-          @confirmText="Yes, Restart"
+          @idleText="Restart Alloc"
+          @cancelText="Cancel Restart"
+          @confirmText="Yes, Restart Alloc"
           @confirmationMessage="Are you sure? This will restart the allocation in-place."
           @awaitingConfirmation={{this.restartAllocation.isRunning}}
           @disabled={{or
diff --git a/ui/app/templates/allocations/allocation/task/index.hbs b/ui/app/templates/allocations/allocation/task/index.hbs
index 49e90ec3c..e6d62dc20 100644
--- a/ui/app/templates/allocations/allocation/task/index.hbs
+++ b/ui/app/templates/allocations/allocation/task/index.hbs
@@ -53,9 +53,9 @@
         <TwoStepButton
           data-test-restart
           @alignRight={{true}}
-          @idleText="Restart"
+          @idleText="Restart Task"
           @cancelText="Cancel"
-          @confirmText="Yes, Restart"
+          @confirmText="Yes, Restart Task"
           @confirmationMessage="Are you sure? This will restart the task in-place."
           @awaitingConfirmation={{this.restartTask.isRunning}}
           @disabled={{this.restartTask.isRunning}}
diff --git a/ui/app/templates/clients/client/index.hbs b/ui/app/templates/clients/client/index.hbs
index 6cb4fa025..c347b6d2e 100644
--- a/ui/app/templates/clients/client/index.hbs
+++ b/ui/app/templates/clients/client/index.hbs
@@ -197,7 +197,7 @@
           data-test-drain-stop
           @idleText="Stop Drain"
           @cancelText="Cancel"
-          @confirmText="Yes, Stop"
+          @confirmText="Yes, Stop Drain"
           @confirmationMessage="Are you sure you want to stop this drain?"
           @awaitingConfirmation={{this.stopDrain.isRunning}}
           @onConfirm={{perform this.stopDrain}}
diff --git a/ui/app/templates/components/job-page/parts/latest-deployment.hbs b/ui/app/templates/components/job-page/parts/latest-deployment.hbs
index f11111a34..17b85ab73 100644
--- a/ui/app/templates/components/job-page/parts/latest-deployment.hbs
+++ b/ui/app/templates/components/job-page/parts/latest-deployment.hbs
@@ -24,7 +24,7 @@
                 confirmButton="is-danger"}}
               @idleText="Fail Deployment"
               @cancelText="Cancel"
-              @confirmText="Yes, Fail"
+              @confirmText="Yes, Fail Deployment"
               @confirmationMessage="Are you sure?"
               @inlineText={{true}}
               @awaitingConfirmation={{this.fail.isRunning}}
diff --git a/ui/app/templates/components/job-page/parts/title.hbs b/ui/app/templates/components/job-page/parts/title.hbs
index 068c0731f..a4fc8ed27 100644
--- a/ui/app/templates/components/job-page/parts/title.hbs
+++ b/ui/app/templates/components/job-page/parts/title.hbs
@@ -18,9 +18,9 @@
       <TwoStepButton
         data-test-stop
         @alignRight={{true}}
-        @idleText="Stop"
+        @idleText="Stop Job"
         @cancelText="Cancel"
-        @confirmText="Yes, Stop"
+        @confirmText="Yes, Stop Job"
         @confirmationMessage="Are you sure you want to stop this job?"
         @awaitingConfirmation={{this.stopJob.isRunning}}
         @onConfirm={{perform this.stopJob}} />
@@ -28,9 +28,9 @@
       <TwoStepButton
         data-test-start
         @alignRight={{true}}
-        @idleText="Start"
+        @idleText="Start Job"
         @cancelText="Cancel"
-        @confirmText="Yes, Start"
+        @confirmText="Yes, Start Job"
         @confirmationMessage="Are you sure you want to start this job?"
         @awaitingConfirmation={{this.startJob.isRunning}}
         @onConfirm={{perform this.startJob}} />
diff --git a/ui/app/templates/components/job-version.hbs b/ui/app/templates/components/job-version.hbs
index a7076d75f..74d4308b9 100644
--- a/ui/app/templates/components/job-version.hbs
+++ b/ui/app/templates/components/job-version.hbs
@@ -17,9 +17,9 @@
             idleButton="is-warning is-outlined"
             confirmButton="is-warning"}}
           @alignRight={{true}}
-          @idleText="Revert"
+          @idleText="Revert Version"
           @cancelText="Cancel"
-          @confirmText="Yes, Revert"
+          @confirmText="Yes, Revert Version"
           @confirmationMessage="Are you sure you want to revert to this version?"
           @inlineText={{true}}
           @fadingBackground={{true}}