From 01e7e4ade83026e78f3fe2737f99e2f4574baee1 Mon Sep 17 00:00:00 2001
From: Priti Shaw <34039705+PritiShaw@users.noreply.github.com>
Date: Mon, 25 Mar 2019 18:32:40 +0530
Subject: [PATCH] Fixed broken links

Added event listener to the links and added some custom styling for the same
---
 app/views/like/_like.html.erb | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/app/views/like/_like.html.erb b/app/views/like/_like.html.erb
index 07e714f373..6bf436d56e 100644
--- a/app/views/like/_like.html.erb
+++ b/app/views/like/_like.html.erb
@@ -73,12 +73,23 @@
 <hr style='clear:both;' />
 
 <div>
-  <p><a href='#' onClick='print_linkless()'><i class='fa fa-print'></i> Print without page links</a></p>
-  <p><a href='#' onClick='print_three_column();'><i class='fa fa-print'></i> Print in 3-column layout</a></p>
+  <p><a id='print-command-no-links'><i class='fa fa-print'></i> Print without page links</a></p>
+  <p><a id='print-command-3-col'><i class='fa fa-print'></i> Print in 3-column layout</a></p>
 </div>
 
-"><span class="caret"></span></li>
+
+">
+<span class="caret"></span></li>
   </ul>
+
+  <style type="text/css">
+    /*Styling the links to remove underline on hover and setting pointer as cursor*/
+    #print-command-3-col , #print-command-no-links {
+      text-decoration : none;
+      cursor : pointer;
+    }
+  </style>
+
   <script>
  
     function print_three_column() {
@@ -89,6 +100,10 @@
       $('body').css('column-count', 1)
                .css('column-gap', 0);
     }
+    // Event listner on CLICK on links
+    $(document).on("click", "#print-command-3-col",print_three_column);
+    $(document).on("click", "#print-command-no-links",print_linkless);
+
     jQuery(document).ready(function() {
       
       // 304 cached response yields no data to insert, which is not useful