ACK Looks good.

----- Original Message -----
> On 01/25/2011 09:16 PM, Adam Young wrote:
> 
> Kyle noticed that the DNS page was off. This fixes it.
> _______________________________________________
> Freeipa-devel mailing list Freeipa-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel Fixes the title.
> It had to be set after the entity was assigned.
> 
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel
From 7873786188e39b6d91645dc66c733477188e6fdf Mon Sep 17 00:00:00 2001
From: Adam Young <ayo...@redhat.com>
Date: Tue, 25 Jan 2011 21:15:05 -0500
Subject: [PATCH] dns container

the dns records page was adding controls to the wrong tag.  This pushes everything down one level, fixing the formatting.
---
 install/ui/policy.js |   58 ++++++++++++++++++++++++++++---------------------
 1 files changed, 33 insertions(+), 25 deletions(-)

diff --git a/install/ui/policy.js b/install/ui/policy.js
index 208b7c9ec55f82166d876fea250c93a37bf85880..fedf40726ac11d05f859cc09045ebe59a783fb94 100644
--- a/install/ui/policy.js
+++ b/install/ui/policy.js
@@ -276,30 +276,21 @@ IPA.records_facet = function (spec){
     };
 
     function create(container) {
+
+        container.attr('title', that.entity_name);
+
+        $('<h1/>',{
+        }).append(IPA.create_network_spinner()).
+            appendTo(container);
+
         var details = $('<div/>', {
             'class': 'content'
         }).appendTo(container);
-    }
-
-    function setup(container){
-
-        that.facet_setup(container);
-
-        that.pkey = $.bbq.getState(that.entity_name + '-pkey', true) || '';
-        that.record = $.bbq.getState(that.entity_name + '-record', true) || '';
-
-        that.container.attr('title', that.entity_name);
-
-        var h2 = $('<h2></h2>',{
-            text: "Records for DNS Zone:" + that.pkey
-        }).appendTo(that.container);
-
 
         var div = $('<div class="search-controls"></div>').
-            appendTo(that.container);
+            appendTo(details);
 
         var control_span =$('<span class="record-filter"></span>').appendTo(div);
-
         control_span.append('Resource');
         control_span.append($('<input />',{
             type: "text",
@@ -311,13 +302,6 @@ IPA.records_facet = function (spec){
 
         create_type_select('dns-record-type-filter',true).
             appendTo(control_span);
-        //commented out until data is searchable
-        //control_span.append('Data');
-        //control_span.append($('<input />',{
-        //    type: "text",
-        //    id: 'dns-record-data-filter',
-        //    name: 'search-' + obj_name + '-filter'
-        //}));
 
 
         IPA.button({
@@ -349,7 +333,7 @@ IPA.records_facet = function (spec){
 
         var records_results = $('<div/>', {
             'class': 'records-results'
-        }).appendTo(that.container);
+        }).appendTo(details);
 
         var records_table = $('<table/>', {
             'class': 'search-table'
@@ -373,6 +357,30 @@ IPA.records_facet = function (spec){
          tr.append($('<th>Record Type</th>'));
         tr.append($('<th>Data</th>'));
 
+    }
+
+    function setup(container){
+
+        that.facet_setup(container);
+
+        that.pkey = $.bbq.getState(that.entity_name + '-pkey', true) || '';
+        that.record = $.bbq.getState(that.entity_name + '-record', true) || '';
+
+
+        $('h1',container).
+            html("<span id='headerpkey' />Records for DNS Zone:" + that.pkey);
+
+
+        //commented out until data is searchable
+        //control_span.append('Data');
+        //control_span.append($('<input />',{
+        //    type: "text",
+        //    id: 'dns-record-data-filter',
+        //    name: 'search-' + obj_name + '-filter'
+        //}));
+
+
+
         refresh();
     }
 
-- 
1.7.3.5

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

Reply via email to