From f85a4366b0719d619a55c1ec5c47916c8b6f6b2a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20K=C3=BChn?= <aku@odoo.com>
Date: Fri, 2 Jun 2023 15:58:47 +0000
Subject: [PATCH] [FIX] mail: away im status should be yellow in discuss
 sidebar
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Follow-up of https://github.com/odoo/odoo/pull/123143

This is the same fix applied to discus sidebar, which used another
component that had the exact same issue.

This omission comes from code duplication as a consequence of
added technical debt during a recent refactoring of Discuss code.

We will avoid making this mistake again in master, by cleaning and
properly sharing code for all showing of im status in Discuss.

closes odoo/odoo#123526

X-original-commit: 576e3caa50f6e23525b49fc3790ab29647903ed1
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
---
 addons/hr_holidays/static/src/thread_icon.patch.xml | 2 +-
 addons/mail/static/src/core_ui/core.scss            | 4 ++++
 addons/mail/static/src/discuss_app/thread_icon.xml  | 2 +-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/addons/hr_holidays/static/src/thread_icon.patch.xml b/addons/hr_holidays/static/src/thread_icon.patch.xml
index a37e93510b0..906c9e929ed 100644
--- a/addons/hr_holidays/static/src/thread_icon.patch.xml
+++ b/addons/hr_holidays/static/src/thread_icon.patch.xml
@@ -4,7 +4,7 @@
         <xpath expr="//*[@name='chat_static']" position="replace">
             <div t-if="chatPartner.im_status === 'leave_online'" class="o-mail-ThreadIcon-online fa fa-fw fa-plane" title="Online"/>
             <div t-elif="chatPartner.im_status === 'leave_offline'" class="fa fa-fw fa-plane" title="Out of office"/>
-            <div t-elif="chatPartner.im_status === 'leave_away'" class="fa fa-fw fa-plane text-warning" title="Away"/>
+            <div t-elif="chatPartner.im_status === 'leave_away'" class="fa fa-fw fa-plane o-yellow" title="Away"/>
             <t t-else="">$0</t>
         </xpath>
     </t>
diff --git a/addons/mail/static/src/core_ui/core.scss b/addons/mail/static/src/core_ui/core.scss
index 8eb7cb7bd34..fe1baac242c 100644
--- a/addons/mail/static/src/core_ui/core.scss
+++ b/addons/mail/static/src/core_ui/core.scss
@@ -5,3 +5,7 @@
 .o-min-width-0 {
     min-width: 0;
 }
+
+.o-yellow {
+    color: $yellow;
+}
diff --git a/addons/mail/static/src/discuss_app/thread_icon.xml b/addons/mail/static/src/discuss_app/thread_icon.xml
index c5b2897d4c7..b9bb11d9ded 100644
--- a/addons/mail/static/src/discuss_app/thread_icon.xml
+++ b/addons/mail/static/src/discuss_app/thread_icon.xml
@@ -12,7 +12,7 @@
                     <t name="chat_static">
                         <div t-if="chatPartner.im_status === 'online'" class="fa fa-fw fa-circle text-success" title="Online"/>
                         <div t-elif="chatPartner.im_status === 'offline'" class="fa fa-fw fa-circle-o" title="Offline"/>
-                        <div t-elif="chatPartner.im_status === 'away'" class="fa fa-fw fa-circle text-warning" title="Away"/>
+                        <div t-elif="chatPartner.im_status === 'away'" class="fa fa-fw fa-circle o-yellow" title="Away"/>
                         <div t-elif="chatPartner.im_status === 'bot'" class="fa fa-fw fa-heart text-success" title="Bot"/>
                         <div t-else="" class="fa fa-fw fa-question-circle" title="No IM status available"/>
                     </t>