From f2ab0a922270d62cb9fab5fa9a240aa4672d8d31 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          | 73 +++++++++++---------
 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(+), 51 deletions(-)

diff --git a/agreement_legal/views/agreement.xml b/agreement_legal/views/agreement.xml
index 8612a32d..153bce11 100644
--- a/agreement_legal/views/agreement.xml
+++ b/agreement_legal/views/agreement.xml
@@ -97,37 +97,48 @@
                             <field name="readonly" invisible="1" />
                         </group>
                     </group>
-                    <field name="description" required="True" />
-                    <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>
                     <group name="parties" string="Parties">
                         <group name="cust_parties">
diff --git a/agreement_legal/views/agreement_appendix.xml b/agreement_legal/views/agreement_appendix.xml
index 9fd2ec16..c1324f7c 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 a419a5a8..63084142 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 e08dbbda..fd3f77fc 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 3809815c..ad1159db 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