From 9822ef340d06832f2e8db9157188710de1b81fcc Mon Sep 17 00:00:00 2001
From: Pablo <pabloqb@gmail.com>
Date: Tue, 19 Feb 2019 19:50:18 +0100
Subject: [PATCH] [WIP] Improve UI and UX

---
 .../hotel_room_type_restriction_views.xml     | 19 +++++++++++++------
 hotel/views/hotel_room_type_views.xml         |  4 ++--
 .../views/channel_hotel_room_type_views.xml   |  3 ++-
 .../views/inherited_hotel_room_type_views.xml |  2 +-
 4 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/hotel/views/hotel_room_type_restriction_views.xml b/hotel/views/hotel_room_type_restriction_views.xml
index cf8c6f906..8467bc021 100644
--- a/hotel/views/hotel_room_type_restriction_views.xml
+++ b/hotel/views/hotel_room_type_restriction_views.xml
@@ -8,12 +8,19 @@
     <field name="arch" type="xml">
       <form string="Restrictions">
         <sheet>
-          <group>
-            <h1><field name="name"/></h1>
-          </group>
-          <group>
-            <field name="active"/>
-          </group>
+          <div class="oe_button_box" name="button_box">
+            <button name="toggle_active" type="object"
+                    class="oe_stat_button" icon="fa-archive">
+              <field name="active" widget="boolean_button"
+                     options='{"terminology": "archive"}'/>
+            </button>
+          </div>
+            <div class="oe_title">
+              <label for="name" string="Name" />
+              <h1>
+                <field name="name" />
+              </h1>
+            </div>
           <div>
             <separator string="Pricelist Items"/>
             <field name="item_ids" nolabel="1">
diff --git a/hotel/views/hotel_room_type_views.xml b/hotel/views/hotel_room_type_views.xml
index ff197ad83..08c0ab097 100644
--- a/hotel/views/hotel_room_type_views.xml
+++ b/hotel/views/hotel_room_type_views.xml
@@ -26,9 +26,9 @@
                       </group>
                     </group>
                     <group colspan="2">
-                        <group>
-                            <field name="total_rooms_count"/>
+                        <group name="room_ids_group">
                             <field name="room_ids" widget="many2many_tags"/>
+                            <field name="total_rooms_count"/>
                         </group>
                     </group>
                     <notebook>
diff --git a/hotel_channel_connector/views/channel_hotel_room_type_views.xml b/hotel_channel_connector/views/channel_hotel_room_type_views.xml
index 9ec38e060..3eb8e5f97 100644
--- a/hotel_channel_connector/views/channel_hotel_room_type_views.xml
+++ b/hotel_channel_connector/views/channel_hotel_room_type_views.xml
@@ -20,6 +20,7 @@
                 <group>
                     <field name="min_price" class="oe_inline"/>
                     <field name="max_price" class="oe_inline"/>
+                    <field name="sync_date" readonly="1"/>
                 </group>
             </form>
         </field>
@@ -29,7 +30,7 @@
         <field name="name">channel.hotel.room.type.tree</field>
         <field name="model">channel.hotel.room.type</field>
         <field name="arch" type="xml">
-            <tree string="Hotel Channel Virtual Room">
+            <tree string="Hotel Channel Room Type">
                 <field name="backend_id"/>
             </tree>
         </field>
diff --git a/hotel_channel_connector/views/inherited_hotel_room_type_views.xml b/hotel_channel_connector/views/inherited_hotel_room_type_views.xml
index 6014b5073..2a1e20ca8 100644
--- a/hotel_channel_connector/views/inherited_hotel_room_type_views.xml
+++ b/hotel_channel_connector/views/inherited_hotel_room_type_views.xml
@@ -13,7 +13,7 @@
                         confirm="Synchronizing a room type automatically updates its values in Odoo. Do you want to proceed?"/>
             </xpath>
 
-            <xpath expr="//field[@name='room_ids']" position="inside">
+            <xpath expr="//field[@name='room_ids']" position="before">
                 <field name="capacity" />
             </xpath>