From e26da7dfa57da77144b22b4f2df20c95185678c8 Mon Sep 17 00:00:00 2001
From: karemarsy <karema@octangrp.com>
Date: Wed, 15 Apr 2020 13:05:21 +0200
Subject: [PATCH 1/5] Add some styling classes in the table

---
 app/views/forms/show.html.erb | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/app/views/forms/show.html.erb b/app/views/forms/show.html.erb
index 5842b30..cd0d10a 100644
--- a/app/views/forms/show.html.erb
+++ b/app/views/forms/show.html.erb
@@ -5,7 +5,11 @@
 <p>
   <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
 </p>
-<table>
+<div class="section">
+<div class="columns">
+<div class="column is-half is-offset-one-quarter">
+
+<table class="table is-fullwidth">
   <thead>
     <tr>
       <% @form.spreadsheet_headers.each do |value| %>
@@ -23,3 +27,6 @@
     <% end %>
   </tbody>
 </table>
+</div>
+</div>
+</div>
\ No newline at end of file

From 3b71f6b00f6f5ae716277b16ef46341b3a2efba2 Mon Sep 17 00:00:00 2001
From: Yannick <musayann@octangrp.com>
Date: Thu, 16 Apr 2020 18:20:06 +0200
Subject: [PATCH 2/5] Fix table styling

---
 app/assets/stylesheets/application.css.scss |  4 ++
 app/views/forms/show.html.erb               | 56 ++++++++++-----------
 2 files changed, 32 insertions(+), 28 deletions(-)

