From 0abd35fbfbcd24bb3ec8994a58c32fc0feb3098f Mon Sep 17 00:00:00 2001
From: Mansi Gandhi <mag@odoo.com>
Date: Mon, 6 Jun 2016 16:55:46 +0530
Subject: [PATCH] [IMP] mail: Notification email template header buttons order

On emails sent from tasks, the button "View Task" should be the main
Call to Action (primary), not New Task (that should be link like unfollow):
---
 addons/mail/data/mail_data.xml | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/addons/mail/data/mail_data.xml b/addons/mail/data/mail_data.xml
index 44fd815a279..3f9d152f389 100644
--- a/addons/mail/data/mail_data.xml
+++ b/addons/mail/data/mail_data.xml
@@ -78,29 +78,27 @@
             </td>
             <td valign="center" align="right" width="340" style="padding:0px 10px 5px 5px; font-size: 12px;">
                 <p>
-                % if ctx.get('actions'):
-                % for action in ctx['actions']:
-                <a href="${action['url']}" style="padding: 5px 10px; font-size: 12px; line-height: 18px; color: #FFFFFF; border-color:#a24689; text-decoration: none; display: inline-block; margin-bottom: 0px; font-weight: 400; text-align: center; vertical-align: middle; cursor: pointer; white-space: nowrap; background-image: none; background-color: #a24689; border: 1px solid #a24689; border-radius:3px">${action['title']}</a>
-                % endfor
+                % if ctx.get('button_access') or ctx.get('button_follow') or ctx.get('button_unfollow'):
+                <a href="${ctx['button_access']['url']}" style="padding: 5px 10px; font-size: 12px; line-height: 18px; color: #FFFFFF; border-color:#a24689; text-decoration: none; display: inline-block; margin-bottom: 0px; font-weight: 400; text-align: center; vertical-align: middle; cursor: pointer; white-space: nowrap; background-image: none; background-color: #a24689; border: 1px solid #a24689; border-radius:3px"><strong>${ctx['button_access']['title']}</strong></a>
                 % endif
 
                 % if ctx.get('button_follow'):
                 % if ctx.get('actions'):
                    |
                 % endif
-                <a href="${ctx['button_follow']['url']}" style="text-decoration:none; color: #a24689;"><strong>${ctx['button_follow']['title']}</strong></a>
+                <a href="${ctx['button_follow']['url']}" style="text-decoration:none; color: #a24689;">${ctx['button_follow']['title']}</a>
                 % elif ctx.get('button_unfollow'):
                 % if ctx.get('actions'):
                    |
                 % endif
-                <a href="${ctx['button_unfollow']['url']}" style="text-decoration:none; color: #a24689;"><strong>${ctx['button_unfollow']['title']}</strong></a>
+                <a href="${ctx['button_unfollow']['url']}" style="text-decoration:none; color: #a24689;">${ctx['button_unfollow']['title']}</a>
                 % endif
 
-                % if ctx.get('button_access') or ctx.get('button_follow') or ctx.get('button_unfollow'):
                 % if ctx.get('actions'):
-                   |
-                % endif
-                <a href="${ctx['button_access']['url']}" style="text-decoration:none; color: #a24689;"><strong>${ctx['button_access']['title']}</strong></a>
+                |
+                % for action in ctx['actions']:
+                <a href="${action['url']}" style="text-decoration:none; color: #a24689;">${action['title']}</a>
+                % endfor
                 % endif
                 </p>
             </td>