From ce054eed227c9f01d8278b80fcbd7e8fcca04355 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Wed, 29 Aug 2012 14:08:18 +0100
Subject: [PATCH] Improve handling of sidebar close link

---
 app/views/site/_sidebar.html.erb | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/views/site/_sidebar.html.erb b/app/views/site/_sidebar.html.erb
index 3b1b547b6..481f89795 100644
--- a/app/views/site/_sidebar.html.erb
+++ b/app/views/site/_sidebar.html.erb
@@ -2,7 +2,7 @@
   <table class="sidebar_title" width="100%">
     <tr>
       <td id="sidebar_title"><% t 'site.sidebar.search_results' %></td>
-      <td id="sidebar_close"><a href="javascript:closeSidebar()"><%= t 'site.sidebar.close' %></a></td>
+      <td id="sidebar_close"><a class="sidebar_close" href="#"><%= t 'site.sidebar.close' %></a></td>
     </tr>
   </table>
   <div id="sidebar_content">
@@ -33,7 +33,7 @@
     onclose = options.onclose;
   }
 
-  function closeSidebar() {
+  $(".sidebar_close").click(function (e) {
     $("#sidebar").css("display", "none");
 
     <%= onclose %>
@@ -42,7 +42,9 @@
        onclose();
        onclose = null;
     }
-  }
+
+    e.preventDefault();
+  });
 
   function updateSidebar(title, content) {
     $("#sidebar_title").html(title);
-- 
2.43.2