From c8f314142c194272ea2ae7d58f3f8fad724ef925 Mon Sep 17 00:00:00 2001
From: Bilal <bilal@saburly.com>
Date: Wed, 1 Jul 2020 10:57:55 +0200
Subject: [PATCH] fix MR comments

---
 app/assets/stylesheets/application.scss      |  9 ---------
 app/views/contract_templates/splash.html.erb | 12 +++++++-----
 2 files changed, 7 insertions(+), 14 deletions(-)

diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss
index 581f952..2942959 100644
--- a/app/assets/stylesheets/application.scss
+++ b/app/assets/stylesheets/application.scss
@@ -426,13 +426,4 @@ a[data-behavior=seekable-timecode] {
 .fix-h-and-w {
   width: 308px;
   height:308px;
-}
-
-// Add checkmark before list item
-#checkmark-list > li {
-  list-style: none;
-  margin-left: -1em;
-}
-#checkmark-list > li:before {
-  content: '\2713\0020';
 }
\ No newline at end of file
diff --git a/app/views/contract_templates/splash.html.erb b/app/views/contract_templates/splash.html.erb
index 20b8b04..f2ef542 100644
--- a/app/views/contract_templates/splash.html.erb
+++ b/app/views/contract_templates/splash.html.erb
@@ -5,7 +5,9 @@
   </div>
 
   <%= link_to t(".actions.book_demo"), 'https://meetings.hubspot.com/bray2', class: "btn btn-primary border align-self-center h-50 ml-auto mr-2 pb-2", target: '_blank' %>
-  <%= link_to t(".actions.create_template"), [:new, @project, :contract_template], class: "btn btn-success border align-self-center h-50 pb-2" %>
+  <% if policy(ContractTemplate).new? %>
+    <%= link_to t(".actions.create_template"), [:new, @project, :contract_template], class: "btn btn-success border align-self-center h-50 pb-2" %>
+<% end %>
 </div>
 
 <hr>
@@ -44,10 +46,10 @@
       <div class="row">
         <div class="col">
           <h3><%= t '.headings.benefits' %></h3>
-          <ul id="checkmark-list">
-            <li><%= t '.list_items.all_releases_available' %></li>
-            <li><%= t '.list_items.manage_large_audience' %></li>
-            <li><%= t '.list_items.add_tags_and_notes' %></li>
+          <ul class="fa-ul ml-5">
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.all_releases_available'))) %>
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.manage_large_audience'))) %>
+            <%= content_tag(:li, fa_icon("check li", text: t('.list_items.add_tags_and_notes'))) %>
           </ul>
         </div>
       </div>