From 0683ed4092b548dbe97dbf689dc9329c915a0a27 Mon Sep 17 00:00:00 2001
From: Jerome Nadaud <jerome.nadaud@prestashop.com>
Date: Wed, 18 Sep 2013 15:25:58 +0200
Subject: [PATCH] // Some list design fix

---
 .../default/template/helpers/list/list_content.tpl   | 12 ++++++------
 .../default/template/helpers/list/list_header.tpl    | 11 ++++++-----
 controllers/admin/AdminProfilesController.php        |  3 ++-
 controllers/admin/AdminTabsController.php            |  7 +++++--
 4 files changed, 19 insertions(+), 14 deletions(-)

diff --git a/admin-dev/themes/default/template/helpers/list/list_content.tpl b/admin-dev/themes/default/template/helpers/list/list_content.tpl
index 0c25c1f19..8c623ccfc 100644
--- a/admin-dev/themes/default/template/helpers/list/list_content.tpl
+++ b/admin-dev/themes/default/template/helpers/list/list_content.tpl
@@ -30,17 +30,17 @@
 	class="{if isset($tr.class)} {$tr.class}{/if}"
 	{if isset($tr.color) && $color_on_bg}style="background-color: {$tr.color}"{/if}
 	>
+		{if {$has_bulk_actions}}
 		<td class="center">
-			{if {$has_bulk_actions}}
-				{if isset($list_skip_actions.delete)}
-					{if !in_array($tr.$identifier, $list_skip_actions.delete)}
-						<input type="checkbox" name="{$table}Box[]" value="{$tr.$identifier}" class="noborder" />
-					{/if}
-				{else}
+			{if isset($list_skip_actions.delete)}
+				{if !in_array($tr.$identifier, $list_skip_actions.delete)}
 					<input type="checkbox" name="{$table}Box[]" value="{$tr.$identifier}" class="noborder" />
 				{/if}
+			{else}
+				<input type="checkbox" name="{$table}Box[]" value="{$tr.$identifier}" class="noborder" />
 			{/if}
 		</td>
+		{/if}
 		{foreach $fields_display AS $key => $params}
 			{block name="open_td"}
 				<td
diff --git a/admin-dev/themes/default/template/helpers/list/list_header.tpl b/admin-dev/themes/default/template/helpers/list/list_header.tpl
index be39232aa..b1fdd575e 100644
--- a/admin-dev/themes/default/template/helpers/list/list_header.tpl
+++ b/admin-dev/themes/default/template/helpers/list/list_header.tpl
@@ -254,8 +254,10 @@
 			>
 			<thead>
 				<tr class="nodrag nodrop">
-					<th class="center">
+					{if $has_bulk_actions}
+					<th class="center fixed-width-xs">
 					</th>
+					{/if}
 					{foreach $fields_display AS $key => $params}
 					<th class="{if isset($params.class)}{$params.class}{/if}{if isset($params.align)} {$params.align}{/if}">
 
@@ -308,12 +310,11 @@
 				</tr>
 			{if !$simple_header && $show_filters}
 				<tr class="nodrag nodrop filter {if $row_hover}row_hover{/if}">
+					{if $has_bulk_actions}
 					<td class="center">
-						{if $has_bulk_actions}
-							--
-						{/if}
+						--
 					</td>
-
+					{/if}
 					{* Filters (input, select, date or bool) *}
 					{foreach $fields_display AS $key => $params}
 					<td {if isset($params.align)} class="{$params.align}" {/if}>
diff --git a/controllers/admin/AdminProfilesController.php b/controllers/admin/AdminProfilesController.php
index 1ae906577..fae0ec52d 100644
--- a/controllers/admin/AdminProfilesController.php
+++ b/controllers/admin/AdminProfilesController.php
@@ -46,7 +46,8 @@ class AdminProfilesControllerCore extends AdminController
 		$this->fields_list = array(
 			'id_profile' => array(
 						'title' => $this->l('ID'), 
-						'align' => 'center'
+						'align' => 'center',
+						'class' => 'fixed-width-xs'
 						),
 			'name' => array('title' => $this->l('Name'))
 			);
diff --git a/controllers/admin/AdminTabsController.php b/controllers/admin/AdminTabsController.php
index 43e211571..28a04406d 100644
--- a/controllers/admin/AdminTabsController.php
+++ b/controllers/admin/AdminTabsController.php
@@ -46,7 +46,9 @@ class AdminTabsControllerCore extends AdminController
 
 		$this->fields_list = array(
 			'id_tab' => array(
-				'title' => $this->l('ID')
+				'title' => $this->l('ID'),
+				'align' => 'center',
+				'class' => 'fixed-width-xs'
 			),
 			'name' => array(
 				'title' => $this->l('Name')
@@ -71,7 +73,8 @@ class AdminTabsControllerCore extends AdminController
 			'position' => array(
 				'title' => $this->l('Position'),
 				'filter_key' => 'a!position',
-				'position' => 'position'
+				'position' => 'position',
+				'class' => 'fixed-width-md'
 			)
 		);