Use jQuery's toggle binding
authorJohn Firebaugh <john.firebaugh@gmail.com>
Fri, 12 Oct 2012 00:27:35 +0000 (17:27 -0700)
committerJohn Firebaugh <john.firebaugh@gmail.com>
Fri, 9 Nov 2012 20:59:27 +0000 (12:59 -0800)
The old code never unbound the existing binding, so they
piled up.

app/assets/javascripts/index/browse.js

index 1792c85..e75ae9d 100644 (file)
@@ -52,8 +52,7 @@ $(document).ready(function () {
     $("#browse_select_box").click(startDrag);
 
     $("#browse_hide_areas_box").html(I18n.t('browse.start_rjs.hide_areas'));
-    $("#browse_hide_areas_box").show();
-    $("#browse_hide_areas_box").click(hideAreas);
+    $("#browse_hide_areas_box").toggle(hideAreas, showAreas);
 
     function updateData() {
       if (browseMode == "auto") {
@@ -128,8 +127,6 @@ $(document).ready(function () {
 
     function hideAreas() {
       $("#browse_hide_areas_box").html(I18n.t('browse.start_rjs.show_areas'));
-      $("#browse_hide_areas_box").show();
-      $("#browse_hide_areas_box").click(showAreas);
 
       areasHidden = true;
 
@@ -138,8 +135,6 @@ $(document).ready(function () {
 
     function showAreas() {
       $("#browse_hide_areas_box").html(I18n.t('browse.start_rjs.hide_areas'));
-      $("#browse_hide_areas_box").show();
-      $("#browse_hide_areas_box").click(hideAreas);
 
       areasHidden = false;