From d2012542a3e4a37b0fb05480f8c4b2da8b20baff Mon Sep 17 00:00:00 2001
From: "DBR (OpenERP)" <dbr@tinyerp.com>
Date: Thu, 21 Oct 2010 17:04:38 +0530
Subject: [PATCH] [IMP]hr_holidays,hr_payroll : Usability Improvment

bzr revid: dbr@tinyerp.com-20101021113438-7yscni6sx7ponvzk
---
 addons/hr_holidays/hr_holidays_view.xml | 17 +++++++++++------
 addons/hr_payroll/hr_payroll_view.xml   | 24 +++++++++++++-----------
 2 files changed, 24 insertions(+), 17 deletions(-)

diff --git a/addons/hr_holidays/hr_holidays_view.xml b/addons/hr_holidays/hr_holidays_view.xml
index be3d1ba9dac..9f911e7c326 100644
--- a/addons/hr_holidays/hr_holidays_view.xml
+++ b/addons/hr_holidays/hr_holidays_view.xml
@@ -361,12 +361,17 @@
                     </group>
 					<notebook colspan="4">
 					  <page string="Details">
-						<group name="selection" col="2" colspan="2">
-							<field name="active"/>
-							<newline/>
-							<field name="double_validation" />
-							<newline/>
-							<field name="limit" />
+						<group name="selection" col="4" colspan="4">
+							<group colspan="2">
+							<separator string="Validation" />
+								<newline/>
+							<field name="active" />
+								<newline/>							
+							<field name="double_validation"/>
+								<newline/>
+							<field name="limit"/>
+							</group>
+							<group name="second" colspan="2"/>
 						</group>
 					  </page>
 					</notebook>
diff --git a/addons/hr_payroll/hr_payroll_view.xml b/addons/hr_payroll/hr_payroll_view.xml
index 3eec88ef26c..641c565704e 100644
--- a/addons/hr_payroll/hr_payroll_view.xml
+++ b/addons/hr_payroll/hr_payroll_view.xml
@@ -403,18 +403,20 @@
             <field name="type">form</field>
             <field name="arch" type="xml">
             <xpath expr="//group[@name='selection']" position="replace">
-				<group col="1">
-		          	<separator colspan="2" string="Validation"/>
-					<newline/>
-					<field name="active"/>
-					<field name="limit"/>
-					<field name="double_validation"/>
+				<group colspan="2">
+							<separator string="Validation" colspan="2"/>
+								<newline/>
+							<field name="active" />
+								<newline/>							
+							<field name="double_validation"/>
+								<newline/>
+							<field name="limit"/>
 			  	</group>
-				<group col="2">						
-					<separator colspan="2" string="Payroll Configurtion"/>
-					<field name="type"/>
-					<field name="head_id"/>
-					<field name="company_id" groups="base.group_multi_company" widget="selection" select="2"/>
+				<group colspan="2">						
+					<separator colspan="2" string="Payroll Configurtion"/><newline/>
+					<field name="type"/><newline/>
+					<field name="head_id"/><newline/>
+					<field name="company_id" groups="base.group_multi_company" widget="selection" select="2"/><newline/>
 					<field name="code" readonly="True"/>
 				</group>
 			</xpath>