Hi,

The attached patch addresses item #5 of this bug:
https://fedorahosted.org/freeipa/ticket/670

A new CSS class has been added for buttons without icons. The
IPA.button() has been modified to use this class if there is
no icons specified.

--
Endi S. Dewata
From 4b7234f363313710ef82b7874d8abfcdb88dc54f Mon Sep 17 00:00:00 2001
From: Endi S. Dewata <edew...@redhat.com>
Date: Tue, 18 Jan 2011 07:10:41 +0700
Subject: [PATCH] CSS class for buttons without icons.

A new CSS class has been added for buttons without icons. The
IPA.button() has been modified to use this class if there is
no icons specified.
---
 install/static/details.js |   10 ++++++++--
 install/static/ipa.css    |    7 +++++++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/install/static/details.js b/install/static/details.js
index 5c6bc83a2c3aac36f2c81619a05d85bc852289cb..aad77a9c4a5ff3b05fa3def8c7c300e12d1fcd67 100644
--- a/install/static/details.js
+++ b/install/static/details.js
@@ -699,12 +699,18 @@ IPA.button = function(spec) {
         id: spec.id,
         html: spec.label,
         title: spec.title || spec.label,
-        'class': 'ui-state-default ui-corner-all input_link'
+        'class': 'ui-state-default ui-corner-all'
     });
 
     if (spec.click) button.click(spec.click);
     if (spec['class']) button.addClass(spec['class']);
-    if (spec.icon) button.append('<span class="ui-icon '+spec.icon+'" ></span> ');
+
+    if (spec.icon) {
+        button.addClass('input_link');
+        button.append('<span class="ui-icon '+spec.icon+'" ></span> ');
+    } else {
+        button.addClass('button-without-icon');
+    }
 
     return button;
 };
diff --git a/install/static/ipa.css b/install/static/ipa.css
index 518e8aa266bb8a25f8335d81e383dcb629a35a0d..66b0509e254a9d222e31387bc2e1406e01b9e9e4 100644
--- a/install/static/ipa.css
+++ b/install/static/ipa.css
@@ -52,6 +52,13 @@ body{
     top: 50%;
 }
 
+.button-without-icon {
+    padding: .4em 1em .4em 1em;
+    text-decoration: none;
+    position: relative;
+    cursor: pointer;
+}
+
 /*  ----  Header  ----  */
 div.header {
     background-color:#0C3B00;
-- 
1.6.6.1

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to