[-] BO : fixed bug #PSCSX-25 - carrier wizard summary design and missing infos

admin-dev/themes/default/template/controllers/carrier_wizard/summary.tpl

@@ -37,10 +37,11 @@
 
 <div class="defaultForm">
 	<div class="panel">
-		{l s='Carrier name:'} <strong id="summary_name"></strong>
+		<div class="panel-heading">{l s='Carrier name:'} <strong id="summary_name"></strong></div>
-		<div id="summary_meta_informations"></div>
+		<div class="panel-body">
-		<div id="summary_shipping_cost"></div>
+			<p id="summary_meta_informations"></p>
-		<div id="summary_range"></div>
+			<p id="summary_shipping_cost"></p>
+			<p id="summary_range"></p>
 			<div>
 			{l s='This carrier will be proposed for those delivery zones:'}
 				<ul id="summary_zones"></ul>
@@ -55,6 +56,7 @@
 				<ul id="summary_shops"></ul>
 			</div>
 			{/if}
+		</div>
+	</div>
 	{$active_form}
 </div>
-</div>

js/admin_carrier_wizard.js

@@ -175,14 +175,14 @@ function displaySummary()
 	$('#summary_zones').html('');
 	$('.input_zone').each(function(){
 		if ($(this).attr('checked'))
-			$('#summary_zones').html($('#summary_zones').html() + '<li><strong>' + $(this).closest().prev().text() + '</strong></li>');
+			$('#summary_zones').html($('#summary_zones').html() + '<li><strong>' + $(this).closest('tr').find('label').text() + '</strong></li>');
 	});
 	
 	// Group restrictions
 	$('#summary_groups').html('');
 	$('input[name$="groupBox[]"]').each(function(){
 		if ($(this).attr('checked'))
-			$('#summary_groups').html($('#summary_groups').html() + '<li><strong>' + $(this).closest().next().next().text() + '</strong></li>');
+			$('#summary_groups').html($('#summary_groups').html() + '<li><strong>' + $(this).closest('tr').find('td:eq(2)').text() + '</strong></li>');
 	});
 	
 	// shop restrictions