diff options
| author | Endi S. Dewata <edewata@redhat.com> | 2011-09-30 16:52:41 -0500 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2011-10-03 15:15:45 +0200 |
| commit | 4db088d7f484c9446c27df32837a033721cb1dcb (patch) | |
| tree | 508b8769e74f7f10f7ecc407b4b7ed4a43e828e7 | |
| parent | 62817a4f7215b37130cd3b29c78581744bb033fd (diff) | |
| download | freeipa-4db088d7f484c9446c27df32837a033721cb1dcb.tar.gz freeipa-4db088d7f484c9446c27df32837a033721cb1dcb.tar.xz freeipa-4db088d7f484c9446c27df32837a033721cb1dcb.zip | |
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
| -rw-r--r-- | install/ui/sudo.js | 26 |
1 files 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() { |