[ https://issues.apache.org/struts/browse/STR-3006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_40258 ]
Frank W. Zammetti commented on STR-3006: ---------------------------------------- I understand what your saying Niall... I just have to think that if the documentation is clear, you've done about all you can, and if a developer makes a mistake... well, at some point that has to fall in the category of their problem. I think a framework can (and should) only think for you to a certain extent :) Ah well, no sense beating a dead horse here... we all seem to be saying this <html:attribute> tag is a good idea, and I think that addresses the concerns of all parties. I think there's even some interest in realizing it in 1.4... I 'll get by my current situation by subclassing I think, but the new tag gives a more general solution, which is what I was trying to provide, so that sounds good long-term. > Struts HTML taglib's select element doesn't support proper onfocus events in > IE7 > -------------------------------------------------------------------------------- > > Key: STR-3006 > URL: https://issues.apache.org/struts/browse/STR-3006 > Project: Struts 1 > Issue Type: Bug > Components: Taglibs > Affects Versions: 1.0 Final, 1.0.1, 1.0.1 Final, 1.0.2, 1.1.1, 1.2.2, > 1.2.4, 1.2.6 Beta, 1.2.7, 1.2.8, 1.2.9, 1.3.5, 1.3.6, 1.3.7, 1.4.0 > Environment: Any > Reporter: Frank W. Zammetti > Priority: Minor > Attachments: BaseHandlerTag.java, BaseHandlerTag.java.diff, > struts-attr-tag-v2.patch, struts-html.tld, struts-html.tld.diff > > > There is a known issue with IE7 and the plain <select> input element... let's > say for example you have this: > <html> > <head> > <title></title> > </head> > <body> > <select onFocus="this.style.backgroundColor='#ff0000';"> > <option value="1">1</option> > <option value="2">2</option> > </select> > </body> > </html> > In IE7, when you click on the dropdown arrow, the list of items will not > immediately show up, you need to click it twice. This is due to the onFocus > handler being attached. This is a known issue with IE. The solution is to > simply add an onFocusIn handler which (usually) mimics the onFocus handler. > However, in Struts 1, because the HTML taglib does not allow arbitrary > attributes (something I believe it should, and I said as much on a BugZilla > ticket which I can't seem to find in JIRA now, and along with it I suggested > adding a strictHTML attribute, which would default to true, to allow (false) > or disallow (true) arbitrary attributes), you cannot properly deal with this > issue at present if you use the HTML taglib. > Attached please find two updated files, BaseHandlerTag.java and > struts-html.tld, which adds support for the onfocusin attribute. > I marked this as Minor, but for those of us dealing with this issue right > now, it's more like a Major :) I also consider it a bug since I don't see > any viable work-around (someone can correct me if I'm wrong... and yes, not > using the HTML taglib is I suppose a valid answer, but I'd be looking for > something else) -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.