From 299d2ea153f5a2051468e465cc9ad733e207290b Mon Sep 17 00:00:00 2001
From: stefanorigano <sri@odoo.com>
Date: Sat, 30 Sep 2017 13:56:43 +0200
Subject: [PATCH] [FIX] mail: thread note bg-color
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
This commit fix 596e88f , using a better bg-color in order to improve
text readability. It apply also apply a minor padding-left to avoid
having text attached to the coloured div border.
Moreover, the bg-colour can not be applied to activities thread.
The little history behind: remove the bg-color was decided by fp and
the UX dep. I’m not aware if 596e88f is approved or not, but the final
result was not acceptable.
---
addons/mail/static/src/less/thread.less | 4 +++-
addons/mail/static/src/xml/activity.xml | 2 +-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/addons/mail/static/src/less/thread.less b/addons/mail/static/src/less/thread.less
index eefffc89eefa..bc6605143256 100644
--- a/addons/mail/static/src/less/thread.less
+++ b/addons/mail/static/src/less/thread.less
@@ -105,7 +105,9 @@
word-wrap: break-word;
&.o_mail_note {
- background-color: @odoo-color-silver-dark;
+ background-color: @mail-thread-note;
+ padding-left: @grid-gutter-width*0.3;
+ border-bottom: 1px solid @gray-lighter-dark;
}
.o_mail_subject {
diff --git a/addons/mail/static/src/xml/activity.xml b/addons/mail/static/src/xml/activity.xml
index 35847804a7f5..bef1d90fdf30 100644
--- a/addons/mail/static/src/xml/activity.xml
+++ b/addons/mail/static/src/xml/activity.xml
@@ -22,7 +22,7 @@
t-att-title="activity.activity_type_id[1]"/>
</div>
</div>
- <div class="o_thread_message_core o_mail_note">
+ <div class="o_thread_message_core">
<div class="o_mail_info">
<strong><span t-attf-class="o_activity_date o_activity_color_#{activity.state}"><t t-esc="activity.label_delay" /></span></strong>:
<strong t-if="activity.summary"> “<t t-esc="activity.summary"/>”</strong>
--
GitLab