[FIX] website_{livechat}: hide chat hub using css only

When editing is enabled, the website loads both the backend and frontend
assets. Until [1], this resulted in two sets of chat windows and bubbles
being displayed.

This commit is a follow-up to the above PR, maintaining the same behavior
but relying only on styles. This avoids the need for cross-iframe or cross
-window communication.

[1]: odoo#212452

closes odoo#216765

X-original-commit: da79aae
Signed-off-by: Alexandre Kühn (aku) <[email protected]>
Signed-off-by: Matthieu Stockbauer (tsm) <[email protected]>

Author: tsm-odoo

addons/im_livechat/static/src/embed/frontend/livechat_root.js

@@ -19,7 +19,5 @@ export class LivechatRoot extends Component {
     setup() {
         useSubEnv({ embedLivechat: true });
         this.overlayService = useService("overlay");
-        this.store = useService("mail.store");
-        this.store.embedLivechat = true;
     }
 }

addons/mail/static/src/core/common/chat_hub.xml

@@ -2,7 +2,7 @@
 <templates xml:space="preserve">
 
 <t t-name="mail.ChatHub">
-    <div t-if="chatHub.show" class="o-mail-ChatHub d-print-none">
+    <div class="o-mail-ChatHub d-print-none">
 
         <t t-set="visibleChatWindows" t-value="chatHub.compact ? chatHub.opened.filter(({ bypassCompact }) => bypassCompact) : chatHub.opened "/>
         <t t-foreach="visibleChatWindows" t-as="cw" t-key="cw.localId">

addons/mail/static/src/core/common/chat_hub_model.js

@@ -146,10 +146,6 @@ export class ChatHub extends Record {
         );
     }
 
-    get show() {
-        return true;
-    }
-
     showConversations = fields.Attr(false, {
         compute() {
             return this.canShowOpened.length + this.canShowFolded.length > 0;

addons/website/static/src/client_actions/website_preview/website_builder_action.js

@@ -76,9 +76,6 @@ export class WebsiteBuilderClientAction extends Component {
         this.component = useComponent();
 
         this.onKeydownRefresh = this._onKeydownRefresh.bind(this);
-        this.env.services["mail.store"].isReady.then(() => {
-            this.env.services["mail.store"].websiteBuilder.on = true;
-        });
 
         onMounted(() => {
             // You can't wait for rendering because the Builder depends on the
@@ -145,9 +142,6 @@ export class WebsiteBuilderClientAction extends Component {
             websiteSystrayRegistry.remove("website.WebsiteSystrayItem");
             this.websiteService.currentWebsiteId = null;
             websiteSystrayRegistry.trigger("EDIT-WEBSITE");
-            this.env.services["mail.store"].isReady.then(() => {
-                this.env.services["mail.store"].websiteBuilder.on = false;
-            });
         });
 
         effect(
@@ -168,14 +162,8 @@ export class WebsiteBuilderClientAction extends Component {
                         websiteSystrayRegistry.trigger("EDIT-WEBSITE");
                         document.querySelector(".o_builder_open .o_main_navbar").classList.add("d-none");
                     }, 200);
-                    this.env.services["mail.store"].isReady.then(() => {
-                        this.env.services["mail.store"].websiteBuilder.editing = true;
-                    });
                 } else {
                     document.querySelector(".o_main_navbar")?.classList.remove("d-none");
-                    this.env.services["mail.store"].isReady.then(() => {
-                        this.env.services["mail.store"].websiteBuilder.editing = false;
-                    });
                 }
             },
             () => [this.state.isEditing]
@@ -294,10 +282,6 @@ export class WebsiteBuilderClientAction extends Component {
     }
 
     onIframeLoad(ev) {
-        this.env.services["mail.store"].isReady.then(() => {
-            this.env.services["mail.store"].websiteBuilder.iframeWindow =
-                this.websiteContent.el.contentWindow;
-        });
         this.websiteService.pageDocument = this.websiteContent.el.contentDocument;
         if (this.translation) {
             deleteQueryParam("edit_translations", this.websiteService.contentWindow, true);

addons/website/static/src/common/chat_hub_model_patch.js

@@ -30,18 +30,18 @@
             "website_livechat/static/src/**/common/**/*",
         ],
         'website.assets_wysiwyg': [
-            'website_livechat/static/src/scss/**/*',
+            "website/static/src/**/common/**/*",
         ],
         'website.assets_editor': [
             'website_livechat/static/src/js/**/*',
         ],
         'web.assets_frontend': [
-            'website_livechat/static/src/patch/assets_frontend/website.scss',
+            "website_livechat/static/src/**/common/**/*",
+            'website_livechat/static/src/**/frontend/**/*',
         ],
         'web.assets_backend': [
             "website_livechat/static/src/**/common/**/*",
             'website_livechat/static/src/**/*',
-            ('remove', 'website_livechat/static/src/scss/**/*'),
         ],
         'web.assets_unit_tests': [
             'website_livechat/static/tests/**/*',

addons/website/static/src/common/store_service_patch.js

@@ -0,0 +1,10 @@
+body.editor_enable .o-livechat-root {
+    &::part(ChatHub),
+    &::part(openChatButton) {
+        visibility: hidden;
+    }
+}
+
+body:has(.o_website_preview):not(.o_builder_open) .o-mail-ChatHub {
+    visibility: hidden;
+}