From 372860264ed1da858618bdc177bb95c55a4a7057 Mon Sep 17 00:00:00 2001
From: "Pedro M. Baeza" <pedro.baeza@tecnativa.com>
Date: Wed, 4 Dec 2024 22:01:31 +0100
Subject: [PATCH] [IMP] account_payment_order: Shorten the vertical used space

Having 2 lines for the reference (one for the type and the other for
the reference itself) steals vertical space to the really important
things in the invoice: the lines, so we compact them into one line
for better ergonomics.
---
 .../views/account_invoice_view.xml            | 20 +++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/account_payment_order/views/account_invoice_view.xml b/account_payment_order/views/account_invoice_view.xml
index a9ceaf6ea..cca2b9d1b 100644
--- a/account_payment_order/views/account_invoice_view.xml
+++ b/account_payment_order/views/account_invoice_view.xml
@@ -58,13 +58,21 @@
             <field name="payment_mode_id" position="after">
                 <field name="payment_order_ok" invisible="1" />
             </field>
+            <!-- First we place the rest of the elements for the new reference distribution -->
             <field name="payment_reference" position="before">
-                <field
-                    name="reference_type"
-                    attrs="{'readonly':[('state','!=','draft')],
-                            'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))],
-                            'required': [('move_type', 'in', ('out_invoice', 'out_refund'))]}"
-                />
+                <label for="payment_reference" />
+                <div name="payment_reference_div" class="d-flex">
+                    <field
+                        name="reference_type"
+                        attrs="{'readonly':[('state','!=','draft')],
+                                'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))],
+                                'required': [('move_type', 'in', ('out_invoice', 'out_refund'))]}"
+                    />
+                </div>
+            </field>
+            <!-- Then we move the std field, as this can only be on a first level -->
+            <field name="reference_type" position="after">
+                <field name="payment_reference" position="move" />
             </field>
         </field>
 </record>