Small design changes in Contacts

2 files changed, 47 insertions, 41 deletions

diff --git a/apps/contacts/templates/part.details.php b/apps/contacts/templates/part.details.php
index 119c88486ac..5048349abc5 100644
--- a/apps/contacts/templates/part.details.php
+++ b/apps/contacts/templates/part.details.php
@@ -1,27 +1,33 @@
-<?php if(array_key_exists('PHOTO',$_['details'])): ?>
-	<img src="photo.php?id=<?php echo $_['id']; ?>">
-<?php endif; ?>
-
 <?php if(array_key_exists('FN',$_['details'])): ?>
-	<?php echo $this->inc('part.property', array('property' => $_['details']['FN'][0])); ?>
-<?php endif; ?>
+	<table>
+		<?php if(isset($_['details']['PHOTO'])): // Emails first ?>
+			<tr class="contacts_details_property">
+				<td class="contacts_details_left">&nbsp;</td>
+				<td class="contacts_details_right">
+					<img src="photo.php?id=<?php echo $_['id']; ?>">
+				</td>
+			</tr>
+		<?php endif; ?>
+		
+		<?php echo $this->inc('part.property', array('property' => $_['details']['FN'][0])); ?>
+		
+		<?php if(isset($_['details']['BDAY'])): // Emails first ?>
+			<?php echo $this->inc('part.property', array('property' => $_['details']['BDAY'][0])); ?>
+		<?php endif; ?>
 
-<?php if(isset($_['details']['BDAY'])): // Emails first ?>
-	<?php echo $this->inc('part.property', array('property' => $_['details']['BDAY'][0])); ?>
-<?php endif; ?>
+		<?php if(isset($_['details']['ORG'])): // Emails first ?>
+			<?php echo $this->inc('part.property', array('property' => $_['details']['ORG'][0])); ?>
+		<?php endif; ?>
 
-<?php if(isset($_['details']['ORG'])): // Emails first ?>
-	<?php echo $this->inc('part.property', array('property' => $_['details']['ORG'][0])); ?>
-<?php endif; ?>
-
-<?php foreach(array('EMAIL','TEL','ADR') as $type): ?>
-	<?php if(isset($_['details'][$type])): // Emails first ?>
-		<br>
-		<?php foreach($_['details'][$type] as $property): ?>
-			<?php echo $this->inc('part.property',array('property' => $property )); ?>
+		<?php foreach(array('EMAIL','TEL','ADR') as $type): ?>
+			<?php if(isset($_['details'][$type])): // Emails first ?>
+				<?php foreach($_['details'][$type] as $property): ?>
+					<?php echo $this->inc('part.property',array('property' => $property )); ?>
+				<?php endforeach; ?>
+			<?php endif; ?>
 		<?php endforeach; ?>
-	<?php endif; ?>
-<?php endforeach; ?>
+	</table>
+<?php endif; ?>
 
 <div id="contacts_cardoptions">
 	<a id="contacts_deletecard"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></a>
diff --git a/apps/contacts/templates/part.property.php b/apps/contacts/templates/part.property.php
index cdb03068128..9e2a5cabad4 100644
--- a/apps/contacts/templates/part.property.php
+++ b/apps/contacts/templates/part.property.php
@@ -1,49 +1,49 @@
-<div class="contacts_property" data-checksum="<?php echo $_['property']['checksum']; ?>">
+<tr class="contacts_details_property" data-checksum="<?php echo $_['property']['checksum']; ?>">
 	<?php if($_['property']['name'] == 'FN'): ?>
-		<div class="contacts_propertyname"><?php echo $l->t('Name'); ?></div>
-		<div class="contacts_propertyvalue">
+		<td class="contacts_details_left"><?php echo $l->t('Name'); ?></td>
+		<td class="contacts_details_right">
 			<?php echo $_['property']['value']; ?>
 			<span style="display:none;" data-use="edit"><img class="svg action" src="<?php echo image_path('', 'actions/rename.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php elseif($_['property']['name'] == 'BDAY'): ?>
