From 6f1e7abeddf3e4eebfe34dd840d030f2f7ca906b Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Tue, 27 Nov 2018 17:10:17 +0100
Subject: [PATCH] Improves styles for milestones documents

---
 app/assets/stylesheets/layout.scss            |  2 +-
 .../milestones/_milestone.html.erb            | 29 ++++++++++++-------
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/app/assets/stylesheets/layout.scss b/app/assets/stylesheets/layout.scss
index 7424e7ee9..7c9c71964 100644
--- a/app/assets/stylesheets/layout.scss
+++ b/app/assets/stylesheets/layout.scss
@@ -2523,7 +2523,7 @@ table {
     border-radius: rem-calc(5);
     display: block;
     margin: $line-height / 2 0;
-    padding: 0 $line-height / 2;
+    padding: $line-height / 2;
     position: relative;
 
     .icon-document {
diff --git a/app/views/budgets/investments/milestones/_milestone.html.erb b/app/views/budgets/investments/milestones/_milestone.html.erb
index 7f71d0f29..d534002ca 100644
--- a/app/views/budgets/investments/milestones/_milestone.html.erb
+++ b/app/views/budgets/investments/milestones/_milestone.html.erb
@@ -29,17 +29,24 @@
     </p>
 
     <% if milestone.documents.present? %>
-      <div class="document-link text-center">
-        <p>
-          <span class="icon-document"></span>&nbsp;
-          <strong><%= t("shared.documentation") %></strong>
-        </p>
-        <% milestone.documents.each do |document| %>
-          <%= link_to document.title,
-                  document.attachment.url,
-                  target: "_blank",
-                  rel: "nofollow" %><br>
-        <% end %>
+      <div class="documents">
+        <div class="document-link text-left small">
+          <p>
+            <strong><%= t("shared.documentation") %></strong>
+          </p>
+
+          <% milestone.documents.each do |document| %>
+              <%= link_to document.title,
+                      document.attachment.url,
+                      target: "_blank",
+                      rel: "nofollow" %><br>
+              <small>
+                <%= document.humanized_content_type %>&nbsp;|&nbsp;
+                <%= number_to_human_size(document.attachment_file_size, precision: 2) %>
+              </small>
+              <br>
+          <% end %>
+        </div>
       </div>
     <% end %>