From 9438330329cb018c5d54063fa95d9a23fffb1a3e Mon Sep 17 00:00:00 2001
From: Michael Lange <dingoeatingfuzz@gmail.com>
Date: Thu, 30 Jan 2020 14:34:43 -0800
Subject: [PATCH] Add an explanatory tooltip to the unauthorized node drain
 popover

---
 ui/app/templates/components/drain-popover.hbs | 7 ++++++-
 ui/app/templates/components/popover-menu.hbs  | 3 ++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/ui/app/templates/components/drain-popover.hbs b/ui/app/templates/components/drain-popover.hbs
index 0b9a4f9b5..a714cc5ac 100644
--- a/ui/app/templates/components/drain-popover.hbs
+++ b/ui/app/templates/components/drain-popover.hbs
@@ -2,7 +2,12 @@
   data-test-drain-popover
   isDisabled=isDisabled
   label=(if client.isDraining "Update Drain" "Drain")
-  triggerClass=(concat "is-small " (if drain.isRunning "is-loading")) as |m|}}
+  tooltip=(if isDisabled "Not allowed to drain clients")
+  triggerClass=(concat
+    "is-small "
+    (if drain.isRunning "is-loading ")
+    (if isDisabled "tooltip is-right-aligned")
+  ) as |m|}}
   <form data-test-drain-popover-form onsubmit={{action (queue (action preventDefault) (perform drain m.actions.close))}} class="form is-small">
     <h4 class="group-heading">Drain Options</h4>
     <div class="field">
diff --git a/ui/app/templates/components/popover-menu.hbs b/ui/app/templates/components/popover-menu.hbs
index 4eb6d1df6..98753c996 100644
--- a/ui/app/templates/components/popover-menu.hbs
+++ b/ui/app/templates/components/popover-menu.hbs
@@ -6,7 +6,8 @@
 >
   <dd.Trigger
     data-test-popover-trigger
-    class={{concat "popover-trigger button is-primary " triggerClass (if isDisabled "is-disabled")}}
+    class={{concat "popover-trigger button is-primary " triggerClass (if isDisabled " is-disabled")}}
+    aria-label={{tooltip}}
     {{on "keydown" (action "openOnArrowDown" dd)}}
   >
     {{label}} {{x-icon "chevron-down" class="is-text"}}