diff options
Diffstat (limited to 'app/views/notify/new_review_email.html.haml')
-rw-r--r-- | app/views/notify/new_review_email.html.haml | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/app/views/notify/new_review_email.html.haml b/app/views/notify/new_review_email.html.haml index 11da7723d8d..afc1bd68215 100644 --- a/app/views/notify/new_review_email.html.haml +++ b/app/views/notify/new_review_email.html.haml @@ -1,3 +1,7 @@ +- if @include_diff_discussion_stylesheet + = content_for :head do + = stylesheet_link_tag 'mailers/highlighted_diff_email' + %table{ border: "0", cellpadding: "0", cellspacing: "0", style: "width:100%;margin:0 auto;border-collapse:separate;border-spacing:0;" } %tbody %tr @@ -5,12 +9,16 @@ %table{ border: "0", cellpadding: "0", cellspacing: "0", style: "width:100%;border-collapse:separate;border-spacing:0;" } %tbody %tr - %td{ style: "color:#333333;border-bottom:1px solid #ededed;font-size:15px;font-weight:bold;line-height:1.4;padding: 20px 0;" } + %td{ style: "color:#333333;border-bottom:1px solid #ededed;font-weight:bold;line-height:1.4;padding: 20px 0;" } - mr_link = link_to(@merge_request.to_reference(@project), project_merge_request_url(@project, @merge_request)) - mr_author_link = link_to(@author_name, user_url(@author)) = _('Merge request %{mr_link} was reviewed by %{mr_author}').html_safe % { mr_link: mr_link, mr_author: mr_author_link } %tr - %td{ style: "overflow:hidden;font-size:14px;line-height:1.4;display:grid;" } + %td{ style: "overflow:hidden;line-height:1.4;display:grid;" } - @notes.each do |note| + -# Get preloaded note discussion + - discussion = @discussions[note.discussion_id] if note.part_of_discussion? + -# Preload project for discussions first note + - discussion.first_note.project = @project if discussion&.first_note - target_url = project_merge_request_url(@project, @merge_request, anchor: "note_#{note.id}") - = render 'note_email', note: note, diff_limit: 3, target_url: target_url, note_style: "border-bottom:1px solid #ededed;" + = render 'note_email', note: note, diff_limit: 3, target_url: target_url, note_style: "border-bottom:1px solid #ededed; padding-bottom: 1em;", include_stylesheet_link: false, discussion: discussion, author: @author |