Permalink
Browse files

BUGFIX Fixed drag link event handling on ComplexTableField.js (#4737)…

… (from r93754)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@93940 467b73ca-7a2a-4603-9d3b-597d59a354a9
  • Loading branch information...
1 parent a0092fb commit 22bc6827aeed14912d76f59c9783716bd33758e2 @chillu chillu committed Nov 30, 2009
Showing with 8 additions and 1 deletion.
  1. +8 −1 javascript/ComplexTableField.js
@@ -20,6 +20,13 @@ ComplexTableField.prototype = {
if(defaultAction) {
rules['#'+this.id+' table.data tbody td'] = {
onclick: function(e) {
+ var elt = Event.element(e);
+ // Check the tag, as otherwise this
+ // function can take over checkbox
+ // click actions etc. See ticket #4737
+ if (elt.tagName != 'TD' && elt.tagName != 'TR') {
+ return;
+ }
var link = $$('.'+defaultAction, Event.element(e).parentNode)[0].href;
this.openPopup(null, link);
return false;
@@ -133,4 +140,4 @@ ComplexTableField.prototype = {
}
}
-ComplexTableField.applyTo('div.ComplexTableField');
+ComplexTableField.applyTo('div.ComplexTableField');

0 comments on commit 22bc682

Please sign in to comment.