[IMP] digest: improve the digest email layout

Purpose of the task is to improve the usability of digest email layout.

- Click on Connect button to connect to the database.
- Use same left padding in the header as in the section title.
- 2 decimal digits for monetary amounts, no space with currency symbol.
- Count real messages only, not notes or notifications.
- Add new tips for video tour.
- Moved recipients to a new tab.
- Changed email author to company's email.
- Some minor layout fixes.

Task-1918364

2 files changed, 25 insertions, 11 deletions

diff --git a/addons/digest/data/digest_data.xml b/addons/digest/data/digest_data.xml
index 76c6e373d80..5c7c2c792ba 100644
--- a/addons/digest/data/digest_data.xml
+++ b/addons/digest/data/digest_data.xml
@@ -2,7 +2,7 @@
 <odoo>
     <data noupdate="1">
         <record id="digest_digest_default" model="digest.digest">
-            <field name="name">Weekly Digest</field>
+            <field name="name">Weekly Stats in Odoo</field>
             <field name="user_ids" eval="[(4, ref('base.user_admin'))]"/>
             <field name="next_run_date" eval="(DateTime.now() + timedelta(days=7)).strftime('%Y-%m-%d')"/>
             <field name="kpi_res_users_connected">True</field>
@@ -41,5 +41,16 @@
 </div>
             </field>
         </record>
+
+        <record id="digest_tip_mail_2" model="digest.tip">
+            <field name="group_id" ref='base.group_system'/>
+            <field name="sequence">0</field>
+            <field name="tip_description" type="html">
+<div style="font-size: 14px;">
+    To reach your full potential with Odoo, have a look at our <a href="https://www.odoo.com/page/tour">apps videos</a> and discover new tools to work better and faster.
+</div>
+            </field>
+        </record>
+
     </data>
 </odoo>
diff --git a/addons/digest/data/digest_template_data.xml b/addons/digest/data/digest_template_data.xml
index aeaa6a54253..72826734049 100644
--- a/addons/digest/data/digest_template_data.xml
+++ b/addons/digest/data/digest_template_data.xml
@@ -3,8 +3,8 @@
     <record id="digest_mail_template" model="mail.template">
         <field name="name">Digest: Default main template</field>
         <field name="model_id" ref="digest.model_digest_digest"/>
-        <field name="email_from">${user.email_formatted | safe}</field>
         <field name="subject">${'%s: %s' % (ctx.get('user', user).company_id.name, object.name)}</field>
+        <field name="email_from">${('%s' % (object.company_id.partner_id.email_formatted or user.email_formatted))|safe}</field>
         <field name="body_html" type="html">
 <table style="width: 100%; border-spacing: 0; font-family: Helvetica,Arial,Verdana,sans-serif;">
     <tr>
@@ -17,12 +17,15 @@
             % set kpis = data.yesterday.keys()
             <table style="width: 100%; max-width: 600px; border-spacing: 0; border: 1px solid #e7e7e7; border-bottom: none; color: #6e7172; line-height: 23px; text-align: left;">
                 <tr>
-                    <td style="border-collapse: collapse; padding: 10px 40px; text-align: left;">
-                        <strong style="margin-left: -22px; color: #000000; font-size: 22px; line-height: 32px;">${company.name} at a glance</strong>
-                        <div style="color: #000000; font-size: 15px; margin-left:-22px;">${datetime.date.today().strftime('%B %d, %Y')}</div>
+                    <td style="border-collapse: collapse; padding: 10px 20px; text-align: left;">
+                        <strong style="color: #000000; font-size: 22px; line-height: 32px;">${object.name}</strong>
+                        <div style="color: #000000; font-size: 15px;">${datetime.date.today().strftime('%B %d, %Y')}</div>
                     </td>
-                    <td style="text-align: right; padding: 10px 40px">
+                    <td style="text-align: right; padding: 10px 20px">
                         <img style="padding: 0px; margin: 0px; height: auto; width: 80px;" src="/logo.png?company=${company.id}"/>
+                        <div style="padding-top: 5px;">
+                            <a href="/web/login" style="background-color: #875A7B; margin-left: 5px; padding: 8px 16px 8px 16px; text-decoration: none; color: #fff; border-radius: 5px; font-size:13px;">Connect</a>
+                        </div>
                     </td>
                 </tr>
                 <tr><td colspan="2" style="text-align: center;">
@@ -85,10 +88,10 @@
                                             </tr>
                                         </table>
                                     </td>
-                                    <td style="border-collapse: collapse; margin: 0; padding: 0; border-top: 2px solid #56b3b5;">
+                                    <td style="border-collapse: collapse; margin: 0; padding: 15px; border-top: 2px solid #56b3b5;">
                                         <table style="width: 100%; max-width: 199px; border-spacing: 0; margin: 0; padding: 0;">
                                             <tr>
-                                                <td style="border-collapse: collapse; margin: 0; padding: 20; text-align: center;">
+                                                <td style="border-collapse: collapse; margin: 0; text-align: center;">
                                                     <span style="color: #56b3b5; font-size: 35px; font-weight: bold; text-decoration: none; line-height: 36px">${data['lastmonth'][kpi][kpi]}</span><br/>
                                                     <span style="color: #888888; display: inline-block; font-size: 12px; line-height: 18px; text-transform: uppercase;">Last 30 Days</span>
                                                     % if data['lastmonth'][kpi]['margin'] != 0.0:
@@ -114,7 +117,7 @@
             % if tips:
                 <table style="width: 100%; max-width: 600px; margin-top: 5px; border: 1px solid #e7e7e7;">
                     <tr>
-                        <td style="border-collapse: collapse; background-color: #ffffff; line-height: 21px; padding: 0px 20px;"><br/>
+                        <td style="border-collapse: collapse; background-color: #ffffff; line-height: 21px; padding: 0px 20px 20px;"><br/>
                             <div style="color: #3d466e; line-height: 23px;">${tips | safe}</div>
                         </td>
                     </tr>
@@ -127,7 +130,7 @@
                     </td>
                 </tr>
                 <tr>
-                    <td>
+                    <td style="padding-bottom:20px;">
                         <div style="text-align: center;"><a href="https://play.google.com/store/apps/details?id=com.odoo.mobile" target="_blank"><img src="/digest/static/src/img/google_play.png" style="display: inline-block; height: 30px; margin-left: auto; margin-right: 12px;"/></a><a href="https://itunes.apple.com/us/app/odoo/id1272543640" target="_blank"><img src="/digest/static/src/img/app_store.png" style="display: inline-block; height: 30px; margin-left: 12px; margin-right: auto;"/></a>
                         </div>
                     </td>
@@ -138,7 +141,7 @@
                     <td style="border-collapse: collapse; margin: 0; padding: 10px 20px;">
                         % if user.has_group('base.group_system'):
                             <div style="margin-top: 20px;"> 
-                                Want to customize the email?
+                                Want to customize this email?
                                 <a href="/web#view_type=form&amp;model=digest.digest&amp;id=${object.id}" target="_blank" style="color: #875A7B;">Choose the metrics you care about</a>
                             </div>
                             <br />