From 3dab31051b78b84e10ed055e41012ff78707835d Mon Sep 17 00:00:00 2001
From: "Sanjay Gohel (Open ERP)" <sgo@tinyerp.com>
Date: Mon, 16 Jul 2012 17:11:58 +0530
Subject: [PATCH] [IMP]hr,mrp,project:res_config view improvement as per
 suggetion

bzr revid: sgo@tinyerp.com-20120716114158-nef7622tzmnzcpxa
---
 addons/hr/res_config_view.xml             |  6 +++---
 addons/hr_payroll/res_config_view.xml     |  2 +-
 addons/hr_recruitment/res_config_view.xml | 19 ++++++++++---------
 addons/mrp/res_config_view.xml            | 10 ++++++----
 addons/project/res_config_view.xml        | 19 +++++++++----------
 5 files changed, 29 insertions(+), 27 deletions(-)

diff --git a/addons/hr/res_config_view.xml b/addons/hr/res_config_view.xml
index cf35654475d..91557c9fb88 100644
--- a/addons/hr/res_config_view.xml
+++ b/addons/hr/res_config_view.xml
@@ -20,8 +20,8 @@
                         or
                         <button string="Cancel" type="object" name="cancel" class="oe_link"/>
                     </header>
-                    <group>
-                        <group string="Additional Features" name="left_column">
+                    <group name="hr_grp">
+                        <group string="Additional Features" name="add_feature">
                             <field name="module_hr_expense"/>
                             <field name="module_hr_evaluation"/>
                             <field name="module_hr_holidays"/>
@@ -35,7 +35,7 @@
                                     attrs="{'invisible': [('module_hr_payroll','=',False)]}"/>
                             </div>
                         </group>
-                        <group string="Timesheets" name="right_column">
+                        <group string="Timesheets">
                             <field name="module_hr_attendance" on_change="onchange_hr_attendance(module_hr_attendance)"/>
                             <field name="module_hr_timesheet" on_change="onchange_hr_timesheet(module_hr_timesheet)"/>
                             <field name="module_hr_timesheet_sheet"/>
diff --git a/addons/hr_payroll/res_config_view.xml b/addons/hr_payroll/res_config_view.xml
index c62281892e8..57de4722310 100644
--- a/addons/hr_payroll/res_config_view.xml
+++ b/addons/hr_payroll/res_config_view.xml
@@ -8,7 +8,7 @@
             <field name="inherit_id" ref="hr.view_human_resources_configuration"/>
             <field name="arch" type="xml">
                 <!-- put after the element <div><field name="module_hr_payroll" class="oe_inline"/> ...</div> -->
-                <xpath expr="//group[@name='left_column']" position="inside">
+                <xpath expr="//group[@name='add_feature']" position="inside">
                     <field name="module_hr_payroll_account"/>
                 </xpath>
             </field>
diff --git a/addons/hr_recruitment/res_config_view.xml b/addons/hr_recruitment/res_config_view.xml
index 60d6d8d152d..f806bfffa72 100644
--- a/addons/hr_recruitment/res_config_view.xml
+++ b/addons/hr_recruitment/res_config_view.xml
@@ -7,15 +7,16 @@
             <field name="type">form</field>
             <field name="inherit_id" ref="hr.view_human_resources_configuration"/>
             <field name="arch" type="xml">
-                <group name="right_column" position="inside">
-                    <separator string="Recruitment" colspan="2"/>
-                    <field name="module_document_ftp"/>
-                    <label for="fetchmail_applicants"/>
-                    <div>
-                        <field name="fetchmail_applicants"/>
-                        <button name="configure_fetchmail_applicants" type="object" string="Configure" icon="gtk-go-forward"
-                            attrs="{'invisible': [('fetchmail_applicants','=',False)]}"/>
-                    </div>
+                <group name="hr_grp" position="inside">
+                    <group string="Recruitment">
+                        <field name="module_document_ftp"/>
+                        <label for="fetchmail_applicants"/>
+                        <div>
+                            <field name="fetchmail_applicants"/>
+                            <button name="configure_fetchmail_applicants" type="object" string="Configure" icon="gtk-go-forward"
+                                attrs="{'invisible': [('fetchmail_applicants','=',False)]}"/>
+                        </div>
+                    </group>
                 </group>
             </field>
         </record>
diff --git a/addons/mrp/res_config_view.xml b/addons/mrp/res_config_view.xml
index 8d81a3d2d0a..9432397b46d 100644
--- a/addons/mrp/res_config_view.xml
+++ b/addons/mrp/res_config_view.xml
@@ -18,15 +18,17 @@
                             <field name="module_stock_planning"/>
                             <field name="module_mrp_subproduct"/>
                             <field name="module_mrp_repair"/>
-                            <separator string="Logistics" colspan="2"/>
+                        </group>
+                        <group string="Bill Of Material">
+                            <field name="group_mrp_routings"/>
+                            <field name="group_mrp_properties"/>
+                        </group>
+                        <group string="Logistics">
                             <field name="module_mrp_jit"/>
                             <field name="module_stock_no_autopicking"/>
                         </group>
                         <group string="Products">
                             <field name="module_product_manufacturer"/>
-                            <separator string="Bill Of Material" colspan="2"/>
-                            <field name="group_mrp_routings"/>
-                            <field name="group_mrp_properties"/>
                         </group>
                     </group>
                 </form>
diff --git a/addons/project/res_config_view.xml b/addons/project/res_config_view.xml
index 6248efde5d2..c7318a159ef 100644
--- a/addons/project/res_config_view.xml
+++ b/addons/project/res_config_view.xml
@@ -13,8 +13,7 @@
                         <button string="Cancel" type="object" name="cancel" class="oe_link"/>
                     </header>
                     <group>
-                        <group>
-                            <separator string="Project" colspan="2"/>
+                        <group string="Project">
                             <field name="module_project_mrp"/>
                             <field name="module_project_timesheet"/>
                             <field name="module_pad"/>
@@ -22,14 +21,14 @@
                             <field name="group_time_work_estimation_tasks"/>
                             <field name="group_manage_delegation_task"/>
                             <field name="time_unit" domain="[('category_id.name','=','Working Time')]"/>
-                        </group>
-                        <group name="project_left_column">
-                            <separator string="Planing" colspan="2"/>
-                            <field name="module_project_long_term"/>
-                            <separator string="Helpdesk and Support" colspan="2"/>
-                            <field name="module_project_issue"/>
-                            <field name="module_project_issue_sheet"/>
-                        </group>
+                       </group>
+                       <group string="Planing">
+                           <field name="module_project_long_term"/>
+                       </group>
+                       <group string="Helpdesk and Support">
+                           <field name="module_project_issue"/>
+                           <field name="module_project_issue_sheet"/>
+                       </group>
                     </group>
                 </form>
             </field>