Unified buttons

Buttons in association dialog and action list have different style and behavior than buttons in dialogs. This patch unifies it by using jquery.button widget.

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

2 files changed, 28 insertions, 12 deletions

diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index 6f6d862b0..ddbe71c1f 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -1238,6 +1238,10 @@ table.scrollable tbody {
     width: 98%;
 }
 
+.adder-dialog-buttons > div:first-child{
+    margin-bottom: 0.5em;
+}
+
 /* ---- Widgets ---- */
 
 .text-widget input {
@@ -1655,11 +1659,6 @@ form#login {
     margin-left: 10px;
 }
 
-/* TODO: replace with button */
- #content .facet-action-list div[name=apply] a.ui-state-default {
-    padding: 3px 5px 1px 5px;
-}
-
 .facet-action-list select {
     font-size: 11px;
     padding-left: 4px;
diff --git a/install/ui/widget.js b/install/ui/widget.js
index 80fc1da1d..9e78c360b 100644
--- a/install/ui/widget.js
+++ b/install/ui/widget.js
@@ -2455,12 +2455,6 @@ IPA.link_widget = function(spec) {
 };
 
 IPA.action_button = function(spec) {
-    var button = IPA.button(spec);
-    button.removeClass("ui-state-default").addClass("action-button");
-    return button;
-};
-
-IPA.button = function(spec) {
 
     spec = spec || {};
 
@@ -2469,7 +2463,7 @@ IPA.button = function(spec) {
         name: spec.name,
         href: spec.href || '#' + (spec.name || 'button'),
         title: spec.title || spec.label,
-        'class': 'ui-state-default ui-corner-all button',
+        'class': 'button action-button',
         style: spec.style,
         click: spec.click,
         blur: spec.blur
@@ -2493,6 +2487,29 @@ IPA.button = function(spec) {
     return button;
 };
 
+IPA.button = function(spec) {
+
+    spec = spec || {};
+
+    var button = $('<a/>', {
+        id: spec.id,
+        name: spec.name,
+        href: spec.href || '#' + (spec.name || 'button')
+    });
+
+    var icons = { primary: spec.icon };
+    var label = spec.label;
+
+    button.button({
+        icons: icons,
+        label: label
+    });
+
+    button.click(spec.click);
+
+    return button;
+};
+
 IPA.button_widget = function(spec) {
 
     spec = spec || {};