From e7a5d9c7168c0261615b0d8ccce79285e2762a43 Mon Sep 17 00:00:00 2001
From: tarteo <d.sluijk@onestein.nl>
Date: Wed, 25 Feb 2026 16:11:05 +0100
Subject: [PATCH] [IMP] agreement_legal: Placeholder creator section layout

---
 agreement_legal/views/agreement.xml          | 74 +++++++++++---------
 agreement_legal/views/agreement_appendix.xml | 17 +++--
 agreement_legal/views/agreement_clause.xml   | 17 +++--
 agreement_legal/views/agreement_recital.xml  | 17 +++--
 agreement_legal/views/agreement_section.xml  | 17 +++--
 5 files changed, 90 insertions(+), 52 deletions(-)

diff --git a/agreement_legal/views/agreement.xml b/agreement_legal/views/agreement.xml
index 02b17ed10..aa802c940 100644
--- a/agreement_legal/views/agreement.xml
+++ b/agreement_legal/views/agreement.xml
@@ -129,38 +129,48 @@
 
             <!-- 12) Add description after main group -->
             <xpath expr="//sheet/group[@name='main']" position="after">
-                <field name="description" required="1" />
-                <!-- 13) Add dynamic fields block after description -->
-                <group class="oe_edit_only">
-                    <group>
-                        <field
-                            name="field_id"
-                            domain="[('model_id', '=', 'agreement'),
-                                        ('ttype', '!=', 'one2many'),
-                                        ('ttype', '!=', 'many2many')]"
-                        />
-                        <field name="sub_object_id" readonly="1" />
-                        <field
-                            name="sub_model_object_field_id"
-                            domain="[('model_id', '=', sub_object_id),
-                                        ('ttype', '!=', 'one2many'),
-                                        ('ttype', '!=', 'many2many')]"
-                            readonly="sub_object_id == False"
-                            required="sub_object_id != False"
-                        />
-                        <field name="default_value" />
-                        <field name="copyvalue" />
-                    </group>
-                    <p>
-                        This section (on the left) allows you to add dynamic fields inside the description and special terms.
-                        <ol>
-                        <li>Select the agreement field</li>
-                        <li>Select the sub-field</li>
-                        <li>Enter the default value if the field is empty</li>
-                        <li
-                        >Copy and paste the placeholder expression in the description or the special terms</li>
-                    </ol>
-                    </p>
+                <group string="Description">
+                    <field
+                        name="description"
+                        required="True"
+                        nolabel="1"
+                        colspan="2"
+                        placeholder="Enter description here..."
+                    />
+                    <div
+                        class="oe_edit_only alert alert-light d-flex justify-content-between flex-xl-row flex-column"
+                        role="status"
+                    >
+                        <group class="flex-fill">
+                            <field
+                                name="field_id"
+                                domain="[('model_id', '=', 'agreement'),
+                                            ('ttype', '!=', 'one2many'),
+                                            ('ttype', '!=', 'many2many')]"
+                            />
+                            <field name="sub_object_id" readonly="1" />
+                            <field
+                                name="sub_model_object_field_id"
+                                domain="[('model_id', '=', sub_object_id),
+                                            ('ttype', '!=', 'one2many'),
+                                            ('ttype', '!=', 'many2many')]"
+                                readonly="sub_object_id == False"
+                                required="sub_object_id != False"
+                            />
+                            <field name="default_value" />
+                            <field name="copyvalue" />
+                        </group>
+                        <p class="px-xl-4">
+                            This section (on the left) allows you to add dynamic fields inside the description and special terms.
+                            <ol>
+                            <li>Select the agreement field</li>
+                            <li>Select the sub-field</li>
+                            <li>Enter the default value if the field is empty</li>
+                            <li
+                            >Copy and paste the placeholder expression in the description or the special terms</li>
+                        </ol>
+                        </p>
+                    </div>
                 </group>
             </xpath>
 
diff --git a/agreement_legal/views/agreement_appendix.xml b/agreement_legal/views/agreement_appendix.xml
index 9fd2ec166..c1324f7ca 100644
--- a/agreement_legal/views/agreement_appendix.xml
+++ b/agreement_legal/views/agreement_appendix.xml
@@ -44,9 +44,16 @@
                             <field name="sequence" />
                         </group>
                     </group>
-                    <field name="content" widget="html" />
-                    <group class="oe_edit_only">
-                        <group>
+                    <field
+                        name="content"
+                        widget="html"
+                        placeholder="Enter content here..."
+                    />
+                    <div
+                        class="oe_edit_only alert alert-light d-flex justify-content-between flex-xl-row flex-column"
+                        role="status"
+                    >
+                        <group class="flex-fill">
                             <field
                                 name="field_id"
                                 domain="[('model_id', '=', 'agreement.appendix'),
