ACK Looks good.

----- Original Message -----
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
From 9611600891b8e594f25cd04f8aa5b2f0c1ca79f9 Mon Sep 17 00:00:00 2001
From: Adam Young <ayo...@redhat.com>
Date: Mon, 24 Jan 2011 22:21:31 -0500
Subject: [PATCH] aci attribute table two columns

---
 install/ui/aci.js  |   31 +++++++++++++++----------------
 install/ui/ipa.css |    9 +++++----
 2 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/install/ui/aci.js b/install/ui/aci.js
index 85cfcaa850a17a94e40c53320c41b11b444b5f3a..eb941cb97a3f201bb17b8cc5828b1e19d1e539b8 100644
--- a/install/ui/aci.js
+++ b/install/ui/aci.js
@@ -42,6 +42,7 @@ IPA.populate_attribute_table = function (table, entity){
             id:'aciattr-'+aciattrs[a].toLowerCase(),
             "class":'aci-attribute'
         }));
+        td =  $('<td/>').appendTo(aci_tr);
         td.append($('<label/>',{
             text:aciattrs[a].toLowerCase()}));
     }
@@ -58,25 +59,23 @@ IPA.attribute_table_widget= function (spec){
         var dd  = $('<dd/>').appendTo(container);
         table =   $('<table/>',{
             id:id,
-            'class':'search-table'}).
+            'class':'search-table aci-attribute-table'}).
             append('<thead/>').
-            append($('<tbody/>',{
-                style:"height:30em;   overflow:auto;"
-            })).
+            append($('<tbody/>')).
             appendTo(dd);
 
-        $('<tr></tr>').
-            append($('<th/>',{
-                style:"height:2em; vertical-align:bottom;",
-                html:$('<input/>',{
-                    type: "checkbox",
-                    click: function(){
-                        $('.aci-attribute').
-                            attr('checked', $(this).attr('checked'));
-                    }}).
-                    after('<label>Attribute</lable>')
-            })).
-            appendTo($('thead', table));
+        var tr = $('<tr></tr>').appendTo($('thead', table));
+        tr.append($('<th/>',{
+            style:"height:2em; vertical-align:bottom;",
+            html:$('<input/>',{
+                type: "checkbox",
+                click: function(){
+                    $('.aci-attribute').
+                        attr('checked', $(this).attr('checked'));
+                }})
+        })).
+            append('<th class="aci-attribute-column">Attribute</th>');
+
         IPA.populate_attribute_table(table, object_type);
     };
 
diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index 5726ad99cae139ea8b7b99f09445579c6ad5d95a..37324f891666872fca91a3c7f4a1df057e310fd8 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -597,12 +597,13 @@ a.action-button-disabled {
     margin-top: 2em;
 }
 
-.search-table tr:nth-child(even){
-#    background-color:#CCC;
+.aci-attribute-table tbody{
+    height:30em;
+    overflow:auto;
 }
 
-.search-table tr:nth-child(odd){
-#    background-color:#FFF;
+.aci-attribute-table th.aci-attribute-column{
+    width: 25em;
 }
 
 .entity-views{
-- 
1.7.3.5

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

Reply via email to