From a013597e7c5ec4b6d665988d8aaaac941a7ff1a9 Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Fri, 30 Sep 2011 16:52:41 -0500
Subject: Disable sudo options Delete button if nothing selected.

The Delete button for sudo options in sudo rule details page now
will only work if there is at least one row selected.

Ticket #1896
---
 install/ui/sudo.js | 26 ++++++++++++++++++++++----
 1 file changed, 22 insertions(+), 4 deletions(-)

(limited to 'install')

diff --git a/install/ui/sudo.js b/install/ui/sudo.js
index 8586f5760..8228951c3 100644
--- a/install/ui/sudo.js
+++ b/install/ui/sudo.js
@@ -503,26 +503,44 @@ IPA.sudo.options_section = function(spec) {
 
             that.table.table_create(container);
 
-            var button = IPA.action_button({
+            that.remove_button = IPA.action_button({
                 name: 'remove',
                 label: IPA.messages.buttons.remove,
                 icon: 'remove-icon',
+                'class': 'action-button-disabled',
                 click: function() {
-                    that.remove_handler();
+                    if (!that.remove_button.hasClass('action-button-disabled')) {
+                        that.remove_handler();
+                    }
                     return false;
                 }
             }).appendTo(that.table.buttons);
 
-            button = IPA.action_button({
+            that.add_button = IPA.action_button({
                 name: 'add',
                 label: IPA.messages.buttons.add,
                 icon: 'add-icon',
                 click: function() {
-                    that.add_handler();
+                    if (!that.add_button.hasClass('action-button-disabled')) {
+                        that.add_handler();
+                    }
                     return false;
                 }
             }).appendTo(that.table.buttons);
         };
+
+        that.table.select_changed = function() {
+
+            var values = that.table.get_selected_values();
+
+            if (that.remove_button) {
+                if (values.length === 0) {
+                    that.remove_button.addClass('action-button-disabled');
+                } else {
+                    that.remove_button.removeClass('action-button-disabled');
+                }
+            }
+        };
     }
 
     that.add_handler = function() {
-- 
cgit