COmanage Developers List

Author: marie
Date: 2012-09-28 08:38:59 -0400 (Fri, 28 Sep 2012)
New Revision: 374

Modified:
   registry/trunk/app/View/CoPersonRoles/fields.inc
   registry/trunk/app/webroot/css/comanage.css
Log:
co360 css changes and changes to view for copersonroles to use tabs

Modified: registry/trunk/app/View/CoPersonRoles/fields.inc
===================================================================
--- registry/trunk/app/View/CoPersonRoles/fields.inc    2012-09-28 12:37:03 
UTC (rev 373)
+++ registry/trunk/app/View/CoPersonRoles/fields.inc    2012-09-28 12:38:59 
UTC (rev 374)
@@ -45,10 +45,6 @@
 
   if($e)
   {
-    echo $this->Html->link(_txt('op.cancel'),
-                            array('controller' => 'co_people', 'action' => 
'edit', $co_people[0]['CoPerson']['id'], 'co' => $cur_co['Co']['id']),
-                            array('class' => 'cancelbutton'));
-    
     // This is for beforeFilter
     echo $this->Form->hidden('Co.id', array('default' => 
$cur_co['Co']['id'])). "\n";
     // And this is to link to the co person
@@ -68,240 +64,296 @@
   // Line number, for rendering
   $l = 1;
 ?>
-<table id="<?php echo $this->action; ?>_co_person_role" class="ui-widget">
-  <tbody>
-    <?php if(!empty($permissions['cous'])): ?>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo Sanitize::html($cur_co['Co']['name']) . " " . 
_txt('fd.cou'); ?><font class="required">*</font>
-      </td>
-      <td>
-        <?php
-          $attrs['value'] = 
(isset($co_person_roles[0]['CoPersonRole']['cou_id'])
-                             ? $co_person_roles[0]['CoPersonRole']['cou_id']
-                             : 0);
-          $attrs['empty'] = false;
-          
-          if($e && !$es) {
-            print $this->Form->select('cou_id',
-                                      $permissions['cous'],
-                                      $attrs);
+
+<script>
+$(function() {
+  // Turn on Tabs
+  var $tabs = $( "#tabs" ).tabs();
+  
+  // If returning to this page via redirect, open last used tab
+  <?php if(isset($this->request->params['named']['tab'])): ?>
+    var selectedtab = "<?php print($this->request->params['named']['tab']); 
?>";
+    $tabs.tabs('select', '#tabs-' + selectedtab );
+  <?php endif; ?>
+});
+</script>
+<div id="<?php print $this->action; ?>_co_person" style=" float:left; 
height:auto;">
+  <div id="tabs" style="min-height:350px; width:550px">
+    <ul>
+      <li>
+        <a href="#tabs-role">
+          <?php print _txt('fd.attrs.copr'); ?>
+        </a>
+      </li>
+      <?php if($this->action != "add"): ?>
+        <li>
+          <a href="#tabs-phone">
+            <?php print _txt('fd.telephone_number.number'); ?>
+          </a>
+        </li>
+        <li>
+          <a href="#tabs-address">
+            <?php print _txt('fd.address'); ?>
+          </a>
+        </li>
+      <?php endif; ?>
+    </ul>
+    <div id="tabs-role">
+      <table id="<?php echo $this->action; ?>_co_person_role" 
class="ui-widget">
+        <tbody>
+          <?php if(!empty($permissions['cous'])): ?>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo Sanitize::html($cur_co['Co']['name']) . " " . 
_txt('fd.cou'); ?><font class="required">*</font>
+            </td>
+            <td>
+              <?php
+                $attrs['value'] = 
(isset($co_person_roles[0]['CoPersonRole']['cou_id'])
+                                   ? 
$co_person_roles[0]['CoPersonRole']['cou_id']
+                                   : 0);
+                $attrs['empty'] = false;
+                
+                if($e && !$es) {
+                  print $this->Form->select('cou_id',
+                                            $permissions['cous'],
+                                            $attrs);
+                  
+                  if($this->Form->isFieldError('cou_id')) {
+                    print $this->Form->error('cou_id');
+                  }
+                } else {
+                  print Sanitize::html($co_person_roles[0]['Cou']['name']);
+                }
+              ?>
+            </td>
+          <?php endif; ?>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo Sanitize::html($cur_co['Co']['name']) . " " . 
_txt('fd.affiliation'); ?><font class="required">*</font>
+            </td>
+            <td>
+              <?php
+                global $cm_lang, $cm_texts;
+                $attrs['value'] = 
(isset($co_person_roles[0]['CoPersonRole']['affiliation'])
+                                   ? 
$co_person_roles[0]['CoPersonRole']['affiliation']
+                                   : "M");
+                $attrs['empty'] = false;
+                
+                if($e && !$es) {
+                  print $this->Form->select('affiliation',
+                                            $cm_texts[ $cm_lang 
]['en.affil'],
+                                            $attrs);
+                  
+                  if($this->Form->isFieldError('affiliation')) {
+                    print $this->Form->error('affiliation');
+                  }
+                } else {
+                  print $cm_texts[ $cm_lang ]['en.affil'][ 
$co_person_roles[0]['CoPersonRole']['affiliation']];
+                }
+              ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo $cur_co['Co']['name'] . " " . _txt('fd.title'); ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es) ? $this->Form->input('title', 
array('default' =>
+                                                                           
isset($co_person_roles[0]['CoPersonRole']['title'])
+                                                                           ? 
$co_person_roles[0]['CoPersonRole']['title']
+                                                                           : 
""))
+                                       : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['title'])); ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo _txt('fd.o'); ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es) ? $this->Form->input('o', 
array('default' => $cur_co['Co']['name']))
+                                       : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['o'])); ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo _txt('fd.ou'); ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es) ? $this->Form->input('ou') : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['ou'])); ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo _txt('fd.sponsor') . " ";
+                    echo _txt('fd.sponsor.desc');
+              ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es) ? 
$this->Form->select('sponsor_co_person_id', $sponsors) : 
Sanitize::html($sponsors[ 
$co_person_roles[0]['CoPersonRole']['sponsor_co_person_id'] ])); 
+              ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo _txt('fd.valid_from'); if($e && !$es) echo " " . 
_txt('fd.valid_from.desc'); ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es)
+                          ? $this->Form->text('valid_from', array('class' => 
'datepicker-f'))
+                          : 
(($co_person_roles[0]['CoPersonRole']['valid_from'] > 0) ? strftime("%F", 
strtotime($co_person_roles[0]['CoPersonRole']['valid_from'])) : "")); ?>
+            </td>
+          </tr>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo _txt('fd.valid_through'); if($e && !$es) echo " " . 
_txt('fd.valid_through.desc'); ?>
+            </td>
+            <td>
+              <?php echo (($e && !$es)
+                          ? $this->Form->text('valid_through', array('class' 
=> 'datepicker-u'))
+                          : 
(($co_person_roles[0]['CoPersonRole']['valid_through'] > 0) ? strftime("%F", 
strtotime($co_person_roles[0]['CoPersonRole']['valid_through'])) : "")); ?>
+            </td>
+          </tr>
+          <?php
+            // Check for extended attributes and render
             