-		<div class="contacts_propertyname"><?php echo $l->t('Birthday'); ?></div>
-		<div class="contacts_propertyvalue">
+		<td class="contacts_details_left"><?php echo $l->t('Birthday'); ?></td>
+		<td class="contacts_details_right">
 		<?php echo $l->l('date',new DateTime($_['property']['value'])); ?>
 			<span style="display:none;" data-use="delete"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php elseif($_['property']['name'] == 'ORG'): ?>
-		<div class="contacts_propertyname"><?php echo $l->t('Organisation'); ?></div>
-		<div class="contacts_propertyvalue">
+		<td class="contacts_details_left"><?php echo $l->t('Organisation'); ?></td>
+		<td class="contacts_details_right">
 			<?php echo $_['property']['value']; ?>
 			<span style="display:none;" data-use="edit"><img class="svg action" src="<?php echo image_path('', 'actions/rename.svg'); ?>" /></span>
 			<span style="display:none;" data-use="delete"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php elseif($_['property']['name'] == 'EMAIL'): ?>
-		<div class="contacts_propertyname"><?php echo $l->t('Email'); ?></div>
-		<div class="contacts_propertyvalue">
+		<td class="contacts_details_left"><?php echo $l->t('Email'); ?></td>
+		<td class="contacts_details_right">
 			<?php echo $_['property']['value']; ?>
 			<span style="display:none;" data-use="edit"><img class="svg action" src="<?php echo image_path('', 'actions/rename.svg'); ?>" /></span>
 			<span style="display:none;" data-use="delete"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php elseif($_['property']['name'] == 'TEL'): ?>
-		<div class="contacts_propertyname"><?php echo $l->t('Telephone'); ?></div>
-		<div class="contacts_propertyvalue">
+		<td class="contacts_details_left"><?php echo $l->t('Telephone'); ?></td>
+		<td class="contacts_details_right">
 			<?php echo $_['property']['value']; ?>
 			<?php if(isset($_['property']['parameters']['TYPE'])): ?>
 				(<?php echo $l->t('tel_'.strtolower($_['property']['parameters']['TYPE'])); ?>)
 			<?php endif; ?>
 			<span style="display:none;" data-use="edit"><img class="svg action" src="<?php echo image_path('', 'actions/rename.svg'); ?>" /></span>
 			<span style="display:none;" data-use="delete"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php elseif($_['property']['name'] == 'ADR'): ?>
-		<div class="contacts_propertyname">
+		<td class="contacts_details_left">
 			<?php echo $l->t('Address'); ?>
 			<?php if(isset($_['property']['parameters']['TYPE'])): ?>
 				<br>
 				(<?php echo $l->t('adr_'.strtolower($_['property']['parameters']['TYPE'])); ?>)
 			<?php endif; ?>
-		</div>
-		<div class="contacts_propertyvalue">
+		</td>
+		<td class="contacts_details_right">
 			<?php echo $l->t('PO Box'); ?> <?php echo $_['property']['value'][0]; ?><br>
 			<?php echo $l->t('Extended Address'); ?> <?php echo $_['property']['value'][1]; ?><br>
 			<?php echo $l->t('Street Name'); ?> <?php echo $_['property']['value'][2]; ?><br>
@@ -53,6 +53,6 @@
 			<?php echo $l->t('Country'); ?> <?php echo $_['property']['value'][6]; ?> 
 			<span style="display:none;" data-use="edit"><img class="svg action" src="<?php echo image_path('', 'actions/rename.svg'); ?>" /></span>
 			<span style="display:none;" data-use="delete"><img class="svg action" src="<?php echo image_path('', 'actions/delete.svg'); ?>" /></span>
-		</div>
+		</td>
 	<?php endif; ?>
-</div>
+</tr>