From 17efed0cc6cf658749b67468286d368f4d041cfa Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Mon, 23 Mar 2015 01:46:47 -0400 Subject: [PATCH] 5198: Update comment. --- apps/workbench/app/helpers/application_helper.rb | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/apps/workbench/app/helpers/application_helper.rb b/apps/workbench/app/helpers/application_helper.rb index 794ce1b77e..a80290cfa0 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. -- 2.39.5