-            if($this->Form->isFieldError('cou_id')) {
-              print $this->Form->error('cou_id');
-            }
-          } else {
-            print Sanitize::html($co_person_roles[0]['Cou']['name']);
-          }
-        ?>
-      </td>
-    <?php endif; ?>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo Sanitize::html($cur_co['Co']['name']) . " " . 
_txt('fd.affiliation'); ?><font class="required">*</font>
-      </td>
-      <td>
-        <?php
-          global $cm_lang, $cm_texts;
-          $attrs['value'] = 
(isset($co_person_roles[0]['CoPersonRole']['affiliation'])
-                             ? 
$co_person_roles[0]['CoPersonRole']['affiliation']
-                             : "M");
-          $attrs['empty'] = false;
-          
-          if($e && !$es) {
-            print $this->Form->select('affiliation',
-                                      $cm_texts[ $cm_lang ]['en.affil'],
-                                      $attrs);
+            $cl = 'Co' . $cur_co['Co']['id'] . 'PersonExtendedAttribute';
             
-            if($this->Form->isFieldError('affiliation')) {
-              print $this->Form->error('affiliation');
+            if(!empty($cur_co['CoExtendedAttribute']))
+            {
+              echo $this->Form->hidden($cl . '.id');
+              
+              foreach($cur_co['CoExtendedAttribute'] as $c):
+          ?>
+          <tr class="line<?php echo ($l % 2); $l++; ?>">
+            <td>
+              <?php echo $c['display_name']; ?>
+            </td>
+            <td>
+              <?php
+                $attr = strtolower($c['name']);  
+                
+                if($c['type'] == 'TIMESTAMP')
+                {
+                  echo ($e && !$es ? $this->Form->text($cl.'.'.$attr, 
array('class' => 'datepicker'))
+                                   : (($co_person_roles[0][$cl][$attr] > 0) 
? strftime("%F", strtotime($co_person_roles[0][$cl][$attr])) : ""));
+                }
+                else
+                {
+                  echo ($e && !$es ? $this->Form->input($cl.'.'.$attr,
+                                                        
isset($co_person_roles[0][$cl][$attr]) ? array('default' => 
$co_person_roles[0][$cl][$attr]) : array())
+                                   : 
Sanitize::html($co_person_roles[0][$cl][$attr]));
+                }
+              ?>
+            </td>
+          </tr>
+          <?php
+              endforeach;
             }
-          } else {
-            print $cm_texts[ $cm_lang ]['en.affil'][ 
$co_person_roles[0]['CoPersonRole']['affiliation']];
-          }
-        ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo $cur_co['Co']['name'] . " " . _txt('fd.title'); ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es) ? $this->Form->input('title', 
array('default' =>
-                                                                     
isset($co_person_roles[0]['CoPersonRole']['title'])
-                                                                     ? 
$co_person_roles[0]['CoPersonRole']['title']
-                                                                     : ""))
-                                 : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['title'])); ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.o'); ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es) ? $this->Form->input('o', array('default' 
=> $cur_co['Co']['name']))
-                                 : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['o'])); ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.ou'); ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es) ? $this->Form->input('ou') : 
Sanitize::html($co_person_roles[0]['CoPersonRole']['ou'])); ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.sponsor') . " ";
-              echo _txt('fd.sponsor.desc');
-        ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es) ? 
$this->Form->select('sponsor_co_person_id', $sponsors) : 
Sanitize::html($sponsors[ 
$co_person_roles[0]['CoPersonRole']['sponsor_co_person_id'] ])); 
-        ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.valid_from'); if($e && !$es) echo " " . 
_txt('fd.valid_from.desc'); ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es)
-                    ? $this->Form->text('valid_from', array('class' => 
'datepicker-f'))
-                    : (($co_person_roles[0]['CoPersonRole']['valid_from'] > 
0) ? strftime("%F", 
strtotime($co_person_roles[0]['CoPersonRole']['valid_from'])) : "")); ?>
-      </td>
-    </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.valid_through'); if($e && !$es) echo " " . 
_txt('fd.valid_through.desc'); ?>
-      </td>
-      <td>
-        <?php echo (($e && !$es)
-                    ? $this->Form->text('valid_through', array('class' => 
'datepicker-u'))
-                    : (($co_person_roles[0]['CoPersonRole']['valid_through'] 
> 0) ? strftime("%F", 
strtotime($co_person_roles[0]['CoPersonRole']['valid_through'])) : "")); ?>
-      </td>
-    </tr>
-    <?php if($this->action != "add"): ?>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.telephone_number.number'); ?>
-      </td>
-      <td>
-        <?php
-          if($e)
+          ?>
+          <tr>
+            <td>
+              <i><font class="required"><?php echo _txt('fd.req'); 
?></font></i><br />
+            </td>
+            <td>
+              <?php
+                if($e)
+                {
+                  print '<div style="float:left; margin:0 10px 0 0;">';
+                    print $this->Form->submit($submit_label);
+                  print '</div>';
+
+                  print $this->Html->link(_txt('op.cancel'),
+                                          array('controller' => 'co_people', 
'action' => 'edit', $co_people[0]['CoPerson']['id'], 'co' => 
$cur_co['Co']['id']),
+                                          array('class' => 'cancelbutton'));
+                }
+              ?>
+            </td>
+          </tr>
+        </tbody>
+      </table>
+
+  </div>
+  <?php if($this->action != "add"): ?>
+
+    <div id="tabs-phone" class="additionalinfo">
+      <?php
+        if($e)
+        {
+          if(isset($co_person_roles[0]['TelephoneNumber']))
           {
-            if(isset($co_person_roles[0]['TelephoneNumber']))
+            foreach($co_person_roles[0]['TelephoneNumber'] as $t)
             {
-              foreach($co_person_roles[0]['TelephoneNumber'] as $t)
-              {
-                // XXX we already checked for $permissions['edit'], but not 
['delete']... should we?
-                echo '<a class="deletebutton" title="' . _txt('op.delete') . 
'" onclick="javascript:js_confirm_delete(\'' . 
_jtxt(Sanitize::html($t['number'])) . '\', \'' . 
$this->Html->url(array('controller' => 'telephone_numbers', 'action' => 
'delete', $t['id'], 'co' => $cur_co['Co']['id'])) . '\')";>' . 
_txt('op.delete') . '</a>' . "\n";
-                echo $this->Html->link(_txt('op.edit'),
-                                        array('controller' => 
'telephone_numbers', 'action' => 'edit', $t['id'], 'co' => 
$cur_co['Co']['id']),
-                                        array('class' => 'editbutton')) . 
"\n";
-                echo $this->Html->link($t['number'], array('controller' => 
'telephone_numbers', 'action' => 'edit', $t['id'], 'co' => 
$cur_co['Co']['id']));
-                echo " (" . _txt('en.contact', null, $t['type']) . ")<br 
/>\n";
-              }
+              print '<div>';
+                print '<div>';
+                             echo $this->Html->link($t['number'], 
array('controller' => 'telephone_numbers', 'action' => 'edit', $t['id'], 'co' 
=> $cur_co['Co']['id']));
+              echo " (" . _txt('en.contact', null, $t['type']) . ")<br />\n";
+              print '</div>';
+                print '<div>';
+                  // XXX we already checked for $permissions['edit'], but 
not ['delete']... should we?
+                  echo '<a class="deletebutton" title="' . _txt('op.delete') 
. '" onclick="javascript:js_confirm_delete(\'' . 
_jtxt(Sanitize::html($t['number'])) . '\', \'' . 
$this->Html->url(array('controller' => 'telephone_numbers', 'action' => 
'delete', $t['id'], 'co' => $cur_co['Co']['id'])) . '\')";>' . 
_txt('op.delete') . '</a>' . "\n";
+                  echo $this->Html->link(_txt('op.edit'),
+                                         array('controller' => 
'telephone_numbers', 'action' => 'edit', $t['id'], 'co' => 
$cur_co['Co']['id']),
+                                         array('class' => 'editbutton')) . 
"\n";
+                print '</div>';
+              print '</div>';
             }
-            
-            echo $this->Html->link(_txt('op.add'),
-                                    array('controller' => 
'telephone_numbers', 'action' => 'add', 'copersonroleid' => 
$co_person_roles[0]['CoPersonRole']['id'], 'co' => $cur_co['Co']['id']),
-                                    array('class' => 'addbutton'));
           }
-          else
-          {
-            if(isset($co_person_roles[0]['TelephoneNumber']))
-              foreach($co_person_roles[0]['TelephoneNumber'] as $t)
-                echo Sanitize::html($t['number']) . " (" . 
_txt('en.contact', null, $t['type']) . ")<br />\n";
-          }
-        ?>
-      </td>
+          
+          echo $this->Html->link(_txt('op.add'),
+                                  array('controller' => 'telephone_numbers', 
'action' => 'add', 'copersonroleid' => 
$co_person_roles[0]['CoPersonRole']['id'], 'co' => $cur_co['Co']['id']),
+                                  array('class' => 'addbutton'));
+        }
+        else
+        {
+          if(isset($co_person_roles[0]['TelephoneNumber']))
+            foreach($co_person_roles[0]['TelephoneNumber'] as $t)
+              echo Sanitize::html($t['number']) . " (" . _txt('en.contact', 
null, $t['type']) . ")<br />\n";
+        }
+      ?>
     </tr>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo _txt('fd.address'); ?>
