You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2009/04/18 07:01:47 UTC
svn commit: r766212 -
/wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js
Author: ivaynberg
Date: Sat Apr 18 05:01:47 2009
New Revision: 766212
URL: http://svn.apache.org/viewvc?rev=766212&view=rev
Log:
speed up IE for focus handling :|
Modified:
wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js
Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js?rev=766212&r1=766211&r2=766212&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/ajax/wicket-ajax.js Sat Apr 18 05:01:47 2009
@@ -2152,7 +2152,11 @@
setFocusOnElements: function (elements)
{
- for (var i=0; i< elements.length; i++)
+ // we need to cache array length because IE will try to recalculate
+ // the collection of elements every time length() is called which can be quiet expensive
+ // if the collection is a result of getElementsByTagName or a similar function.
+ var len=elements.length;
+ for (var i=0; i< len; i++)
{
if (elements[i].wicketFocusSet != true)
{