From 6b0a59c9510beba548e712efea32dd9940c58b5a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20K=C3=BChn?= <aku@odoo.com>
Date: Tue, 22 Mar 2022 14:37:07 +0000
Subject: [PATCH] [IMP] mail: turn 'MessageView/isClicked' into field
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Task-2793280

closes odoo/odoo#86966

Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
---
 addons/mail/static/src/components/message/message.js  | 9 ++-------
 addons/mail/static/src/components/message/message.xml | 2 +-
 addons/mail/static/src/models/message_view.js         | 7 +++++++
 3 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/addons/mail/static/src/components/message/message.js b/addons/mail/static/src/components/message/message.js
index 248ab23cfb5..898e7823193 100644
--- a/addons/mail/static/src/components/message/message.js
+++ b/addons/mail/static/src/components/message/message.js
@@ -29,11 +29,6 @@ export class Message extends Component {
              * it displays message actions.
              */
              isHovered: false,
-            /**
-             * Determine whether the message is clicked. When message is in
-             * clicked state, it keeps displaying actions even if not hovered.
-             */
-            isClicked: false,
         });
         useUpdate({ func: () => this._update() });
         /**
@@ -94,7 +89,7 @@ export class Message extends Component {
     get isActive() {
         return Boolean(
             this.state.isHovered ||
-            this.state.isClicked ||
+            (this.messageView && this.messageView.isClicked) ||
             (
                 this.messageView &&
                 this.messageView.messageActionList &&
@@ -345,7 +340,7 @@ export class Message extends Component {
             !isEventHandled(ev, 'MessageReactionGroup.Click') &&
             !isEventHandled(ev, 'MessageInReplyToView.ClickMessageInReplyTo')
         ) {
-            this.state.isClicked = !this.state.isClicked;
+            this.messageView.update({ isClicked: !this.messageView.isClicked });
         }
     }
 
diff --git a/addons/mail/static/src/components/message/message.xml b/addons/mail/static/src/components/message/message.xml
index ee1969793af..f240145cc7f 100644
--- a/addons/mail/static/src/components/message/message.xml
+++ b/addons/mail/static/src/components/message/message.xml
@@ -6,7 +6,7 @@
             <div class="o_Message position-relative pt-1"
                 t-att-class="{
                     'o_Message_active': isActive,
-                    'o-clicked': state.isClicked,
+                    'o-clicked': messageView.isClicked,
                     'o-discussion': messageView.message.is_discussion or messageView.message.is_notification,
                     'o-has-message-selection': messageView.threadView and messageView.threadView.replyingToMessageView,
                     'o-highlighted': messageView.message.isHighlighted or messageView.isHighlighted,
diff --git a/addons/mail/static/src/models/message_view.js b/addons/mail/static/src/models/message_view.js
index 88f277390f0..641157d221c 100644
--- a/addons/mail/static/src/models/message_view.js
+++ b/addons/mail/static/src/models/message_view.js
@@ -233,6 +233,13 @@ registerModel({
          * id of the current timeout that will reset isHighlighted to false.
          */
         highlightTimeout: attr(),
+        /**
+         * Determines whether the message is clicked. When message is in
+         * clicked state, it keeps displaying actions even if not hovered.
+         */
+        isClicked: attr({
+            default: false
+        }),
         /**
          * Whether the message should be forced to be isHighlighted. Should only
          * be set through @see highlight()