You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Anna Vo (JIRA)" <ta...@jakarta.apache.org> on 2007/02/28 17:59:56 UTC
[jira] Created: (TAPESTRY-1309) clearValidationDecorations in
validation.js has huge performance issues with many form elements
clearValidationDecorations in validation.js has huge performance issues with many form elements
-----------------------------------------------------------------------------------------------
Key: TAPESTRY-1309
URL: https://issues.apache.org/jira/browse/TAPESTRY-1309
Project: Tapestry
Issue Type: Bug
Components: JavaScript
Affects Versions: 4.1.2
Environment: Win IE6, IE7, FF1.5(mac os x) and Safari(mac os x)
Reporter: Anna Vo
In validation.js this method causes performances issues in IE6, IE7, FF1.5(mac) and Safari if you have around 4000 or more checkboxes within the form (in the case of adding and removing category tree nodes). This was not an issue with Tapestry 4.0.2.
/**
* Clears out previous css classes set on fields
* in error.
*/
clearValidationDecorations:function(form, props){
for (var i=0; i< form.elements.length; i++) {
if (dj_undef("type", form.elements[i]) || typeof form.elements[i].type == "undefined"
|| form.elements[i].type == "submit"
|| form.elements[i].type == "hidden") { continue; }
dojo.html.removeClass(form.elements[i], this.missingClass);
dojo.html.removeClass(form.elements[i], this.invalidClass);
}
},
The performance issues that we found:
IE6/7 - browser locks up with hour glass for close to 3 minutes before it hits the listener method
Safari - browser locks up and eventually quits
FF (mac) - long running script processing prompt that asks to continue or stop script
When we commented this method out the performance issues went away. We also tried adding || form.elements[i].type == "checkbox" to the continue statement which worked but was still slower than usability testing allows.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org
[jira] Updated: (TAPESTRY-1309) clearValidationDecorations in
validation.js has huge performance issues with many form elements
Posted by "Jesse Kuhnert (JIRA)" <de...@tapestry.apache.org>.
[ https://issues.apache.org/jira/browse/TAPESTRY-1309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jesse Kuhnert updated TAPESTRY-1309:
------------------------------------
Fix Version/s: 4.1.2
> clearValidationDecorations in validation.js has huge performance issues with many form elements
> -----------------------------------------------------------------------------------------------
>
> Key: TAPESTRY-1309
> URL: https://issues.apache.org/jira/browse/TAPESTRY-1309
> Project: Tapestry
> Issue Type: Bug
> Components: JavaScript
> Affects Versions: 4.1.2
> Environment: Win IE6, IE7, FF1.5(mac os x) and Safari(mac os x)
> Reporter: Anna Vo
> Fix For: 4.1.2
>
>
> In validation.js this method causes performances issues in IE6, IE7, FF1.5(mac) and Safari if you have around 4000 or more checkboxes within the form (in the case of adding and removing category tree nodes). This was not an issue with Tapestry 4.0.2.
> /**
> * Clears out previous css classes set on fields
> * in error.
> */
> clearValidationDecorations:function(form, props){
> for (var i=0; i< form.elements.length; i++) {
> if (dj_undef("type", form.elements[i]) || typeof form.elements[i].type == "undefined"
> || form.elements[i].type == "submit"
> || form.elements[i].type == "hidden") { continue; }
>
> dojo.html.removeClass(form.elements[i], this.missingClass);
> dojo.html.removeClass(form.elements[i], this.invalidClass);
> }
> },
> The performance issues that we found:
> IE6/7 - browser locks up with hour glass for close to 3 minutes before it hits the listener method
> Safari - browser locks up and eventually quits
> FF (mac) - long running script processing prompt that asks to continue or stop script
> When we commented this method out the performance issues went away. We also tried adding || form.elements[i].type == "checkbox" to the continue statement which worked but was still slower than usability testing allows.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org
[jira] Resolved: (TAPESTRY-1309) clearValidationDecorations in
validation.js has huge performance issues with many form elements
Posted by "Jesse Kuhnert (JIRA)" <de...@tapestry.apache.org>.
[ https://issues.apache.org/jira/browse/TAPESTRY-1309?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jesse Kuhnert resolved TAPESTRY-1309.
-------------------------------------
Resolution: Fixed
Assignee: Jesse Kuhnert
I think this new method will be much quicker - but please of course re-open / file a new issue if it still isn't quite fast enough. (I got lazy and didn't actually create a sample form with thousands of input nodes but do know what I've done will be faster ...so we'll see )
> clearValidationDecorations in validation.js has huge performance issues with many form elements
> -----------------------------------------------------------------------------------------------
>
> Key: TAPESTRY-1309
> URL: https://issues.apache.org/jira/browse/TAPESTRY-1309
> Project: Tapestry
> Issue Type: Bug
> Components: JavaScript
> Affects Versions: 4.1.2
> Environment: Win IE6, IE7, FF1.5(mac os x) and Safari(mac os x)
> Reporter: Anna Vo
> Assigned To: Jesse Kuhnert
> Fix For: 4.1.2
>
>
> In validation.js this method causes performances issues in IE6, IE7, FF1.5(mac) and Safari if you have around 4000 or more checkboxes within the form (in the case of adding and removing category tree nodes). This was not an issue with Tapestry 4.0.2.
> /**
> * Clears out previous css classes set on fields
> * in error.
> */
> clearValidationDecorations:function(form, props){
> for (var i=0; i< form.elements.length; i++) {
> if (dj_undef("type", form.elements[i]) || typeof form.elements[i].type == "undefined"
> || form.elements[i].type == "submit"
> || form.elements[i].type == "hidden") { continue; }
>
> dojo.html.removeClass(form.elements[i], this.missingClass);
> dojo.html.removeClass(form.elements[i], this.invalidClass);
> }
> },
> The performance issues that we found:
> IE6/7 - browser locks up with hour glass for close to 3 minutes before it hits the listener method
> Safari - browser locks up and eventually quits
> FF (mac) - long running script processing prompt that asks to continue or stop script
> When we commented this method out the performance issues went away. We also tried adding || form.elements[i].type == "checkbox" to the continue statement which worked but was still slower than usability testing allows.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org