Refactor user blocks information to use a dl

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

diff --git a/app/views/user_blocks/show.html.erb b/app/views/user_blocks/show.html.erb
index 4dc03f4e487ec6645c48bc1e73c5a452cf3a6262..64f7299eff306aa122325e7929827fb0092ae1d9 100644 (file)
--- a/app/views/user_blocks/show.html.erb
+++ b/app/views/user_blocks/show.html.erb
@@ -1,38 +1,38 @@
-<% @title = t('.title',
+<% @title = t(".title",

               :block_on => @user_block.user.display_name,
               :block_by => @user_block.creator.display_name) %>
 
 <% content_for :heading do %>
               :block_on => @user_block.user.display_name,
               :block_by => @user_block.creator.display_name) %>
 
 <% content_for :heading do %>

-  <h1><%= raw t('.heading',
-                :block_on => link_to(
-                                     h(@user_block.user.display_name),
-                                     user_path(@user_block.user)),
-                :block_by => link_to(
-                                     h(@user_block.creator.display_name),
-                                     user_path(@user_block.creator))) %></h1>
+  <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>

 <ul class='secondary-actions clearfix'>
   <% if @user_block.ends_at > Time.now.getutc %>
     <% if current_user and current_user.id == @user_block.creator_id %>
 <ul class='secondary-actions clearfix'>
   <% if @user_block.ends_at > Time.now.getutc %>
     <% if current_user and current_user.id == @user_block.creator_id %>

-      <li><%= link_to t('.edit'), edit_user_block_path(@user_block) %></li>
+      <li><%= link_to t(".edit"), edit_user_block_path(@user_block) %></li>

     <% end %>
     <% end %>

-    <% if current_user and current_user.moderator? %>
-      <li><%= link_to(t('.revoke'),{:controller => 'user_blocks', :action => 'revoke', :id => @user_block.id}) %></li>
+    <% if can?(:revoke, UserBlock) %>
+      <li><%= link_to(t(".revoke"), :controller => "user_blocks", :action => "revoke", :id => @user_block.id) %></li>

     <% end %>
   <% end %>
     <% end %>
   <% end %>

-  <li><%= link_to t('.back'), user_blocks_path %></li>
+  <li><%= link_to t(".back"), user_blocks_path %></li>

 </ul>
 <% end %>
 
 </ul>
 <% end %>
 

-<% if @user_block.revoker %>
-<p>
-  <b><%= t'.revoker' %></b>
-  <%= link_to h(@user_block.revoker.display_name), user_path(@user_block.revoker) %>
-</p>
-<% end %>
-
-<p><b><%= t'.created' %></b>: <%= raw t'.ago', :time => friendly_date(@user_block.created_at) %></p>
-
-<p><b><%= t'.status' %></b>: <%= block_status(@user_block) %></p>
+<dl class="row">
+  <% if @user_block.revoker %>
+    <dt class="col-sm-3"><%= t ".revoker" %></dt>
+    <dd class="col-sm-9"><%= link_to @user_block.revoker.display_name, user_path(@user_block.revoker) %></dd>
+  <% end %>

 
 

-<p><b><%= t'.reason' %></b></p>
-<div class="richtext"><%= @user_block.reason.to_html %></div>
+  <dt class="col-sm-3"><%= t ".created" %></dt>
+  <dd class="col-sm-9"><%= friendly_date_ago(@user_block.created_at) %></dd>
+  <dt class="col-sm-3"><%= t ".duration" %></dt>
+  <dd class="col-sm-9"><%= block_duration_in_words(@user_block.ends_at - @user_block.created_at) %></dd>
+  <dt class="col-sm-3"><%= t ".status" %></dt>
+  <dd class="col-sm-9"><%= block_status(@user_block) %></dd>
+  <dt class="col-sm-3"><%= t ".reason" %></dt>
+  <dd class="col-sm-9"><div class="richtext text-break"><%= @user_block.reason.to_html %></div></dd>
+</dl>