You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by an...@apache.org on 2007/09/28 18:55:17 UTC
svn commit: r580416 -
/tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js
Author: andyhot
Date: Fri Sep 28 09:55:17 2007
New Revision: 580416
URL: http://svn.apache.org/viewvc?rev=580416&view=rev
Log:
slight change to the implementation of exception display handling
Modified:
tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js
Modified: tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js?rev=580416&r1=580415&r2=580416&view=diff
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js (original)
+++ tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js Fri Sep 28 09:55:17 2007
@@ -604,21 +604,24 @@
var elms=dojo.html.getElementsByClass('toggle');
if(elms && elms.length > 0){
for(var i=0;i<elms.length;i++){
- elms[i].onclick = function() {
- dojo.html.toggleShowing(dojo.byId(this.id + 'Data'));
- if(dojo.html.hasClass(this, "toggleSelected"))
- dojo.html.removeClass(this, "toggleSelected")
+ dojo.event.connect(elms[i], "onclick", function(e) {
+ var thisLink = e.target;
+ dojo.html.toggleShowing(dojo.byId(thisLink.id + 'Data'));
+ if(dojo.html.hasClass(thisLink, "toggleSelected"))
+ dojo.html.removeClass(thisLink, "toggleSelected")
else
- dojo.html.addClass(this, "toggleSelected");
+ dojo.html.addClass(thisLink, "toggleSelected");
+ if (e.preventDefault)
+ dojo.event.browser.stopEvent(e);
return false;
- };
+ });
dojo.html.toggleShowing(elms[i].id+'Data');
}
}
// but show last exception's content
elms=dojo.html.getElementsByClass('exception-link');
if(elms && elms.length > 0){
- elms[elms.length-1].onclick();
+ elms[elms.length-1].onclick({target:elms[elms.length-1]});
}
},