From 4db088d7f484c9446c27df32837a033721cb1dcb 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/ui/sudo.js')

diff --git a/install/ui/sudo.js b/install/ui/sudo.js
index 6855283e..46a558d8 100644
--- a/install/ui/sudo.js
+++ b/install/ui/sudo.js
@@ -539,26 +539,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