From c3fc472a46f29da0f93a8f9928d68a36ef306882 Mon Sep 17 00:00:00 2001
From: "Sanjay Gohel (Open ERP)" <sgo@tinyerp.com>
Date: Thu, 5 Jul 2012 14:48:01 +0530
Subject: [PATCH] [IMP]improve configuaration view for hr

bzr revid: sgo@tinyerp.com-20120705091801-kapmj1qkh61cs4yg
---
 addons/hr/res_config_view.xml             | 12 ++++--------
 addons/hr_recruitment/res_config_view.xml | 19 +++++++++----------
 2 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/addons/hr/res_config_view.xml b/addons/hr/res_config_view.xml
index b41ccd0e935..1916760635d 100644
--- a/addons/hr/res_config_view.xml
+++ b/addons/hr/res_config_view.xml
@@ -21,13 +21,11 @@
                         <button string="Cancel" type="object" name="cancel" class="oe_link"/>
                     </header>
                     <sheet>
-                        <group string="Additional Features">
-                            <group>
+                        <group>
+                            <group string="Additional Features" name="left_column">
                                 <field name="module_hr_expense"/>
                                 <field name="module_hr_evaluation"/>
                                 <field name="module_hr_holidays"/>
-                            </group>
-                            <group>
                                 <field name="module_hr_recruitment"/>
                                 <field name="module_hr_contract"/>
                                 <label for="module_hr_payroll"/>
@@ -38,14 +36,12 @@
                                         attrs="{'invisible': [('module_hr_payroll','=',False)]}"/>
                                 </div>
                             </group>
-                        </group>
-                        <group>
-                            <group string="Timesheets">
+                            <group name="right_column">
+                                <separator string="Timesheets" colspan="2"/>
                                 <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"/>
                             </group>
-                            <group name="config_recruitment"/>
                         </group>
                     </sheet>
                 </form>
diff --git a/addons/hr_recruitment/res_config_view.xml b/addons/hr_recruitment/res_config_view.xml
index 011e3816c2c..60d6d8d152d 100644
--- a/addons/hr_recruitment/res_config_view.xml
+++ b/addons/hr_recruitment/res_config_view.xml
@@ -7,16 +7,15 @@
             <field name="type">form</field>
             <field name="inherit_id" ref="hr.view_human_resources_configuration"/>
             <field name="arch" type="xml">
-                <group name="config_recruitment" position="replace">
-                    <group name="config_recruitment" 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 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>
             </field>
         </record>