-      </td>
-      <td>
-        <?php
-          if($e)
+
+
+    </div>
+
+    <div id="tabs-address" class="additionalinfo">
+      <?php
+        if($e)
+        {
+          if(isset($co_person_roles[0]['Address']))
           {
-            if(isset($co_person_roles[0]['Address']))
+            foreach($co_person_roles[0]['Address'] as $addr)
             {
-              foreach($co_person_roles[0]['Address'] as $addr)
-              {
-                // XXX we already checked for $permissions['edit'], but not 
['delete']... should we?
-                echo '<a class="deletebutton" title="' . _txt('op.delete') . 
'" onclick="javascript:js_confirm_delete(\'' . 
_jtxt(Sanitize::html($addr['line1'])) . '\', \'' . 
$this->Html->url(array('controller' => 'addresses', 'action' => 'delete', 
$addr['id'], 'co' => $cur_co['Co']['id'])) . '\')";>' . _txt('op.delete') . 
'</a>' . "\n";
-                echo $this->Html->link(_txt('op.edit'),
-                                        array('controller' => 'addresses', 
'action' => 'edit', $addr['id'], 'co' => $cur_co['Co']['id']),
-                                        array('class' => 'editbutton')) . 
"\n";
-                echo $this->Html->link($addr['line1'], array('controller' => 
'addresses', 'action' => 'edit', $addr['id'], 'co' => $cur_co['Co']['id']));
-                echo " (" . _txt('en.contact', null, $addr['type']) . ")<br 
/>\n";
-              }
+              print '<div>';
+                print '<div>';
+                  echo $this->Html->link($addr['line1'], array('controller' 
=> 'addresses', 'action' => 'edit', $addr['id'], 'co' => 
$cur_co['Co']['id']));
+                  echo " (" . _txt('en.contact', null, $addr['type']) . 
")<br />\n";
+                print '</div>';
+                print '<div>';
+              // XXX we already checked for $permissions['edit'], but not 
['delete']... should we?
+              echo '<a class="deletebutton" title="' . _txt('op.delete') . 
'" onclick="javascript:js_confirm_delete(\'' . 
_jtxt(Sanitize::html($addr['line1'])) . '\', \'' . 
$this->Html->url(array('controller' => 'addresses', 'action' => 'delete', 
$addr['id'], 'co' => $cur_co['Co']['id'])) . '\')";>' . _txt('op.delete') . 
'</a>' . "\n";
+              echo $this->Html->link(_txt('op.edit'),
+                                      array('controller' => 'addresses', 
'action' => 'edit', $addr['id'], 'co' => $cur_co['Co']['id']),
+                                      array('class' => 'editbutton')) . "\n";
+                print '</div>';
+              print '</div>';
+
             }
-                
-            echo $this->Html->link(_txt('op.add'),
-                                    array('controller' => 'addresses', 
'action' => 'add', 'copersonroleid' => 
$co_person_roles[0]['CoPersonRole']['id'], 'co' => $cur_co['Co']['id']),
-                                    array('class' => 'addbutton'));
           }
-          else
-          {
-            if(isset($co_person_roles[0]['Address']))
-              foreach($co_person_roles[0]['Address'] as $addr)
-                echo Sanitize::html($addr['line1']) . " (" . 
_txt('en.contact', null, $addr['type']) . ")<br />\n";
-          }
-        ?>
-      </td>
-    </tr>
-    <?php endif; // add ?>
-    <?php
-      // Check for extended attributes and render
+              
+          echo $this->Html->link(_txt('op.add'),
+                                  array('controller' => 'addresses', 
'action' => 'add', 'copersonroleid' => 
$co_person_roles[0]['CoPersonRole']['id'], 'co' => $cur_co['Co']['id']),
+                                  array('class' => 'addbutton'));
+        }
+        else
+        {
+          if(isset($co_person_roles[0]['Address']))
+            foreach($co_person_roles[0]['Address'] as $addr)
+              echo Sanitize::html($addr['line1']) . " (" . 
_txt('en.contact', null, $addr['type']) . ")<br />\n";
+        }
+      ?>
       
