From 348d734c59432a740b52924ac18ff47b30dad54e Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Thu, 3 Feb 2011 21:42:50 -0600
Subject: Restructuring details page.

Previously the IPA.details_list_section can only be used with widgets
that generates <dd> tag because it uses the following structure:

<dl>
  <dt>Telephone Number:</dt>
  <span name="teleponenumber">
    <dd>111-1111</dd>
    <dd>222-2222</dd>
  </span>
</dl>

The <dd> tag was previously used to handle multi-valued attributes.
Since multi-valued attributes are now handled by the recently added
IPA.multivalued_text_widget, the structure can be changed as follows:

<dl>
  <dt>Telephone Number:</dt>
  <dd>
    <span name="telephonenumber">
      <div>111-1111</div>
      <div>222-2222</div>
    </span>
  </dd>
</dl>

This allows IPA.details_list_section to be used with any widgets
without requiring the <dd> tag.
---
 install/ui/host.js | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

(limited to 'install/ui/host.js')

diff --git a/install/ui/host.js b/install/ui/host.js
index 453b61788..cc60fb511 100644
--- a/install/ui/host.js
+++ b/install/ui/host.js
@@ -140,21 +140,21 @@ IPA.host_details_facet = function (spec) {
         that.add_section(section);
 
         //TODO: use i18n labels
-        section.create_field({
+        section.text({
             name: 'fqdn',
             label: 'Fully Qualified Host Name'
         });
 
-        section.create_field({'name': 'krbprincipalname'});
+        section.text({'name': 'krbprincipalname'});
 
         //TODO: add this to the host plugin
         //TODO: use i18n labels
-        section.create_field({
+        section.text({
             'name': 'serverhostname',
             'label': 'Host Name'
         });
 
-        section.create_field({'name': 'description'});
+        section.text({'name': 'description'});
 
         //TODO: use i18n labels
         section = IPA.details_list_section({
@@ -225,13 +225,9 @@ function host_provisioning_status_widget(spec) {
 
         that.widget_create(container);
 
-        var dd = $('<dd/>', {
-            'class': 'first'
-        }).appendTo(container);
-
         var div = $('<div/>', {
             name: 'kerberos-key-valid'
-        }).appendTo(dd);
+        }).appendTo(container);
 
         $('<img/>', {
             src: 'check.png',
@@ -255,7 +251,7 @@ function host_provisioning_status_widget(spec) {
 
         div = $('<div/>', {
             name: 'kerberos-key-missing'
-        }).appendTo(dd);
+        }).appendTo(container);
 
         $('<img/>', {
             src: 'caution.png',
-- 
cgit