fix logo adding/removing option

Logo can have two views: image and a text one, and so we pass the view of the one that's displayed right now. Or, if the logo is hidden, we pass the image logo.
odoo-dev · agau-odoo · Apr 29, 2025 · May 21, 2025 · May 20, 2025 · May 22, 2025
commit 04ac5addcd6aff11a1a6f6419534c3900d1892e1
diff --git a/addons/website/static/src/builder/plugins/options/header_element_option.js b/addons/website/static/src/builder/plugins/options/header_element_option.js
@@ -0,0 +1,25 @@
+import { BaseOptionComponent } from "@html_builder/core/utils";
+
+export class HeaderElementOption extends BaseOptionComponent {
+    static template = "website.headerElementOption";
+
+    setup(){
+        super.setup();
+        this.customizeWebsite = this.env.editor.shared.customizeWebsite;
+        const views = [
+            "website.option_header_brand_logo", 
+            "website.option_header_brand_name"
+        ];
+        this.customizeWebsite.loadConfigKey({ views });
+    }
+
+    get websiteLogoParams(){
+        const views = this.customizeWebsite.getConfigKey("website.option_header_brand_name") 
+                        ? ["website.option_header_brand_name"] 
+                        : ["website.option_header_brand_logo"];
+        return {
+            views,
+            resetViewArch: true,
+        }
+    }
+}
diff --git a/addons/website/static/src/builder/plugins/options/header_option.xml b/addons/website/static/src/builder/plugins/options/header_option.xml
@@ -383,10 +383,7 @@
                 <BuilderButton
                     title.translate="Show/hide logo"
                     className="'flex-grow-1'"
-                    actionParam="{
-                        views: ['website.option_header_brand_name', 'website.option_header_brand_logo'],
-                        resetViewArch: true,
-                    }"
+                    actionParam="websiteLogoParams"
                 >
                     <Img src="'/website/static/src/img/snippets_options/header_extra_element_logo.svg'"/>
                 </BuilderButton>

diff --git a/addons/website/static/src/builder/plugins/options/header_option_plugin.js b/addons/website/static/src/builder/plugins/options/header_option_plugin.js
@@ -12,6 +12,7 @@ import { Plugin } from "@html_editor/plugin";
 import { withSequence } from "@html_editor/utils/resource";
 import { registry } from "@web/core/registry";
 import { HeaderBorderOption } from "./header_border_option";
+import { HeaderElementOption } from "./header_element_option";
 
 const [
     HEADER_TEMPLATE,
@@ -74,7 +75,7 @@ class HeaderOptionPlugin extends Plugin {
             }),
             withSequence(HEADER_ELEMENT, {
                 editableOnly: false,
-                template: "website.headerElementOption",
+                OptionComponent: HeaderElementOption,
                 selector: "#wrapwrap > header",
                 groups: ["website.group_website_designer"],
             }),