[IMP] repair: make templates more modern and updated with Odoo guidelines

In this commit we improve templates used in repair. Purpose of this commit
is to have templates that embed or use standard Odoo email layouts to make
them look modern and have a common style across all emails.

Main guidelines

 * better use of div / p / br to try to lessen layout issues, especially
   when updating templates using the editor;
 * correctly sequence the templates fields definition;
 * correctly set templates values notably auto_delete and user_signature
   fields to avoid confusion;
 * correctly layout the email content using light notification email. It
   can either propagate the layout choice through various send mail methods
   or directly embed the styling in the templates for more technical or
   complex templates;
 * use email_formatted computed field when possible to avoid having hand-made
   from / to addresses;
 * fix various typos and improve subjects when necessary;

Content of emails is not necessarily updated as the purpose of this task is
about styling, not content itself.

This commit is linked to task ID 1843395 (and 1868112) and to PR #25294
(and #25889).

addons/repair/data/repair_data.xml

@@ -1,31 +1,35 @@
 <?xml version="1.0" encoding="utf-8"?>
 <odoo>
     <data noupdate="1">
-
-        <!--Email template -->
         <record id="mail_template_repair_quotation" model="mail.template">
-            <field name="name">Repair Quotation - Send by Email</field>
-            <field name="email_from">${(object.create_uid.email and '%s &lt;%s&gt;' % (object.create_uid.name, object.create_uid.email) or '')|safe}</field>
-            <field name="subject">${object.partner_id.name} Repair Orders (Ref ${object.name or 'n/a' })</field>
-            <field name="partner_to">${object.partner_id.id}</field>
+            <field name="name">Repair Quotation: Send by email</field>
             <field name="model_id" ref="repair.model_repair_order"/>
+            <field name="subject">${object.partner_id.name} Repair Orders (Ref ${object.name or 'n/a' })</field>
+            <field name="email_from">${(object.create_uid.email and '%s &lt;%s&gt;' % (object.create_uid.name, object.create_uid.email) or '')|safe}</field>
+            <field name="partner_to">${object.partner_id.id}</field>
+            <field name="body_html" type="xml">
+<div style="margin: 0px; padding: 0px;">
+    <p style="margin: 0px; padding: 0px;font-size: 13px;">
+        Hello ${object.partner_id.name},<br/>
+        Here is your repair order ${doc_name} <strong>${object.name}</strong>
+        % if object.origin:
+            (with reference: ${object.origin} )
+        % endif
+        % if object.invoice_method != 'none':
+            amounting in <strong>${format_amount(object.amount_total, object.pricelist_id.currency_id)}.</strong><br/>
+        % else:
+            .<br/>
+        % endif
+        You can reply to this email if you have any questions.
+        <br/><br/>
+        Thank you,
+    </p>
+</div></field>
             <field name="report_template" ref="action_report_repair_order"/>
             <field name="report_name">${(object.name or '').replace('/','_')}</field>
             <field name="lang">${object.partner_id.lang}</field>
-            <field name="body_html" type="xml">
-                <p>Dear ${object.partner_id.name}</p>
-                <p>
-                Here is your repair order ${doc_name} <strong>${object.name}</strong>
-                % if object.origin:
-                (with reference: ${object.origin} )
-                % endif
-                % if object.invoice_method != 'none':
-                amounting in <strong>${format_amount(object.amount_total, object.pricelist_id.currency_id)}.</strong>
-                % endif
-                </p>
-                <p>You can reply to this email if you have any questions.</p>
-                <p>Thank you,</p>
-            </field>
+            <field name="auto_delete" eval="True"/>
+            <field name="user_signature" eval="True"/>
         </record>
     </data>
 </odoo>

addons/repair/models/repair.py

@@ -256,7 +256,8 @@ class Repair(models.Model):
             'default_res_id': self.id,
             'default_use_template': bool(template_id),
             'default_template_id': template_id,
-            'default_composition_mode': 'comment'
+            'default_composition_mode': 'comment',
+            'custom_layout': 'mail.mail_notification_light',
         }
         return {
             'type': 'ir.actions.act_window',