@@ -65,7 +72,7 @@
                             <field name="default_value" />
                             <field name="copyvalue" />
                         </group>
-                        <p>
+                        <p class="px-xl-4">
                             This section (on the left) allows you to add dynamic fields inside the content.
                             <ol>
                             <li>Select the appendix field</li>
@@ -75,7 +82,7 @@
                             >Copy and paste the placeholder expression in the content</li>
                         </ol>
                         </p>
-                    </group>
+                    </div>
                 </sheet>
             </form>
         </field>
diff --git a/agreement_legal/views/agreement_clause.xml b/agreement_legal/views/agreement_clause.xml
index a419a5a8d..630841422 100644
--- a/agreement_legal/views/agreement_clause.xml
+++ b/agreement_legal/views/agreement_clause.xml
@@ -49,9 +49,16 @@
                             <field name="sequence" />
                         </group>
                     </group>
-                    <field name="content" widget="html" />
-                    <group class="oe_edit_only">
-                        <group>
+                    <field
+                        name="content"
+                        widget="html"
+                        placeholder="Enter content here..."
+                    />
+                    <div
+                        class="oe_edit_only alert alert-light d-flex justify-content-between flex-xl-row flex-column"
+                        role="status"
+                    >
+                        <group class="flex-fill">
                             <field
                                 name="field_id"
                                 domain="[('model_id', '=', 'agreement.clause'),
@@ -70,7 +77,7 @@
                             <field name="default_value" />
                             <field name="copyvalue" />
                         </group>
-                        <p>
+                        <p class="px-xl-4">
                             This section (on the left) allows you to add dynamic fields inside the content.
                             <ol>
                             <li>Select the clause field</li>
@@ -80,7 +87,7 @@
                             >Copy and paste the placeholder expression in the content</li>
                         </ol>
                         </p>
-                    </group>
+                    </div>
                 </sheet>
             </form>
         </field>
diff --git a/agreement_legal/views/agreement_recital.xml b/agreement_legal/views/agreement_recital.xml
index e08dbbdae..fd3f77fc9 100644
--- a/agreement_legal/views/agreement_recital.xml
+++ b/agreement_legal/views/agreement_recital.xml
@@ -44,9 +44,16 @@
                             <field name="sequence" />
                         </group>
                     </group>
-                    <field name="content" widget="html" />
-                    <group class="oe_edit_only">
-                        <group>
+                    <field
+                        name="content"
+                        widget="html"
+                        placeholder="Enter content here..."
+                    />
+                    <div
+                        class="oe_edit_only alert alert-light d-flex justify-content-between flex-xl-row flex-column"
+                        role="status"
+                    >
+                        <group class="flex-fill">
                             <field
                                 name="field_id"
                                 domain="[('model_id', '=', 'agreement.recital'),
@@ -65,7 +72,7 @@
                             <field name="default_value" />
                             <field name="copyvalue" />
                         </group>
-                        <p>
+                        <p class="px-xl-4">
                             This section (on the left) allows you to add dynamic fields inside the content.
                             <ol>
                             <li>Select the recital field</li>
@@ -75,7 +82,7 @@
                             >Copy and paste the placeholder expression in the content</li>
                         </ol>
                         </p>
-                    </group>
+                    </div>
                 </sheet>
             </form>
         </field>
diff --git a/agreement_legal/views/agreement_section.xml b/agreement_legal/views/agreement_section.xml
index 3809815c5..ad1159db9 100644
--- a/agreement_legal/views/agreement_section.xml
+++ b/agreement_legal/views/agreement_section.xml
@@ -46,9 +46,16 @@
                     </group>
                     <notebook>
                         <page string="Content">
-                            <field name='content' nolabel="1" />
-                            <group class="oe_edit_only">
-                                <group>
+                            <field
+                                name='content'
+                                nolabel="1"
+                                placeholder="Enter content here..."
+                            />
+                            <div
+                                class="oe_edit_only alert alert-light d-flex justify-content-between flex-xl-row flex-column"
+                                role="status"
+                            >
+                                <group class="flex-fill">
                                     <field
                                         name="field_id"
                                         domain="[('model_id', '=', 'agreement.section'),
@@ -67,7 +74,7 @@
                                     <field name="default_value" />
                                     <field name="copyvalue" />
                                 </group>
-                                <p>
+                                <p class="px-xl-4">
                                     This section (on the left) allows you to add dynamic fields inside the content.
                                     <ol>
                                     <li>Select the section field</li>
@@ -78,7 +85,7 @@
                                     >Copy and paste the placeholder expression in the content</li>
                                 </ol>
                                 </p>
-                            </group>
+                            </div>
                         </page>
                         <page string="Clauses">
                             <field