diff --git a/app/assets/stylesheets/application.css.scss b/app/assets/stylesheets/application.css.scss
index 71872c4..7955a05 100644
--- a/app/assets/stylesheets/application.css.scss
+++ b/app/assets/stylesheets/application.css.scss
@@ -52,4 +52,8 @@ body {
 .button.google {
   background: rgb(225, 98, 89);
   color: white;
+}
+
+.table-wrapper{
+  box-shadow: 0 1px 3px 0 rgba(0, 0, 0, 0.1), 0 1px 2px 0 rgba(0, 0, 0, 0.06);
 }
\ No newline at end of file
diff --git a/app/views/forms/show.html.erb b/app/views/forms/show.html.erb
index cd0d10a..64a82d1 100644
--- a/app/views/forms/show.html.erb
+++ b/app/views/forms/show.html.erb
@@ -1,32 +1,32 @@
-<h1><%= @form.title %></h1>
-<p>
-  <%= submission_url(@form) %>
-</p>
-<p>
-  <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
-</p>
-<div class="section">
-<div class="columns">
-<div class="column is-half is-offset-one-quarter">
+<div class="container">
+  <div class="content is-paddingless">
+    <h1><%= @form.title %></h1>
+    <p>
+      <%= submission_url(@form) %>
+    </p>
+    <p>
+      <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
+    </p>
+  </div>
 
-<table class="table is-fullwidth">
-  <thead>
-    <tr>
-      <% @form.spreadsheet_headers.each do |value| %>
-        <th><%= value %></th>
-      <% end %>
-    </tr>
-  </thead>
-  <tbody>
-    <% @submissions.each do |submission| %>
-      <tr>
-        <% @form.spreadsheet_headers.each do |column| %>
+  <div class="box table-wrapper">
+    <table class="table is-fullwidth is-strip is-hoverable">
+      <thead>
+        <tr>
+          <% @form.spreadsheet_headers.each do |value| %>
+          <th><%= value %></th>
+          <% end %>
+        </tr>
+      </thead>
+      <tbody>
+        <% @submissions.each do |submission| %>
+        <tr>
+          <% @form.spreadsheet_headers.each do |column| %>
           <td><%= submission.data[column] %></td>
+          <% end %>
+        </tr>
         <% end %>
-      </tr>
-    <% end %>
-  </tbody>
-</table>
-</div>
-</div>
+      </tbody>
+    </table>
+  </div>
 </div>
\ No newline at end of file

From 5ce78138d2d991108d9b661804498f8347a6c1c3 Mon Sep 17 00:00:00 2001
From: Yannick <musayann@octangrp.com>
Date: Thu, 16 Apr 2020 18:44:31 +0200
Subject: [PATCH 3/5] Add link styling

---
 app/assets/stylesheets/application.css.scss |  6 ++++-
 app/views/forms/show.html.erb               | 30 ++++++++++++++++-----
 2 files changed, 28 insertions(+), 8 deletions(-)

diff --git a/app/assets/stylesheets/application.css.scss b/app/assets/stylesheets/application.css.scss
index 7955a05..8405ae7 100644
--- a/app/assets/stylesheets/application.css.scss
+++ b/app/assets/stylesheets/application.css.scss
@@ -54,6 +54,10 @@ body {
   color: white;
 }
 
-.table-wrapper{
+.button {
+  vertical-align: middle;
+}
+
+.table-wrapper {
   box-shadow: 0 1px 3px 0 rgba(0, 0, 0, 0.1), 0 1px 2px 0 rgba(0, 0, 0, 0.06);
 }
\ No newline at end of file
diff --git a/app/views/forms/show.html.erb b/app/views/forms/show.html.erb
index 64a82d1..666faac 100644
--- a/app/views/forms/show.html.erb
+++ b/app/views/forms/show.html.erb
@@ -1,12 +1,28 @@
 <div class="container">
   <div class="content is-paddingless">
-    <h1><%= @form.title %></h1>
-    <p>
-      <%= submission_url(@form) %>
-    </p>
-    <p>
-      <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
-    </p>
+    <h1 class="is-family-secondary"><%= @form.title %></h1>
+    <div class="columns">
+      <div class="column">
+        <p>
+          <button class="button">
+            <span class="icon is-small">
+              <i class="fas fa-link"></i>
+            </span>
+          </button>
+          <%= submission_url(@form) %>
+        </p>
+      </div>
+      <div class="column">
+        <p>
+          <button class="button">
+            <span class="icon is-small">
+              <i class="fas fa-file-excel"></i>
+            </span>
+          </button>
+          <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
+        </p>
+      </div>
+    </div>
   </div>
 
   <div class="box table-wrapper">

From d3f6ff9ce34ab060cbe836b5abc0b49e78536cb0 Mon Sep 17 00:00:00 2001
From: karemarsy <karema@octangrp.com>
Date: Mon, 20 Apr 2020 10:44:33 +0200
Subject: [PATCH 4/5] Redesign the table

---
 app/views/forms/show.html.erb | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/views/forms/show.html.erb b/app/views/forms/show.html.erb
index 666faac..0d6af67 100644
--- a/app/views/forms/show.html.erb
+++ b/app/views/forms/show.html.erb
@@ -1,3 +1,4 @@
+<div class="section">
 <div class="container">
   <div class="content is-paddingless">
     <h1 class="is-family-secondary"><%= @form.title %></h1>
@@ -14,12 +15,12 @@
       </div>
       <div class="column">
         <p>
-          <button class="button">
+          <button class="button is-success">
             <span class="icon is-small">
               <i class="fas fa-file-excel"></i>
             </span>
           </button>
-          <%= link_to 'Google spreadsheet', @form.spreadsheet_url %>
+          <%= link_to 'Google spreadsheet', @form.spreadsheet_url, { :class => "has-text-dark"} %>
         </p>
       </div>
     </div>
@@ -45,4 +46,5 @@
       </tbody>
     </table>
   </div>
+</div>
 </div>
\ No newline at end of file

From 0b33a5d981b9e1076b78789e18798f9523e355c3 Mon Sep 17 00:00:00 2001
From: Michael Bumann <hello@michaelbumann.com>
Date: Mon, 20 Apr 2020 11:35:31 +0200
Subject: [PATCH 5/5] Update app/views/forms/show.html.erb

---
 app/views/forms/show.html.erb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/forms/show.html.erb b/app/views/forms/show.html.erb
index 0d6af67..7c103b4 100644
--- a/app/views/forms/show.html.erb
+++ b/app/views/forms/show.html.erb
@@ -27,7 +27,7 @@
   </div>
 
   <div class="box table-wrapper">
-    <table class="table is-fullwidth is-strip is-hoverable">
+    <table class="table is-fullwidth is-striped is-hoverable">
       <thead>
         <tr>
           <% @form.spreadsheet_headers.each do |value| %>
@@ -47,4 +47,4 @@
     </table>
   </div>
 </div>
-</div>
\ No newline at end of file
+</div>