Simplify link_to user calls on block pages

[rails.git] / app / views / user_blocks / revoke.html.erb

diff --git a/app/views/user_blocks/revoke.html.erb b/app/views/user_blocks/revoke.html.erb
index df48747dd2bff4e9a452191b2a7a6ea048c6aad6..6bf87c4c3cce00a12c11df92e4650e4416431a11 100644 (file)
--- a/app/views/user_blocks/revoke.html.erb
+++ b/app/views/user_blocks/revoke.html.erb
@@ -1,33 +1,31 @@
 <% @title = t(".title",
-              :block_on => h(@user_block.user.display_name),
-              :block_by => h(@user_block.creator.display_name)) %>
+              :block_on => @user_block.user.display_name,
+              :block_by => @user_block.creator.display_name) %>
 
 <% content_for :heading do %>
   <h1><%= t(".heading_html",
-            :block_on => link_to(@user_block.user.display_name,
-                                 user_path(@user_block.user)),
-            :block_by => link_to(@user_block.creator.display_name,
-                                 user_path(@user_block.creator))) %></h1>
+            :block_on => link_to(@user_block.user.display_name, @user_block.user),
+            :block_by => link_to(@user_block.creator.display_name, @user_block.creator)) %></h1>
 <% end %>
 
 <% if @user_block.ends_at > Time.now %>
-<p><b>
-  <%= t(".time_future", :time => distance_of_time_in_words_to_now(@user_block.ends_at)) %>
-</b></p>
+  <p>
+    <%= t(".time_future_html", :time => friendly_date(@user_block.ends_at)) %>
+  </p>
 
-<%= form_for :revoke, :url => { :action => "revoke" } do |f| %>
-  <%= f.error_messages %>
-<p>
-  <%= check_box_tag "confirm", "yes" %>
-  <%= label_tag "confirm", t(".confirm") %>
-</p>
-<p>
-  <%= submit_tag t(".revoke") %>
-</p>
-<% end %>
+  <%= bootstrap_form_for :revoke, :url => { :action => "revoke" } do |f| %>
+    <div class="mb-3">
+      <div class="form-check">
+        <%= check_box_tag "confirm", "yes", false, { :class => "form-check-input" } %>
+        <%= label_tag "confirm", t(".confirm"), { :class => "form-check-label" } %>
+      </div>
+    </div>
+
+    <%= f.primary t(".revoke") %>
+  <% end %>
 
 <% else %>
-<p>
-  <%= t(".past", :time => time_ago_in_words(@user_block.ends_at, :scope => :'datetime.distance_in_words_ago')) %>
-</p>
+  <p>
+    <%= t(".past_html", :time => friendly_date_ago(@user_block.ends_at)) %>
+  </p>
 <% end %>