Issue Details (XML | Word | Printable)

Key: STR-2826
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Don Brown
Reporter: Zarar Siddiqi
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Struts 1

If a field's display property is set to 'none' using CSS or JavaScript, the focus code fails.

Created: 08/Apr/06 04:44 AM   Updated: 28/Apr/06 11:48 PM
Return to search
Component/s: Tag Libraries
Affects Version/s: 1.2.9
Fix Version/s: 1.3.3

File Attachments:
  Size
Text File focusControl_css_dispay_bug_fix.patch 2006-04-08 04:44 AM Zarar Siddiqi 0.2 kB
Environment:
Operating System: other
Platform: Other

Bugzilla Id: 39249


 Description  « Hide
If a field's display property is set to 'none' using CSS or JavaScript, the
current code still tries to give the field focus which will result in an error
in IE.

A patch is for the org.apache.struts.taglib.html.FormTag.java file and is attached:

663c663
< " if (focusControl.type != \"hidden\" && !focusControl.disabled) {");
---
> " if (focusControl.type != \"hidden\" && !focusControl.disabled
&& focusControl.style.display != \"none\") {");

 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Don Brown made changes - 23/Apr/06 04:52 AM
Field Original Value New Value
issue.field.bugzillaimportkey 39249 28117
Don Brown made changes - 28/Apr/06 11:59 AM
Status Open [ 1 ] Resolved [ 5 ]
Assignee Struts Developer Mailing List [ dev@struts.apache.org ] Don Brown [ mrdon ]
Fix Version/s 1.3.3 [ 21710 ]
Resolution Fixed [ 1 ]
Zarar Siddiqi made changes - 28/Apr/06 11:48 PM
Status Resolved [ 5 ] Closed [ 6 ]