Dressed up gpx notification mails

diff --git a/app/views/notifier/_gpx_description.html.erb b/app/views/notifier/_gpx_description.html.erb
index 2825799197f2a8a3f779c25b02be377342de087f..8d44336c9910715bbfd59f009b23cdd9a798e86f 100644 (file)
--- a/app/views/notifier/_gpx_description.html.erb
+++ b/app/views/notifier/_gpx_description.html.erb
@@ -1,14 +1,12 @@
-<%= t'notifier.gpx_notification.greeting' %>
-

 <%= t'notifier.gpx_notification.your_gpx_file' %>
 <%= t'notifier.gpx_notification.your_gpx_file' %>

-
-  <%= @trace_name %>
-
+<strong><%= @trace_name %></strong>

 <%= t'notifier.gpx_notification.with_description' %>
 <%= t'notifier.gpx_notification.with_description' %>

-
-  <%= @trace_description %>
+<em><%= @trace_description %></em>

 <% if @trace_tags.length>0 %>
 <% if @trace_tags.length>0 %>

-<%= t'notifier.gpx_notification.and_the_tags' %>
-<% @trace_tags.each do |tag| %>
-  <%= tag.tag.rstrip %><% end %><% else %>
-<%= t'notifier.gpx_notification.and_no_tags' %><% end %>
+  <%= t'notifier.gpx_notification.and_the_tags' %>
+  <em><% @trace_tags.each do |tag| %>
+    <%= tag.tag.rstrip %>
+  <% end %></em>
+<% else %>
+  <%= t'notifier.gpx_notification.and_no_tags' %>
+<% end %>

diff --git a/app/views/notifier/gpx_failure.html.erb b/app/views/notifier/gpx_failure.html.erb
index f59aa3dbbf58056853de136a5311c6b1360fd310..d1319acbd1286bff9e0f106aee025481c3747a50 100644 (file)
--- a/app/views/notifier/gpx_failure.html.erb
+++ b/app/views/notifier/gpx_failure.html.erb
@@ -1,9 +1,18 @@
-<%= render :partial => "gpx_description" %>
-<%= t'notifier.gpx_notification.failure.failed_to_import' %>
+<% content_for :body do %>
+  <p style="margin: 0"><%= t'notifier.gpx_notification.greeting' %></p>

 
 

-  <%= @error %>
+  <p>
+    <%= render :partial => "gpx_description" %>
+    <%= t'notifier.gpx_notification.failure.failed_to_import' %>
+  </p>

 
 

-<%= t'notifier.gpx_notification.failure.more_info_1' %>
-<%= t'notifier.gpx_notification.failure.more_info_2' %>
+  <blockquote>
+    <%= @error %>
+  </blockquote>

 
 

-  <%= t'notifier.gpx_notification.failure.import_failures_url' %>
+  <p>
+    <%= t'notifier.gpx_notification.failure.more_info_1' %>
+    <%= t'notifier.gpx_notification.failure.more_info_2' %>
+    <%= t'notifier.gpx_notification.failure.import_failures_url' %>
+  </p>
+<% end %>

diff --git a/app/views/notifier/gpx_success.html.erb b/app/views/notifier/gpx_success.html.erb
index 1983fe71c2e563d1ef58ff42c75fb7be1d8c0497..97267944e8776515923840ab705235774f992084 100644 (file)
--- a/app/views/notifier/gpx_success.html.erb
+++ b/app/views/notifier/gpx_success.html.erb
@@ -1,2 +1,8 @@
-<%= render :partial => "gpx_description" %>
-<%= t'notifier.gpx_notification.success.loaded_successfully', :trace_points => @trace_points, :possible_points => @possible_points %>
+<% content_for :body do %>
+  <p style="margin: 0"><%= t'notifier.gpx_notification.greeting' %></p>
+
+  <p>
+    <%= render :partial => "gpx_description" %>
+    <%= t'notifier.gpx_notification.success.loaded_successfully', :trace_points => @trace_points, :possible_points => @possible_points %>
+  </p>
+<% end %>