[ARVADOS] updated: 17efed0cc6cf658749b67468286d368f4d041cfa

git at public.curoverse.com git at public.curoverse.com
Mon Mar 23 01:50:44 EDT 2015


Summary of changes:
 apps/workbench/app/helpers/application_helper.rb | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

       via  17efed0cc6cf658749b67468286d368f4d041cfa (commit)
      from  165a594bf8606864c62f86405e318c68c2426c38 (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 17efed0cc6cf658749b67468286d368f4d041cfa
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon Mar 23 01:46:47 2015 -0400

    5198: Update comment.

diff --git a/apps/workbench/app/helpers/application_helper.rb b/apps/workbench/app/helpers/application_helper.rb
index 794ce1b..a80290c 100644
--- a/apps/workbench/app/helpers/application_helper.rb
+++ b/apps/workbench/app/helpers/application_helper.rb
@@ -52,11 +52,12 @@ module ApplicationHelper
     ArvadosBase::resource_class_for_uuid(attrvalue, opts)
   end
 
-  # When using {remote:true} or {method:...}, move the target URI from
-  # href to data-remote-href. Otherwise, browsers offer features like
-  # "open in new window" and "copy link address" which bypass Rails'
-  # click handler and therefore end up at incorrect/nonexistent routes
-  # (by ignoring data-method) and expect to receive pages rather than
+  # When using {remote:true}, or using {method:...} to use an HTTP
+  # method other than GET, move the target URI from href to
+  # data-remote-href. Otherwise, browsers offer features like "open in
+  # new window" and "copy link address" which bypass Rails' click
+  # handler and therefore end up at incorrect/nonexistent routes (by
+  # ignoring data-method) and expect to receive pages rather than
   # javascript responses.
   #
   # See assets/javascripts/link_to_remote.js for supporting code.

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list