From ac7df79a43732cead50f83e31220b0bf2d0230f4 Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvoborni@redhat.com>
Date: Tue, 22 Jul 2014 16:43:28 +0200
Subject: [PATCH] webui: remove remaining action-button-disabled occurrences

Buttons in hbactest check for 'action-button-disabled' but it's never set.

https://fedorahosted.org/freeipa/ticket/4258

Reviewed-By: Endi Sukma Dewata <edewata@redhat.com>
---
 install/ui/ipa.css                 |  9 --------
 install/ui/src/freeipa/hbactest.js | 35 +++++-------------------------
 2 files changed, 5 insertions(+), 39 deletions(-)

diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index ec79688a2..2e70a1adc 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -252,15 +252,6 @@ div[name=settings].facet-group li a {
     word-wrap: break-word;
 }
 
-.action-button-disabled,
-.action-button-disabled:focus,
-.action-button-disabled:hover {
-    color: gray;
-    cursor: default;
-    text-decoration: none;
-    outline: none;
-}
-
 .aci-attribute-table tbody {
     height: 10em;
 }
diff --git a/install/ui/src/freeipa/hbactest.js b/install/ui/src/freeipa/hbactest.js
index 7a9d85ab3..9ac4e8293 100644
--- a/install/ui/src/freeipa/hbactest.js
+++ b/install/ui/src/freeipa/hbactest.js
@@ -184,12 +184,7 @@ IPA.hbac.test_facet = function(spec) {
                 name: 'prev',
                 label: '@i18n:widget.prev',
                 icon: 'fa-chevron-left',
-                click: function() {
-                    if (!that.prev_button.hasClass('action-button-disabled')) {
-                        that.prev();
-                    }
-                    return false;
-                }
+                click: that.prev
             }).appendTo(buttons);
 
             buttons.append(' ');
@@ -199,12 +194,7 @@ IPA.hbac.test_facet = function(spec) {
             name: 'next',
             label: '@i18n:widget.next',
             icon: 'fa-chevron-right',
-            click: function() {
-                if (!that.next_button.hasClass('action-button-disabled')) {
-                    that.next();
-                }
-                return false;
-            }
+            click: that.next
         }).appendTo(buttons);
     };
 
@@ -535,12 +525,7 @@ IPA.hbac.test_run_facet = function(spec) {
             name: 'run_test',
             label: '@i18n:objects.hbactest.run_test',
             icon: 'fa-gear',
-            click: function() {
-                if (!that.run_button.hasClass('action-button-disabled')) {
-                    that.run();
-                }
-                return false;
-            }
+            click: that.run
         }).appendTo(button_panel);
 
         var result_panel = $('<div/>', {
@@ -608,12 +593,7 @@ IPA.hbac.test_run_facet = function(spec) {
             name: 'prev',
             label: '@i18n:widget.prev',
             icon: 'fa-chevron-left',
-            click: function() {
-                if (!that.prev_button.hasClass('action-button-disabled')) {
-                    that.prev();
-                }
-                return false;
-            }
+            click: that.prev
         }).appendTo(buttons);
 
         buttons.append(' ');
@@ -622,12 +602,7 @@ IPA.hbac.test_run_facet = function(spec) {
             name: 'new_test',
             label: '@i18n:objects.hbactest.new_test',
             icon: 'fa-repeat',
-            click: function() {
-                if (!that.new_test_button.hasClass('action-button-disabled')) {
-                    that.new_test();
-                }
-                return false;
-            }
+            click: that.new_test
         }).appendTo(buttons);
     };