From 4db088d7f484c9446c27df32837a033721cb1dcb Mon Sep 17 00:00:00 2001 From: "Endi S. Dewata" Date: Fri, 30 Sep 2011 16:52:41 -0500 Subject: [PATCH] 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(-) diff --git a/install/ui/sudo.js b/install/ui/sudo.js index 6855283ec..46a558d85 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() {