From a5b3ec8f57079c04ce62ef9f072552661913fc2c Mon Sep 17 00:00:00 2001
From: Pablo <pabloqb@gmail.com>
Date: Thu, 28 Mar 2019 18:49:41 +0100
Subject: [PATCH] [IMP] Update fields layout and UX

---
 hotel/wizard/massive_changes.xml                  | 14 +++++++-------
 .../wizard/inherited_massive_changes.xml          | 15 +++++++++------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/hotel/wizard/massive_changes.xml b/hotel/wizard/massive_changes.xml
index 88b3483a5..0163ec07c 100644
--- a/hotel/wizard/massive_changes.xml
+++ b/hotel/wizard/massive_changes.xml
@@ -16,13 +16,13 @@
 				<group colspan="8" col="8">
 					<field name="date_start" required="1"  colspan="4"/>
 					<field name="date_end" required="1" colspan="4" />
-					<field name="dmo" colspan="2" />
-					<field name="dtu" colspan="2" />
-					<field name="dwe" colspan="2" />
-					<field name="dth" colspan="2" />
-					<field name="dfr" colspan="2" />
-					<field name="dsa" colspan="2" />
-					<field name="dsu" colspan="2" />
+					<field name="dmo" />
+					<field name="dtu" />
+					<field name="dwe" />
+					<field name="dth" />
+					<field name="dfr" />
+					<field name="dsa" />
+					<field name="dsu" />
 				</group>
 				<!-- Restricion Fields -->
 				<group col="4" colspan="4" attrs="{'invisible':[('section', '!=', 'restrictions')]}">
diff --git a/hotel_channel_connector/wizard/inherited_massive_changes.xml b/hotel_channel_connector/wizard/inherited_massive_changes.xml
index 99c0111a5..964b3935c 100644
--- a/hotel_channel_connector/wizard/inherited_massive_changes.xml
+++ b/hotel_channel_connector/wizard/inherited_massive_changes.xml
@@ -8,17 +8,20 @@
 			<xpath expr="//form/group[last()]" position="after">
 				<!-- Availability Fields -->
 				<group col="8" colspan="8" attrs="{'invisible':[('section', '!=', 'avail')]}">
-						<field name="change_max_avail" string=""/>
+						<field name="change_max_avail"/>
 						<field name="max_avail"
-							   attrs="{'readonly':[('change_max_avail', '=', False)]}" />
+							   attrs="{'readonly':[('change_max_avail', '=', False)],
+							   'invisible':[('change_max_avail', '=', False)]}"/>
 
-						<field name="change_quota" string=""/>
+						<field name="change_quota"/>
 						<field name="quota"
-							   attrs="{'readonly':[('change_quota', '=', False)]}" />
+							   attrs="{'readonly':[('change_quota', '=', False)],
+							   'invisible':[('change_quota', '=', False)]}"/>
 
-						<field name="change_no_ota" string=""/>
+						<field name="change_no_ota"/>
 						<field name="no_ota"
-							   attrs="{'readonly':[('change_no_ota', '=', False)]}" />
+							   attrs="{'readonly':[('change_no_ota', '=', False)],
+							   'invisible':[('change_no_ota', '=', False)]}"/>
 				</group>
 			</xpath>
 		</field>