Avoid various uses of html_safe

[rails.git] / app / views / issues / _comments.html.erb

diff --git a/app/views/issues/_comments.html.erb b/app/views/issues/_comments.html.erb
index 7ff0948db5e961fa53379ebea39e50c166aeb901..a0402230421ad894ee674fd4758bb4baf18b68ce 100644 (file)
--- a/app/views/issues/_comments.html.erb
+++ b/app/views/issues/_comments.html.erb
@@ -1,20 +1,20 @@
 <div class="issue-comments">
   <% comments.each do |comment| %>
     <div class="comment">
 <div class="issue-comments">
   <% comments.each do |comment| %>
     <div class="comment">

-      <div style="float:left">
+      <div class="float-left">

         <%= link_to user_thumbnail(comment.user), user_path(comment.user) %>
       </div>
         <%= link_to user_thumbnail(comment.user), user_path(comment.user) %>
       </div>

-      <b> <%= link_to comment.user.display_name, user_path(comment.user) %> </b> <br />
-      <%= comment.body %>
+      <p class="text-muted mb-0">
+        <%= t ".comment_from_html", :user_link => link_to(comment.user.display_name, user_path(comment.user)),
+                                    :comment_created_at => l(comment.created_at.to_datetime, :format => :friendly) %>
+      </p>
+      <p><%= comment.body %></p>

     </div>
     </div>

-    <span class="deemphasize">
-      <%= t(".created_at", :datetime => l(comment.created_at.to_datetime, :format => :friendly)) %>
-    </span>

     <hr>
   <% end %>
 </div>
 <br />
     <hr>
   <% end %>
 </div>
 <br />

-<div class="comment">
+<div class="comment standard-form">

   <%= form_for @new_comment, :url => issue_comments_path(@issue) do |f| %>
   <%= richtext_area :issue_comment, :body, :cols => 10, :rows => 8, :required => true %>
   <%= label_tag :reassign, t(".reassign_param") %> <%= check_box_tag :reassign, true %>
   <%= form_for @new_comment, :url => issue_comments_path(@issue) do |f| %>
   <%= richtext_area :issue_comment, :body, :cols => 10, :rows => 8, :required => true %>
   <%= label_tag :reassign, t(".reassign_param") %> <%= check_box_tag :reassign, true %>