Hi,

The attached patch should fix item #4 and #5 of this bug:
https://fedorahosted.org/freeipa/ticket/670

The OTP field has been moved into a separate row to avoid line
wrapping. The line height inside tables has been increased to
avoid overlapping buttons in certificate status panel.

--
Endi S. Dewata
From 47a8cb33366a1eb566e56a33bd74aa57785b934d Mon Sep 17 00:00:00 2001
From: Endi S. Dewata <edew...@redhat.com>
Date: Thu, 13 Jan 2011 17:20:12 +0700
Subject: [PATCH] Certificate and Kerberos key status adjustments.

The OTP field has been moved into a separate row to avoid line
wrapping. The line height inside tables has been increased to
avoid overlapping buttons in certificate status panel.
---
 install/static/certificate.js |    4 ++++
 install/static/host.js        |   10 +++++++++-
 install/static/ipa.css        |    8 ++++++++
 3 files changed, 21 insertions(+), 1 deletions(-)

diff --git a/install/static/certificate.js b/install/static/certificate.js
index 5d4313ef16ae2afe29d721a0a27f0dc52288eac7..75ac7e476be05a06dad7216ee8ebe26be1f63390 100755
--- a/install/static/certificate.js
+++ b/install/static/certificate.js
@@ -432,12 +432,16 @@ function certificate_status_widget(spec) {
             'value': 'Get'
         }).appendTo(td);
 
+        td.append(' ');
+
         if (!that.is_selfsign()) {
             $('<input/>', {
                 'type': 'button',
                 'name': 'revoke',
                 'value': 'Revoke'
             }).appendTo(td);
+
+            td.append(' ');
         }
 
         $('<input/>', {
diff --git a/install/static/host.js b/install/static/host.js
index be96d79a962035153fd7d3dafb1b7c32cdcc013b..8cd8d0e9d27c93d536dc9eef6726b64426cea8db 100644
--- a/install/static/host.js
+++ b/install/static/host.js
@@ -255,12 +255,20 @@ function host_provisioning_status_widget(spec) {
 
         td = $('<td/>').appendTo(tr);
 
+        tr = $('<tr/>').appendTo(table);
+
+        td = $('<td/>').appendTo(tr);
+
+        td = $('<td/>').appendTo(tr);
+
         $('<input/>', {
             'type': 'text',
             'name': 'otp',
-            'size': 10
+            'class': 'otp'
         }).appendTo(td);
 
+        td = $('<td/>').appendTo(tr);
+
         $('<input/>', {
             'type': 'button',
             'name': 'enroll',
diff --git a/install/static/ipa.css b/install/static/ipa.css
index 38b68dd0572ab47ee4d99f71ab7010b0d0de943f..76c34d3636e0275af76e5035ccc83fd8cf76ea52 100644
--- a/install/static/ipa.css
+++ b/install/static/ipa.css
@@ -22,6 +22,10 @@ body{
     margin: 0;
 }
 
+table {
+    line-height: 2;
+}
+
 .center-container {
         margin-left: auto;
         margin-right: auto;
@@ -215,6 +219,10 @@ dl.entryattrs input {
     min-width: 27.5em;
 }
 
+dl.entryattrs input.otp {
+    min-width: 15em;
+    width: 98%;
+}
 
 span.attrhint {
     font-size: 8pt;
-- 
1.6.6.1

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

Reply via email to