[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