[ARVADOS] updated: 6bc895c1aa2015562b0ad98f2905cdd2a5fc0a38

git at public.curoverse.com git at public.curoverse.com
Tue Dec 2 23:21:01 EST 2014


Summary of changes:
 apps/workbench/app/views/collections/_show_files.html.erb | 15 ++-------------
 1 file changed, 2 insertions(+), 13 deletions(-)

       via  6bc895c1aa2015562b0ad98f2905cdd2a5fc0a38 (commit)
      from  948b853e79639e9c1a9e3b6ef098819f4c16f54d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 6bc895c1aa2015562b0ad98f2905cdd2a5fc0a38
Author: radhika <radhika at curoverse.com>
Date:   Tue Dec 2 23:12:28 2014 -0500

    4525: update select_all and unselect_all functions to no longer use each.

diff --git a/apps/workbench/app/views/collections/_show_files.html.erb b/apps/workbench/app/views/collections/_show_files.html.erb
index 39a1b36..43c0e24 100644
--- a/apps/workbench/app/views/collections/_show_files.html.erb
+++ b/apps/workbench/app/views/collections/_show_files.html.erb
@@ -1,21 +1,10 @@
 <script>
-// The "each" loop in select_all_files() and unselect_all_files()
-// is needed because .trigger("change") does not play well with clippy.
-// Once clippy has been retired, we should be able to compress this
-// into .filter(":visible").prop("checked", true).trigger("change").
-//
 function select_all_files() {
-  $("#collection_files :checkbox").filter(":visible").each(
-    function() {
-      $(this).prop("checked", true).trigger("change");
-    });
+  $("#collection_files :checkbox").filter(":visible").prop("checked", true).trigger("change");
 }
 
 function unselect_all_files() {
-  $("#collection_files :checkbox").filter(":visible").each(
-    function() {
-      $(this).prop("checked", false).trigger("change");
-    });
+  $("#collection_files :checkbox").filter(":visible").prop("checked", false).trigger("change");
 }
 </script>
 

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list