From 27cbd4e1b106c79b78dafb65594daa7a5412c7df Mon Sep 17 00:00:00 2001
From: Simon <mons.web.solution@gmail.com>
Date: Tue, 24 Apr 2018 20:21:28 +0200
Subject: [PATCH] Various admin style fixes (#937)

- remove clipboard icons #867
- cards overflow and breakword #929
- deposit details ui #928
---
 app/assets/stylesheets/admin.scss             |  8 ++++++++
 app/helpers/application_helper.rb             |  2 +-
 app/views/admin/deposits/fiats/show.html.erb  | 12 ++++++------
 app/views/admin/withdraws/fiats/show.html.erb | 12 +++---------
 app/views/shared/admin/_account.html.erb      |  2 +-
 5 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/app/assets/stylesheets/admin.scss b/app/assets/stylesheets/admin.scss
index ec6ec63d1e..a17021a7d7 100644
--- a/app/assets/stylesheets/admin.scss
+++ b/app/assets/stylesheets/admin.scss
@@ -27,6 +27,14 @@
     flex-direction: row-reverse;
     text-align: right;
     font-weight: bold;
+    max-width: 30%;
+  }
+  dd {
+    margin-bottom: 0;
+    span {
+      width: 100%;
+      word-wrap: break-word;
+    }
   }
 }
 
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 068d438b9e..81a0b54164 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -175,7 +175,7 @@ def language_path(lang=nil)
   end
 
   def description_for(name, &block)
-    content_tag :div, class: "dl-#{name} dl-horizontal" do
+    content_tag :dl, class: "dl-#{name} dl-horizontal" do
       capture(&block)
     end
   end
diff --git a/app/views/admin/deposits/fiats/show.html.erb b/app/views/admin/deposits/fiats/show.html.erb
index 78d113e5b9..e0a2d987f3 100644
--- a/app/views/admin/deposits/fiats/show.html.erb
+++ b/app/views/admin/deposits/fiats/show.html.erb
@@ -1,16 +1,16 @@
 <div class="row">
-  <div class="col-6">
+  <div class="col-md-4">
     <div class="card card-primary">
       <div class="card-header">
         <h3 class="card-title">Deposit details</h3>
       </div>
       <div class="card-body">
         <%= description_for :member do %>
-          <b>ID</b>: <%= @deposit.sn %><br>
-          <b>Member</b>: <%= @deposit.member.email %><br>
-          <b>Amount</b>: <%= number_with_precision(@deposit.amount, precision: 2) %><br>
-          <b>Created at</b>: <%= t(@deposit.created_at, format: :long) %><br>
-          <b>State</b>: <%= @deposit.aasm_state_text %>
+          <dt>ID</dt> <dd><%= @deposit.sn %></dd>
+          <dt>Member</dt> <dd><%= @deposit.member.email %></dd>
+          <dt>Amount</dt> <dd><%= number_with_precision(@deposit.amount, precision: 2) %></dd>
+          <dt>Created at</dt> <dd><%= t(@deposit.created_at, format: :long) %></dd>
+          <dt>State</dt> <dd><%= @deposit.aasm_state_text %></dd>
         <% end %>
       </div>
       <% if @deposit.may_accept? %>
diff --git a/app/views/admin/withdraws/fiats/show.html.erb b/app/views/admin/withdraws/fiats/show.html.erb
index 53a433341a..1e719bedfc 100644
--- a/app/views/admin/withdraws/fiats/show.html.erb
+++ b/app/views/admin/withdraws/fiats/show.html.erb
@@ -12,19 +12,13 @@
           <%= item_for @withdraw, :aasm_state_text %>
           <hr class="split">
           <%= item_for @withdraw.member, :email do %>
-            <span class="fa fa-copy" data-clipboard-text="<%= @withdraw.member.email %>">
-              <%= @withdraw.member.email %>
-            </span>
+            <%= @withdraw.member.email %>
           <% end %>
           <%= item_for @withdraw, 'Recipient ID' do %>
-            <span class="fa fa-copy" data-clipboard-text="<%= @withdraw.rid %>">
-              <%= @withdraw.rid %>
-            </span>
+            <%= @withdraw.rid %>
           <% end %>
           <%= item_for @withdraw, :amount do %>
-            <span class="fa fa-copy" data-clipboard-text="<%= @withdraw.amount %>">
-              <%= @withdraw.amount %>
-            </span>
+            <%= @withdraw.amount %>
           <% end %>
           <ul class="list-inline w-100 px-3 mt-3 pt-3 border-top">
             <% if @withdraw.may_accept? || @withdraw.may_process? || @withdraw.may_success? %>
diff --git a/app/views/shared/admin/_account.html.erb b/app/views/shared/admin/_account.html.erb
index 6e387bad6a..031df9ffcb 100644
--- a/app/views/shared/admin/_account.html.erb
+++ b/app/views/shared/admin/_account.html.erb
@@ -5,7 +5,7 @@
     <span><%= account.currency.code.upcase %></span>
   </div>
   <div class="card-body">
-    <dl>
+    <dl class="dl-horizontal">
       <%= item_for account, :amount %>
       <%= item_for account, :locked %>
       <%= item_for account, :balance %>