-      $cl = 'Co' . $cur_co['Co']['id'] . 'PersonExtendedAttribute';
-      
-      if(!empty($cur_co['CoExtendedAttribute']))
-      {
-        echo $this->Form->hidden($cl . '.id');
-        
-        foreach($cur_co['CoExtendedAttribute'] as $c):
-    ?>
-    <tr class="line<?php echo ($l % 2); $l++; ?>">
-      <td>
-        <?php echo $c['display_name']; ?>
-      </td>
-      <td>
-        <?php
-          $attr = strtolower($c['name']);  
-          
-          if($c['type'] == 'TIMESTAMP')
-          {
-            echo ($e && !$es ? $this->Form->text($cl.'.'.$attr, 
array('class' => 'datepicker'))
-                             : (($co_person_roles[0][$cl][$attr] > 0) ? 
strftime("%F", strtotime($co_person_roles[0][$cl][$attr])) : ""));
-          }
-          else
-          {
-            echo ($e && !$es ? $this->Form->input($cl.'.'.$attr,
-                                                  
isset($co_person_roles[0][$cl][$attr]) ? array('default' => 
$co_person_roles[0][$cl][$attr]) : array())
-                             : 
Sanitize::html($co_person_roles[0][$cl][$attr]));
-          }
-        ?>
-      </td>
-    </tr>
-    <?php
-        endforeach;
-      }
-    ?>
-    <tr>
-      <td>
-        <i><font class="required"><?php echo _txt('fd.req'); 
?></font></i><br />
-      </td>
-      <td>
-        <?php
-          if($e)
-            echo $this->Form->submit($submit_label);
-        ?>
-      </td>
-    </tr>
-  </tbody>
-</table>
+    </div>
+  <?php endif; // add ?>
+
+</div>

Modified: registry/trunk/app/webroot/css/comanage.css
===================================================================
--- registry/trunk/app/webroot/css/comanage.css 2012-09-28 12:37:03 UTC (rev 
373)
+++ registry/trunk/app/webroot/css/comanage.css 2012-09-28 12:38:59 UTC (rev 
374)
@@ -196,14 +196,21 @@
         padding: 5px;
         margin: 0 0 5px 0;
         overflow:hidden;
+        border-radius:10px;
       }
       .additionalinfo>div:last-child{
         margin:0;
       }
-      .additionalinfo>div div{
+      .additionalinfo>div>div:first-child{
         float: left;
         padding: 7px 0 0 10px;
       }
+      .additionalinfo>div>div:last-child{
+        border-left:1px solid white;
+        border-left: 1px solid white;
+        float: right;
+        padding-left: 5px;
+      }
 
       .additionalinfo div a{
         float:left;
@@ -213,7 +220,9 @@
       }
 
       #OrgIdentityEditForm .addbutton,
-      #view_org_identity .addbutton{
+      #view_org_identity .addbutton,
+      #CoPersonEditForm .addbutton,
+      #CoPersonRoleEditForm .addbutton{
         float:right;
       }