From b8a8c28554ab08d95d7e023c59d91525373e66a5 Mon Sep 17 00:00:00 2001
From: David Beguin <dbe@odoo.com>
Date: Fri, 7 Dec 2018 13:38:50 +0000
Subject: [PATCH] [IMP] im_livechat : review channel history layout

The Live chat history was taking too much space and was hard to read.
The goal here is to display more information on the screen.
The pictures and padding are then reduced.

Note that kanban view is not a solution in long term.
At it does not allow to squash messages per user (if consecutive
messages from same user at close interval of time).
As the thread widget will be reviewed by discuss team in a
near future, we keep this at the moment and will integrate the
new thread widget later, mainly to be able to squash the messages.

Task ID : 1913869
Closes PR #30343
---
 .../static/src/scss/im_livechat_history.scss           | 10 +++++++++-
 addons/im_livechat/views/mail_channel_views.xml        |  4 ++--
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/addons/im_livechat/static/src/scss/im_livechat_history.scss b/addons/im_livechat/static/src/scss/im_livechat_history.scss
index 6fa231c58cb..ee4d6b4f834 100644
--- a/addons/im_livechat/static/src/scss/im_livechat_history.scss
+++ b/addons/im_livechat/static/src/scss/im_livechat_history.scss
@@ -11,9 +11,17 @@
                 flex-flow: column;
                 max-height:500px;
                 overflow:auto;
+                padding:2px 0px;
+                .rounded-circle {
+                    width:32px;
+                    height: 32px;
+                }
                 .o_kanban_record {
+                    padding: 0px 8px;
+                    margin: -1px 8px;
                     min-width: 300px;
                     width: 98%;
+                    word-break: break-all;
                 }
             }
             .oe_module_vignette {
@@ -23,7 +31,7 @@
                 padding-top: 8px;
             }
             .oe_module_desc {
-                padding: 8px 8px 8px 70px;
+                padding: 8px 8px 0px 64px;
             }
         }
 
diff --git a/addons/im_livechat/views/mail_channel_views.xml b/addons/im_livechat/views/mail_channel_views.xml
index fedcfec4004..f3acbb82a47 100644
--- a/addons/im_livechat/views/mail_channel_views.xml
+++ b/addons/im_livechat/views/mail_channel_views.xml
@@ -53,7 +53,7 @@
                             <div class="o_history_kanban_container">
                                 <div class="o_history_kanban_sub_container">
                                     <field name="message_ids" mode="kanban">
-                                        <kanban>
+                                        <kanban default_order="create_date DESC">
                                             <field name="author_id"/>
                                             <field name="body"/>
                                             <field name="create_date" groups="base.group_no_one"/>
@@ -65,7 +65,7 @@
                                                         <div class="o_kanban_image">
                                                             <div>
                                                                  <t t-if="record.author_avatar.raw_value">
-                                                                    <img t-att-src="kanban_image('mail.message', 'author_avatar', record.id.raw_value)" alt="Avatar" class="rounded-circle" width="56px" heigth="56px"/>
+                                                                    <img t-att-src="kanban_image('mail.message', 'author_avatar', record.id.raw_value)" alt="Avatar" class="rounded-circle"/>
                                                                  </t>
                                                                  <t t-else=""><img alt="Anonymous" src="/mail/static/src/img/smiley/avatar.jpg" class="rounded-circle"/></t>
                                                             </div>