Add missing check for parent not editable for element to be unremovable

Author: sobo-odoo

addons/html_builder/static/src/core/remove_plugin.js

@@ -5,6 +5,7 @@ import { resizeGrid } from "@html_builder/utils/grid_layout_utils";
 import { getVisibleSibling } from "./move_plugin";
 import { unremovableNodePredicates as deletePluginPredicates } from "@html_editor/core/delete_plugin";
 import { isUnremovableQWebElement as qwebPluginPredicate } from "@html_editor/others/qweb_plugin";
+import { isEditable } from "@html_builder/utils/utils";
 
 // TODO (see forceNoDeleteButton) make a resource in the options plugins to not
 // duplicate some selectors.
@@ -18,12 +19,10 @@ const unremovableSelectors = [
     ".s_table_of_content_navbar_wrap",
     ".s_table_of_content_main",
     ".nav-item",
-    "header",
-    "main",
-    "footer",
 ].join(", ");
 
 const unremovableNodePredicates = [
+    (node) => !isEditable(node.parentNode),
     ...deletePluginPredicates,
     qwebPluginPredicate,
     (node) => node.parentNode.matches('[data-oe-type="image"]'),

addons/html_builder/static/src/utils/utils.js

@@ -150,3 +150,25 @@ export function getSelectorParams(builderOptions, optionClass) {
     }
     return selectorParams;
 }
+
+/**
+ * Checks if the given element is editable.
+ *
+ * @param {HTMLElement} node the element
+ * @returns {Boolean}
+ */
+export function isEditable(node) {
+    let currentNode = node;
+    while (currentNode) {
+        if (currentNode.className && typeof currentNode.className === "string") {
+            if (currentNode.className.includes("o_not_editable")) {
+                return false;
+            }
+            if (currentNode.className.includes("o_editable")) {
+                return true;
+            }
+        }
+        currentNode = currentNode.parentNode;
+    }
+    return false;
+}