# HG changeset patch
# User Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
# Date 2018-02-06 21:28:13
# Node ID 51906236eed3fcfbb6c06f997f4632f0c291b8ef
# Parent  3a3d9c8432edd715c4cf8d295723b043f25e1267

HTML email templates: restrict amount of visible hyperlinks

Users report that some HTML email templates are confusing due to the large
number of hyperlinks in them. They typically click the first thing they see
that looks clickable without actually reading what they are clicking on,
expecting that it all points to the same thing.

In pullrequest invitations, the pullrequest and changeset URLs should be
clickable, but the source and target repository should not: this is not what
you typically want to see when taking action on this mail.

Similarly in comment emails, the corresponding repository should not be
clickable.

Unfortunately, even if we remove the <a> hyperlink tags on these repository
URLs, most mail clients will still recognize the text as being URLs
and make them clickable anyway. Worse, they will do so in a different
styling than the other links.

Therefore, do add <a> tags in the HTML, but make use of styling like that of
data_style, i.e. regular text color, grey background with dark grey border.
The links _will_ thus be clickable but it will not be as inviting as before,
thus removing confusion.

diff --git a/kallithea/model/notification.py b/kallithea/model/notification.py
--- a/kallithea/model/notification.py
+++ b/kallithea/model/notification.py
@@ -341,6 +341,7 @@ class EmailNotificationModel(object):
                 "data_style": "border:%(color_border)s 1px solid;background:%(color_background_grey)s" % _kwargs,
                 "emph_style": "font-weight:600;color:%(color_emph)s" % _kwargs,
                 "link_style": "color:%(color_link)s;text-decoration:none" % _kwargs,
+                "link_text_style": "color:%(color_text)s;text-decoration:none;border:%(color_border)s 1px solid;background:%(color_background_grey)s" % _kwargs,
                 })
 
         log.debug('rendering tmpl %s with kwargs %s', base, _kwargs)
diff --git a/kallithea/templates/email_templates/changeset_comment.html b/kallithea/templates/email_templates/changeset_comment.html
--- a/kallithea/templates/email_templates/changeset_comment.html
+++ b/kallithea/templates/email_templates/changeset_comment.html
@@ -18,7 +18,7 @@
         <td>
             <div>
                 ${_('Changeset on')}
-                <a style="${link_style}"
+                <a style="${link_text_style}"
                    href="${cs_target_repo}">${cs_target_repo}</a>
                 ${_('branch')}
                 <span style="${data_style}">${branch}</span>:
diff --git a/kallithea/templates/email_templates/pull_request.html b/kallithea/templates/email_templates/pull_request.html
--- a/kallithea/templates/email_templates/pull_request.html
+++ b/kallithea/templates/email_templates/pull_request.html
@@ -10,12 +10,12 @@
         <td>
             <div>
                 ${_('Pull request from')}
-                <a style="${link_style}"
+                <a style="${link_text_style}"
                    href="${pr_source_repo}">${pr_source_repo}</a>
                 ${_('at')}
                 <span style="${data_style}">${pr_source_branch}</span>
                 ${_('to')}
-                <a style="${link_style}"
+                <a style="${link_text_style}"
                    href="${pr_target_repo}">${pr_target_repo}</a>
                 ${_('at')}
                 <span style="${data_style}">${pr_target_branch}</span>
diff --git a/kallithea/templates/email_templates/pull_request_comment.html b/kallithea/templates/email_templates/pull_request_comment.html
--- a/kallithea/templates/email_templates/pull_request_comment.html
+++ b/kallithea/templates/email_templates/pull_request_comment.html
@@ -18,12 +18,12 @@
         <td>
             <div>
                 ${_('Pull request from')}
-                <a style="${link_style}"
+                <a style="${link_text_style}"
                    href="${pr_source_repo}">${pr_source_repo}</a>
                 ${_('branch')}
                 <span style="${data_style}">${pr_source_branch}</span>
                 ${_('to')}
-                <a style="${link_style}"
+                <a style="${link_text_style}"
                    href="${pr_target_repo}">${pr_target_repo}</a>
                 ${_('branch')}
                 <span style="${data_style}">${pr_target_branch}</span>:
diff --git a/kallithea/tests/models/test_dump_html_mails.ref.html b/kallithea/tests/models/test_dump_html_mails.ref.html
--- a/kallithea/tests/models/test_dump_html_mails.ref.html
+++ b/kallithea/tests/models/test_dump_html_mails.ref.html
@@ -111,7 +111,7 @@ View Comment: http://comment.org
         <td>
             <div>
                 Changeset on
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://example.com/repo_target">http://example.com/repo_target</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">brunch</span>:
@@ -270,7 +270,7 @@ View Comment: http://comment.org
         <td>
             <div>
                 Changeset on
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://example.com/repo_target">http://example.com/repo_target</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">brunch</span>:
@@ -447,7 +447,7 @@ View Comment: http://comment.org
         <td>
             <div>
                 Changeset on
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://example.com/repo_target">http://example.com/repo_target</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">brunch</span>:
@@ -624,7 +624,7 @@ View Comment: http://comment.org
         <td>
             <div>
                 Changeset on
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://example.com/repo_target">http://example.com/repo_target</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">brunch</span>:
@@ -957,12 +957,12 @@ View Pull Request: http://pr.org/7
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 at
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 at
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>
@@ -1146,12 +1146,12 @@ View Pull Request: http://pr.org/7
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 at
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 at
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>
@@ -1365,12 +1365,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -1530,12 +1530,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -1713,12 +1713,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -1896,12 +1896,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -2078,12 +2078,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -2260,12 +2260,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -2448,12 +2448,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>:
@@ -2636,12 +2636,12 @@ View Comment: http://pr.org/comment
         <td>
             <div>
                 Pull request from
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="https://dev.org/repo">https://dev.org/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">devbranch</span>
                 to
-                <a style="color:#395fa0;text-decoration:none"
+                <a style="color:#202020;text-decoration:none;border:#ddd 1px solid;background:#f9f9f9"
                    href="http://mainline.com/repo">http://mainline.com/repo</a>
                 branch
                 <span style="border:#ddd 1px solid;background:#f9f9f9">trunk</span>: