Putting the custom input styling back onto the checkboxes, with Histo…

…ry fixes in place
clearbold · Oct 2, 2014 · 74fac40 · 74fac40
1 parent d0b64f3
commit 74fac40
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 14 deletions.
diff --git a/_site/ui/js/scripts-concat.min.js b/_site/ui/js/scripts-concat.min.js
diff --git a/_source/ui/js/scripts-concat.js b/_source/ui/js/scripts-concat.js
@@ -24863,7 +24863,7 @@ Artbot.historyList = {
         });
     },
     bindAttendanceCheckboxes: function() {
-        //Artbot.customCheckboxes.init("#history-form");
+        Artbot.customCheckboxes.init("#history-form");
 
         // Set up click event for History Attendance checkboxes
         $("#history-form").find("input[type=checkbox]").click(function() {
@@ -24872,7 +24872,7 @@ Artbot.historyList = {
     },
     unbindAttendanceCheckboxes: function() {
 
-        //Artbot.customCheckboxes.destroy("#history-form");
+        Artbot.customCheckboxes.destroy("#history-form");
 
         // Remove click event for History Attendance checkboxes
         $("#history-form").find("input[type=checkbox]").unbind("click");
@@ -24951,8 +24951,7 @@ Artbot.historyList = {
                         var $thisCheckbox = $("#history-form").find("input[data-user-favorite-id=" + userFavoriteID + "]");
 
                         if ($thisCheckbox.prop("checked") !== true) {
-                            //$thisCheckbox.trigger("click");
-                            $thisCheckbox.prop("checked", true);
+                            $thisCheckbox.prop("checked", true).trigger("updateState");
                             console.log("Double-checking: is the checkbox checked as expected after the click event? " + $thisCheckbox.prop("checked"));
                         }
                     }
@@ -25162,7 +25161,6 @@ Artbot.interests = {
                     }
 
                     /* Make the actual Ajax request to handle the interest
-                    TODO: add success/fail/error handling, etc.
                     No Load More functionality, possibly a future enhancement. */
                     $.mobile.loading('show');
 

diff --git a/_source/ui/js/scripts-concat.min.js b/_source/ui/js/scripts-concat.min.js
diff --git a/_source/ui/js/scripts.js b/_source/ui/js/scripts.js
@@ -1875,7 +1875,7 @@ Artbot.historyList = {
         });
     },
     bindAttendanceCheckboxes: function() {
-        //Artbot.customCheckboxes.init("#history-form");
+        Artbot.customCheckboxes.init("#history-form");
 
         // Set up click event for History Attendance checkboxes
         $("#history-form").find("input[type=checkbox]").click(function() {
@@ -1884,7 +1884,7 @@ Artbot.historyList = {
     },
     unbindAttendanceCheckboxes: function() {
 
-        //Artbot.customCheckboxes.destroy("#history-form");
+        Artbot.customCheckboxes.destroy("#history-form");
 
         // Remove click event for History Attendance checkboxes
         $("#history-form").find("input[type=checkbox]").unbind("click");
@@ -1963,8 +1963,7 @@ Artbot.historyList = {
                         var $thisCheckbox = $("#history-form").find("input[data-user-favorite-id=" + userFavoriteID + "]");
 
                         if ($thisCheckbox.prop("checked") !== true) {
-                            //$thisCheckbox.trigger("click");
-                            $thisCheckbox.prop("checked", true);
+                            $thisCheckbox.prop("checked", true).trigger("updateState");
                             console.log("Double-checking: is the checkbox checked as expected after the click event? " + $thisCheckbox.prop("checked"));
                         }
                     }
@@ -2174,7 +2173,6 @@ Artbot.interests = {
                     }
 
                     /* Make the actual Ajax request to handle the interest
-                    TODO: add success/fail/error handling, etc.
                     No Load More functionality, possibly a future enhancement. */
                     $.mobile.loading('show');